Java кода
Материал из BiTel WiKi
(Различия между версиями)
Stark (Обсуждение | вклад) |
Stark (Обсуждение | вклад) |
||
| (1 промежуточная версия не показана) | |||
| Строка 1: | Строка 1: | ||
Для сверки правильности форматирования, проверки отступов и т.п. следует использовать форматер Eclipse. Текущая версия: [[Медиа:bitel_2014.xml]] | Для сверки правильности форматирования, проверки отступов и т.п. следует использовать форматер Eclipse. Текущая версия: [[Медиа:bitel_2014.xml]] | ||
| - | |||
{| border="1" cellpadding="2" cellspacing="0" | {| border="1" cellpadding="2" cellspacing="0" | ||
| Строка 108: | Строка 107: | ||
<!-- "немые" return ы и ошибки --> | <!-- "немые" return ы и ошибки --> | ||
| + | |||
| + | Старая версия(оставили для истории): [[Медиа:formater.xml]] | ||
Текущая версия на 12:02, 30 июля 2014
Для сверки правильности форматирования, проверки отступов и т.п. следует использовать форматер 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
