Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

How to remove Checkstyle info (wrong order for import org.apache.log4j.Logger)

I can see a Checkstyle info which says - Wrong order for import, org.apache.log4j.Logger. I could not get much info on why I am getting this. Any help would be appreciated. Below is the code snippet:

import jxl.Cell;
import jxl.Sheet;
import jxl.Workbook;
import jxl.read.biff.BiffException;
import jxl.write.Label;
import jxl.write.WritableCell;
import jxl.write.WritableSheet;
import jxl.write.WritableWorkbook;
import jxl.write.WriteException;
import jxl.write.biff.RowsExceededException;

import org.apache.log4j.Logger;

import com.company.department.team.test.Configuration;
like image 517
Saikat Avatar asked Jan 31 '14 10:01

Saikat


2 Answers

ctrl+shift+o (organize imports) will make Eclipse order your imports correctly.

There is a convention according to which imports should be ordered, and checkstyle is telling you that you have not listed your imports in that order.

You can read more about it in the ImportOrder section of the documentation:

Checks the ordering/grouping of imports. Features are:

  • groups imports: ensures that groups of imports come in a specific order (e.g., java. comes first, javax. comes second, then everything else)
  • adds a separation between groups : ensures that a blank line sit between each group
  • sorts imports inside each group: ensures that imports within each group are in lexicographic order
  • sorts according to case: ensures that the comparison between imports is case sensitive
  • groups static imports: ensures the relative order between regular imports and static imports (see import orders)
like image 70
Zoltán Avatar answered Oct 24 '22 15:10

Zoltán


You can also modify your check file to go by what eclipse does by default. You need to change the module "CustomImportOrder" and change "customImportOrderRules".

See http://checkstyle.sourceforge.net/config_imports.html#CustomImportOrder on how to customize it more.

This is what I am currently using:

<module name="CustomImportOrder">
    <property name="specialImportsRegExp" value="gov." />
    <property name="sortImportsInGroupAlphabetically" value="true" />
    <property name="customImportOrderRules"
        value="STATIC###SPECIAL_IMPORTS###STANDARD_JAVA_PACKAGE###THIRD_PARTY_PACKAGE" />
</module>
like image 37
rveach Avatar answered Oct 24 '22 15:10

rveach