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

Saikat picture Saikat · Jan 31, 2014 · Viewed 20.5k times · Source

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;

Answer

Zoltán picture Zoltán · Jan 31, 2014

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)