Модераторы: LSD, AntonSaburov
  

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Задачи на числа. Решение. Покритикуйте. (часть №1) 
:(
    Опции темы
m1st
  Дата 15.3.2012, 15:05 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 73
Регистрация: 1.11.2006

Репутация: нет
Всего: нет
Цитата

Ввести с консоли n целых чисел и поместить их в массив. На консоль вывести: 
1.    Четные и нечетные числа.
2.    Наибольшее и наименьшее число.
3.    Числа, которые делятся на 3 или на 9.
4.    Числа, которые делятся на 5 и на 7.
5.    Элементы, расположенные методом пузырька по убыванию модулей.
6.    Все трехзначные числа, в десятичной записи которых нет одинаковых цифр.
7.    Наибольший общий делитель и наименьшее общее кратное этих чисел.
8.    Простые числа.
9.    Отсортированные числа в порядке возрастания и убывания.
10.    Числа в порядке  убывания частоты встречаемости чисел.
11.    “Счастливые” числа. 
12.    Числа Фибоначчи: f0 = f1 = 1, f (n) = f (n–1) + f (n–2).
13.    Числа-палиндромы, значения которых в прямом и обратном порядке совпадают.
14.    Элементы, которые равны полусумме соседних эле¬ментов.
15.    Период десятичной дроби p = m/n для первых двух целых положительных чисел n и m, расположенных подряд. 
16.    Построить треугольник Паскаля для первого положительного числа. 


Привет, решение данных заданий на Java - ниже. Покритикуйте.

Код наиболее последней версии тут: http://ideone.com/a72cO

Это сообщение отредактировал(а) m1st - 15.4.2012, 06:57
PM MAIL   Вверх
Samotnik
Дата 15.3.2012, 21:32 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Super star !
****


Профиль
Группа: Awaiting Authorisation
Сообщений: 7192
Регистрация: 4.11.2006
Где: Минск City

Репутация: 8
Всего: 191
m1st, что именно критиковать? ООП тут нет, паттернов тоже. 300 строк кода алгоритмов и условий врядли кто-то смотреть будет. Работает - и хорошо.  smile 
Если вызывает сомнение какой-то конкретный алгоритм то его и укажи. 

Праметризируй Map, Comparator.
PM MAIL   Вверх
m1st
Дата 16.3.2012, 08:38 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 73
Регистрация: 1.11.2006

Репутация: нет
Всего: нет
Цитата(Samotnik @  15.3.2012,  21:32 Найти цитируемый пост)
m1st, что именно критиковать?
Все, к чему душа лежит.

Цитата(Samotnik @  15.3.2012,  21:32 Найти цитируемый пост)
ООП тут нет
Как нет? поясните...

Цитата(Samotnik @  15.3.2012,  21:32 Найти цитируемый пост)
300 строк кода алгоритмов и условий врядли кто-то смотреть будет. Работает - и хорошо.   Если вызывает сомнение какой-то конкретный алгоритм то его и укажи. 
Обьем да, но на мой взгляд - читается как стихи.
Не сомнение движет мною, а исключительно тяга к познанию. Желание написать код лучше. ж)
PM MAIL   Вверх
jk1
Дата 16.3.2012, 08:45 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Участник
Сообщений: 1168
Регистрация: 17.10.2008
Где: Санкт-Петербург

Репутация: 40
Всего: 75
Цитата

Обьем да, но на мой взгляд - читается как стихи.


Нееет, от хорошо читаемого кода это далековато. Читаемый код надо хотя бы минимально документировать.
Вот эта конструкция например нечитабельна совершенно
Код

if (num.length() == 4
                    && num.charAt(0) + num.charAt(1) == num.charAt(2)
                            + num.charAt(3))

о её назначении можно догадаться только из-за элементарности задачи.


--------------------
Opinions are like assholes — everybody has one
PM MAIL   Вверх
m1st
Дата 16.3.2012, 09:48 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 73
Регистрация: 1.11.2006

Репутация: нет
Всего: нет
jk1, согласен, когда говорил о читаемости и имел ввиду "элементарность задачи".
PM MAIL   Вверх
m1st
Дата 20.3.2012, 14:04 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 73
Регистрация: 1.11.2006

Репутация: нет
Всего: нет
Сделал рефактор почти всего кода. 

Это сообщение отредактировал(а) m1st - 22.3.2012, 18:00
PM MAIL   Вверх
m1st
Дата 22.3.2012, 17:59 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 73
Регистрация: 1.11.2006

Репутация: нет
Всего: нет
Добавил в первый пост свежую версию программы.
PM MAIL   Вверх
m1st
Дата 28.3.2012, 20:39 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 73
Регистрация: 1.11.2006

Репутация: нет
Всего: нет
Исправленная и улучшенная версия добавлена в первое сообщение.
PM MAIL   Вверх
m1st
Дата 10.4.2012, 16:51 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 73
Регистрация: 1.11.2006

Репутация: нет
Всего: нет
Исправленная и улучшенная версия добавлена в первое сообщение.
PM MAIL   Вверх
m1st
Дата 15.4.2012, 06:58 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 73
Регистрация: 1.11.2006

Репутация: нет
Всего: нет
Код наиболее последней версии переехал сюда: http://ideone.com/a72cO
PM MAIL   Вверх
  
 Ответ в темуСоздание новой темы Создание опроса
Правила форума "Java"
LSD   AntonSaburov
powerOn   tux
javastic
  • Прежде, чем задать вопрос, прочтите это!
  • Книги по Java собираются здесь.
  • Документация и ресурсы по Java находятся здесь.
  • Используйте теги [code=java][/code] для подсветки кода. Используйтe чекбокс "транслит", если у Вас нет русских шрифтов.
  • Помечайте свой вопрос как решённый, если на него получен ответ. Ссылка "Пометить как решённый" находится над первым постом.
  • Действия модераторов можно обсудить здесь.
  • FAQ раздела лежит здесь.

Если Вам помогли, и атмосфера форума Вам понравилась, то заходите к нам чаще! С уважением, LSD, AntonSaburov, powerOn, tux, javastic.
 
1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | Java: Общие вопросы | Следующая тема »

Подписаться на тему | Подписка на этот форум | Скачать/Распечатать тему


 




[ Время генерации скрипта: 0.0732 ]   [ Использовано запросов: 21 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
 По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.