vlad0058 Dec 27 2012 at 10:18

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

10 min

20K

Java*API*

From sandbox

Comments 15

matiouchkine Dec 27 2012 at 10:33

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

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

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

vlad0058 Dec 27 2012 at 11:01

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

medved13 Dec 27 2012 at 13:58

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

vlad0058 Dec 27 2012 at 15:51

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

-1

KREGI Dec 27 2012 at 11:01

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

vlad0058 Dec 27 2012 at 11:02

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

Nuru Dec 27 2012 at 12:33

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

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

vlad0058 Dec 27 2012 at 15:09

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

Nuru Dec 27 2012 at 15:37

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

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

Verneal Dec 27 2012 at 15:22

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

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

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

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

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