|
|
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
всем привет гляньте на код = может он слишком избыточен, есть ошибки кодинга или оформления кода? Код: java 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29. 30. 31. 32. 33. 34. 35. заранее благодарен ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 09:36:54 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
На счет оформления кода: откройте Netbeans или другую IDE и, щелкнув правой кнопкой по коду, выберите Format. или вы в блокноте все пишите? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:17:15 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
почему метод form() у вас ничего не возвращает? и нет его характеристики относительно доступности для других классов. должно быть примерно так: Код: java 1. 2. 3. 4. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:21:03 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
Nixicпочему метод form() у вас ничего не возвращает? и нет его характеристики относительно доступности для других классов потому что это конструктор в котором инициализируется что-то ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:28:14 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
переменные можно, конечно, и так объявить как у вас Код: java 1. но лучше каждую отдельно и если вам не надо их использовать в других классах, сделать из private. Код: java 1. 2. 3. 4. 5. 6. Можно вообще модификатор доступа не указывать, тогда переменные становятся package-private, как я понял, доступны в рамках пакета, где класс лежит.. Но никогда такое не использовал. Подробнее здесь . ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:28:29 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
buyboyNixicпочему метод form() у вас ничего не возвращает? и нет его характеристики относительно доступности для других классов потому что это конструктор в котором инициализируется что-то Тогда имя класса (даже не тогда, а всегда) должно быть с большой буквы. И конструктора - соответственно тоже. Методы называют с маленькой буквы, я поэтому и подумал что это метод, не смотря на имя класса. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:30:36 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
у конструктора, кстати, тоже должен быть модификатор доступа ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:33:18 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
После extends я бы написал сразу JFrame, т.е. без пути. "javax.swing." и вынес бы в импорт import javax.swing.JFrame; ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:36:06 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
buyboy, жесть Код: java 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29. 30. 31. 32. 33. 34. 35. 36. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:40:19 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
Nixicно лучше каждую отдельно так нагляднее - согласен, но захотелось сократить количество строк ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:46:10 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
NixicПосле extends я бы написал сразу JFrame, т.е. без пути. "javax.swing." и вынес бы в импорт ценно! спасибо! ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:46:59 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
Не все так страшно ))) 1. Имена классов называют с большой буквы. 2. Методы класса называют с маленькой буквы. так принято. 3. Поля классов делают private, а обращаются к ним из других классов через геттеры и сеттеры. 4. У конструктора должен стоять модификатор public, а имя точно такое же как у класса. 5. Переменные объявляйте каждую отдельно, так нагляднее и проще удалить (всю строку) в случае ненадобности. В принципе это я итоги подвел )) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:48:46 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
NixicНе все так страшно ))) мне побыть ламером не хочется - но придется ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:51:52 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
Nixic, Nixic4. У конструктора должен стоять модификатор public, Не должен, а может. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 10:53:31 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
а что касается взаиморасположения блоков объявления полей, конструктора и главного метода? если объявдяется много переменных и/или конструктор получается громоздким - может все это можно располагать в нижней части класса? ну чтобы не скролить каждый раз до рабочего метода если строк кода слишком много? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 12:03:44 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
buyboy, а смысл? любая IDE имеет тот или иной навигатор по методам, скроллить никуда не надо, и main такой же метод как все остальные. Не загоняйтесь по форме пока, загоняйтесь по содержанию. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 12:07:09 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
buyboy, Вам очень не хватает вот этого http://www.oracle.com/technetwork/java/codeconv-138413.html Так же очень рекомендую вот этот сайт http://www.javapractices.com/home/HomeAction.do И вот эту книгу http://www.ozon.ru/context/detail/id/1425895/ Для быстрого старта самое то. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 12:09:58 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
Blazkowicz, что-то вы сегодня всем это советуете. Супчик дня?)) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 12:11:12 |
|
||
|
какие ошибки допущены?
|
|||
|---|---|---|---|
|
#18+
buyboyа что касается взаиморасположения блоков объявления полей, конструктора и главного метода? если объявдяется много переменных и/или конструктор получается громоздким - может все это можно располагать в нижней части класса? ну чтобы не скролить каждый раз до рабочего метода если строк кода слишком много? пофиг-веники, могу ошибаться, но я всегда делаю так: класс { переменные. конструктор. методы. } Можно все в конструктор пихать, а можно в отдельный метод, но после объявления его исполнять (мне так больше нравится). Так что расположение не особо важно. Кроме того, есть у меня классы почти по 3 тыс строк (интерфейс там нагроможденный сильно), так что там без разницы, так и так приходится скролить, а иногда даже пейдждаунить, а иногда даже использовать Navigator по классу! ))) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.08.2012, 12:12:43 |
|
||
|
|

start [/forum/topic.php?fid=59&msg=37918894&tid=2131132]: |
0ms |
get settings: |
19ms |
get forum list: |
24ms |
check forum access: |
7ms |
check topic access: |
7ms |
track hit: |
38ms |
get topic data: |
19ms |
get forum data: |
5ms |
get page messages: |
82ms |
get tp. blocked users: |
2ms |
| others: | 321ms |
| total: | 524ms |

| 0 / 0 |
