vlad0058 27 дек 2012 в 06:18

Погодник на java для начинающих и постарше

10 мин

21K

Проектирование API * Java *

Из песочницы

Комментарии 15

matiouchkine 27 дек 2012 в 06:33

    String imgType;
    String humidity;// Влажность
    String tom, tomNight;

Это очень трогательно ;-)
Хер знает, кто такой imgType, не говоря о tom и tomNight. Но комментарий оказался у очевидной хумидити.

Я вам добавил кармы, пишите, у вас неплохо получается. Но ставьте комментарии либо нигде (нидзя-стайл), либо везде (профешнл).

vlad0058 27 дек 2012 в 07:01

спасибо, понял, imgType — я кажется прокомментировал (номер картинки), а вот про tom, tomNight забыл (завтра, завтраВечер («tommorow», «tommorowNight»))

medved13 27 дек 2012 в 09:58

А почему переменные tom и tomNight, а не tomorrow и tomorrowNight? У вас места на диске под исходники не хватает? Или автодополнение в ИДЕ сломалось? Всё-таки, Java это не JavaScript, там не надо экономить на байтах. А self-commented названия переменных очень хороший стиль, имхо.

vlad0058 27 дек 2012 в 11:51

нет, я просто не любитель писать переменные с длинными названиями

KREGI 27 дек 2012 в 07:01

Приложите так же исходники в архиве, пожалуйста.
Вам нравится Swing, awt? Мне вот лично больше понравилось работать с JavaFX, может Вам тоже будет это интересней…

vlad0058 27 дек 2012 в 07:02

сейчас добавлю

Nuru 27 дек 2012 в 08:33

Все это конечно круто, но зачем извращаться с ручным проходом по дом дереву.

Вы когда — то слышали про jSoup или Jodd Jerry?

vlad0058 27 дек 2012 в 11:09

Теперь слышал, но все же я хотел обойтись без внешних библиотек.
А вообще спасибо, почитал, ознакомился, если кому еще интересно:
jsoup.org/cookbook/introduction/parsing-a-document
jodd.org/doc/jerry/index.html

Nuru 27 дек 2012 в 11:37

Неужели это проблема? Если говорить про Jerry — это очень легковестный анализатор дерева который дает ва jQuery style выборки.

ИМХО, для этого библиотеки и пишут, чтобы люди свои велосипеды не изобретали)

Verneal 27 дек 2012 в 11:22

Краткое имхо после скачиваниея архива на народе и беглого просмотра проекта =)

Настоятельно рекомендую сделать в эклипсе кодировку по умолчанию UTF-8. Сейчас у вас Cp1251.
(Window->Preferences. Потом в поисковой строке введите encoding, а дальше заменить везде Cp1251 на UTF-8).

Комменты на русском — лучше бы их не было. А если очень надо, тогда уж на английском.

Хардкод желательно бы убрать. btn = new JButton(«Хочу все знать»); — никуда не годится. Используйте файлы *.properties. Ну или на крайняк, вынесите это в private stаtic final String поля.

Кроме того, хотелось бы классы по пакетам разложить. А то в дефолтном некошерно…