Java кода

Материал из BiTel WiKi

Версия от 12:02, 30 июля 2014; Stark (Обсуждение | вклад)
(разн.) ← Предыдущая | Текущая версия (разн.) | Следующая → (разн.)
Перейти к: навигация, поиск

Для сверки правильности форматирования, проверки отступов и т.п. следует использовать форматер Eclipse. Текущая версия: Медиа:bitel_2014.xml

НЕ ПРАВИЛЬНО ПРАВИЛЬНО
Не допускаются фрагменты кода после условного оператора, в операторе switch и т.п. не заключенные к двойные скобки.
if( pathDepth > 0 )
   pathDepth--;
if( pathDepth > 0 )
{
   pathDepth--;
}
Не допускаются два и более пробелов между фрагментами кода.
public void auth( DBInfo dbInfo )
{
	UserAuth userAuthDialog = new UserAuth( this, setup );
	if ( dbInfo != null)
	{
		System.out.println( dbInfo.getDbServerTitle() );
		System.out.println( dbInfo.getDBServerTitle() );
 
 
 
		userAuthDialog.setDbInfo( dbInfo );
		userAuthDialog.setSelectedDB( dbInfo.getDbServerKey() );
	...
public void auth( DBInfo dbInfo )
{
	UserAuth userAuthDialog = new UserAuth( this, setup );
	if ( dbInfo != null)
	{
		System.out.println( dbInfo.getDbServerTitle() );
		System.out.println( dbInfo.getDBServerTitle() );
 
		userAuthDialog.setDbInfo( dbInfo );
		userAuthDialog.setSelectedDB( dbInfo.getDbServerKey() );
	.....
По возможности используйте конструкторы с параметрами, для сокращения и повышения читаемости кода, если строка не становится слишком длинной.
JPanel jPanel1 = new JPanel();
jPanel1.setLayout( new GridBagLayout() );
JPanel jPanel1 = new JPanel( new GridBagLayout() );
Избегайте лишних переменных, которые в дальнейшем нигде не будут использованы.
JScrollPane scrollPane = new JScrollPane( xmlText ); 
xmlPanel.add( scrollPane, ...
xmlPanel.add( new JScrollPane( xmlText ), ...
Избегайте функций с большим количеством аргументов, более 6. В случае необходимости передачи такого количества параметров целесообразно создание класса-структуры, с public полями. Помимо улучшения читаемости кода данный метод упрощает последующий рефакторинг.
public PaymentRegister loadRegister( Connection con, Date date, String title, byte[] file, StringBuilder loadLog.. )
{
     ...
public class PaymentLoadData
{
    public Date date;
    public String title;
    public byte[] file;
    public StringBuilder loadLog = new StringBuilder( 1000 );
    public int lineCount;
    public int okCount;
    public int commentCount;
    public int errorCount;
    public BigDecimal summa = BigDecimal.ZERO;
}
...
public PaymentRegister loadRegister( Connection con, PaymentLoadData data )
{
     ...


Старая версия(оставили для истории): Медиа:formater.xml

Личные инструменты