This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Almost always when I wan't to format copy&paste code, code formater will do mess in it or won't format at all. Sample code: class ColorTable extends JTable { boolean newitem[][]; ColorTable(int row, int col) { super(row,col); newitem= new boolean[row][col]; } public void setValueAt(Object aValue, int row, int column,boolean isnew) { super.setValueAt(aValue,row,column); newitem[row][column]=isnew; } public void updateUI() { super.updateUI(); int columnCount = getColumnCount(); TableColumnModel columnModel= getColumnModel(); for(int i=0; i<columnCount; i++) { columnModel.getColumn(i).setCellRenderer(new ColorCellRenderer()); } } } class ColorCellRenderer extends DefaultTableCellRenderer { public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) { super.getTableCellRendererComponent(table,value,isSelected,hasFocus,row,column); ColorTable ct=(ColorTable)table; if(ct.newitem[row][column]) { setBackground(Color.cyan); } else { setBackground(Color.white); } return this; } } Result: class ColorTable extends JTable { boolean newitem[][]; ColorTable(int row, int col) { super(row, col); newitem = new boolean[row][col]; } public void setValueAt(Object aValue, int row, int column, boolean isnew) { super.setValueAt(aValue, row, column); newitem[row][column] = isnew; } @Override public void updateUI() { super.updateUI(); int columnCount = getColumnCount(); TableColumnModel columnModel = getColumnModel(); for (int i = 0; i & lt; columnCount; i++ ) { columnModel.getColumn(i) .setCellRenderer ( new ColorCellRenderer()); } } } class ColorCellRenderer extends DefaultTableCellRenderer { @Override public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) { super.getTableCellRendererComponent(table,value,isSelected,hasFocus,row,column); ColorTable ct=(ResultsTable)table; if(ct.newitem[row][column]) { setBackground(Color.cyan); } else { setBackground(Color.white); } return this; } } -== What I've done? 1. Copy code from web page and paste to my project. 2. Select pasted code and use code formating from context menu.
The code has wrong syntax - there is < instead of < so the formatting which depends on syntax trees is broken. Maybe the formatting should not try to format incorrect code, but just indent it
I've tried to format also good code (none errors were found by netbeans), with this same results. In my point of view if mistakes don't mess with code structure formatter should do his work.
Can you please attach the correct code which is messed by formatter?
> I've tried to format also good code (none errors were found by netbeans), with this same results. Which code did you use then? This is the essential information that can help us to deal with some bug in formatting... I have fixed the < character in the sample code and it worked well. I agree this could be done a little smarter. Note: use of < instead of < messes with the code structure, as you suddenly replace token "relation operator" with a token "identifier" and automaton analyzing the code have (understandably) problems with it... Note that if you use unknown identifier in the code there an identifier should be, formatting works well...
Ok, i've done this once again (with original source code without this <) and it's work, but: I've got two classes, when there is mistake, formatter dosn't work at all, but should at least format class that's correc. The best thing will be if formatter won't format only metods that contains mistakes. I've changed Issue type.
To add one more detail to this formating issue, I have noticed that when adding the keyword "else" right after if's closing bracket, it goes to the beginning of the line when it is suppose to be below the closing bracket. See the situation below: @Override public void mousePressed(MouseEvent e) { if(segment && (e.getButton() == MouseEvent.BUTTON1)) { startDrag = new Point(e.getX(), e.getY()); endDrag = startDrag; repaint(); } else }