Всем привет.

Предыстория: Начал изучать Java по книжке "Самоучитель Java с примерами и программами" Васильев А.Н. Дошел до 4-ой главы "Управляющие инструкции" и решил не разбирать пример, как я делал это в предыдущих главах, а сначала самому попробовать написать программу, а уже потом посмотреть, как она написана в книге. Задача следующая: необходимо составить программу деления одного числа на другое, при этом учесть, что на "0" делить нельзя. 

Написал, проверил, программа работает. Вот листинг:


А вот листинг программы, написанной в самоучителе:


Вопрос: Подскажите, какой из этих кодов является правильным с точки зрения организации кода в java. Несмотря на то, что написанный мной код меньше и более понятен для меня (раз уж я его сам написал))) подозреваю, что он, все-таки, организован не лучшим образом, раз в книге код написан по-другому. Хотя, повторяю, код, написанный в книге мною воспринимается сложнее... Вопрос принципиальный для меня, поскольку хотелось бы понять общее направление - куда надо двигаться и на что обращать внимание, составляя структуру кода?

Оба варианта нормальные 

для такой маленькой учебной программки это не имеет большого значения, но вообще - разделяй и властвуй. один компонент - одна обязанность.
учебная задача состоит из трех подзадач:
1. ввести числитель и знаменатель
2. вычислить частное
3. вывести результат

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

Добавлено через 1 минуту и 13 секунд

чушь. не говоря уж что "одна из парадигм ООП" это что-то новенькое)))

Добавлено через 2 минуты и 54 секунды
Zadnica, паттерны и прочие подходы это всего лишь инструменты. их используют не ради использования, а когда они нужны.

Добавлено через 7 минут и 20 секунд
ЗЫ: если уж на то пошло, синглтон тут есть - это main. да-да, именно функция а не класс (который в известном смысле тоже синглтон, но не сам по себе, а постольку). если возникли сомнения на этот счет, прежде чем писать, перечитай вдумчиво

Не кормите тролля.

Спасибо за мнение.


Спасибо. Я тоже сразу обратил внимание, что у автора только один вызов вывода результата. Правда, я на момент написания не знал, что в условии if-else {} можно задать только одну переменную, без команды вывода, а команду написать уже после этого (P.S. общую для двух условий). В таком случае те три подзадачи, которые вы описали, выделить визуально можно, а вот что значит выделить отдельными функциями я не понимаю...


Это сообщение отредактировал(а) PrinceOFF - 21.10.2013, 15:24

было бы неплохо обработать также ситуацию, когда вместо числа вводится, например, буква:

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



все вместе



Это сообщение отредактировал(а) baldina - 21.10.2013, 18:38

Pawl, baldina, спасибо. Для меня это усложнение кода на данном этапе - я еще такого не проходил... (((
Буду разбираться с этими кодами, когда познакомплюсь с данными операторами и функциями.

Для самоучителя может и сойдет но в реальном проекте за такой код могут и побить 
1. Переменные должны иметь понятные имена, не x,y а numerator, denominator (ну или num, denom).
2. Комментарии в духе Капитана Очевидность только загромождают код и мешают его читать, удалить нафиг.
3. Скобки и переводы строки должны быть в одном стиле.
4. Переменные объявляй "по месту" не надо их объявлять заранее. Если значение переменной не меняется и используется один раз, то и переменная не нужна.

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