|
233 / 130 / 27
Регистрация: 24.08.2016
Сообщений: 875
|
||||||||||||||||
Покритикуйте, пожалуйста14.10.2019, 07:38. Показов 2421. Ответов 7
Метки нет (Все метки)
Написала свой пермутатор. Мне нравится. Но понимаю, конечно, что найдется много разных косяков. Большая просьба к опытным программистам, если не лень, одним глазком взглянуть и потыкать меня носом в ошибки.
AlicesPermutator
Test
Результаты теста
Есть три вопроса: 1. Как написать конструктор, чтобы принимал на входе enum? 2. Можно ли написать конструктор, принимающий на входе переменное число аргументов? 3. В 53 строке IDEA ругается на кастинг из Object[] в обобщенный массив. Это как-то лечится, или лучше оставить так?
0
|
||||||||||||||||
| 14.10.2019, 07:38 | |
|
Ответы с готовыми решениями:
7
Покритикуйте новичка Покритикуйте код Многопоточность. Покритикуйте |
|
53 / 27 / 9
Регистрация: 04.05.2013
Сообщений: 83
|
||||||||||||||||||||||||||
| 14.10.2019, 10:36 | ||||||||||||||||||||||||||
Сообщение было отмечено alicesmagic как решение
Решение
Переменную source можно сделать более общим типом
В методах permutation, levelUp, getAll, getNext и toColumnString можно изменить тип переменных, возвращаемых значений и параметров методов на более общий. Например,
В методах getNext и toColumnString можно выкинуть данную строку и использовать результат вызова getAll напрямую, без копирования в памяти, так как aal далее не меняется.
Для тестов лучше использовать библиотеку JUnit, разнести тесты по разным методам и добавить проверку корректности работы класса.
1
|
||||||||||||||||||||||||||
|
233 / 130 / 27
Регистрация: 24.08.2016
Сообщений: 875
|
|
| 14.10.2019, 12:25 [ТС] | |
|
Ого, сколько ценных замечаний! Спасибо большое, Thousbe, буду потихоньку доводить до ума свой "шедевр"
0
|
|
|
|
|
| 16.10.2019, 11:13 | |
Сообщение было отмечено alicesmagic как решение
Решение
alicesmagic, давайте пробежимся.
Ошибка №1 общеидеологическая, фундаментальная. Вы строите и храните перестановки физически, вместо индексного построения. Соответственно у вас куча лишних по сути действий. Ошибка №2 архитектурная, частично следует из №1. Надо отметить, что вы почти подошли самостоятельно к ее исправлению. Вы не смогли продумать и ответить четко на вопрос "а что должен делать класс?". В итоге он у вас и генерит вообще все перестановки, и выдает поочередно и форматирует в строку. Это плохо. Хорошо то, что в процессе написания тестов, вы сделали getNext и hasNext. Но вы не довели мысль до конца и не пришли к выводу, что в таком случае ваш класс должен быть собственно итератором. Чувствуете, как от этого все упрощается? Ошибка №3 общее программирование. Бизнес-логика в конструкторе -- плохо! У вас вообще по сути вся логика там. Замечание №1 общее. Код плохо читаем и ввиду отсутствия комментариев плохо понятен по сути. Как то: - что такое levelUp в контексте перестановок? Какой физический смысл?- если метод getAll выдает все перестановки, то почему в виде HashSet<ArrayList<T>> когда одна перестановка выдается в виде T[]?Для ознакомления гляньте, как сделано тут: Комбинаторика с использованием PermutationIterator Там не идеально, но полезные вещи для себя думаю найдете.
3
|
|
|
233 / 130 / 27
Регистрация: 24.08.2016
Сообщений: 875
|
|
| 16.10.2019, 12:49 [ТС] | |
|
HighPredator, спасибо! Попробую разобраться с этим. Вообще, я еще только начала изучать коллекции, и об итераторах имею самое поверхностное представление, но, думаю, ваш пример будет мне хорошим подспорьем.
![]() levelUp - получая на входе список-полуфабрикат и очередной символ, поднимает список на следующий уровень. И так, пока не обработает все символы из исходного списка.Вот по такой схеме: [] -> [A] -> [[А, В], [В, А]] -> [[А, В, C], [C, В, А], [В, C, А], [А, C, В], [В, А, C], [C, А, В]] и т.д.
0
|
|
|
|
|
| 16.10.2019, 16:14 | |
|
alicesmagic, не про
levelUp я то разобрался. Мысль была в том, что вы ввели термин (уровень) и направление (вверх) формально не имеющие отношения собственно перестановкам как к кортежам из кобинаторики. Кстати, обратите внимание, вы когда стали мне объяснять, вы выбрали линейную схему и из нее ни уровень, ни направление верха не понятно. Как следствие у людей возникнет еще больше вопросов.
1
|
|
|
233 / 130 / 27
Регистрация: 24.08.2016
Сообщений: 875
|
|
| 16.10.2019, 17:10 [ТС] | |
|
HighPredator, ну я с комбинаторикой дела не имела до сих пор и с ее терминологией совсем не знакома. Когда выбирала имя метода, то представляла себе level, как текущую длину вложенных списков, а up, как повышение этого уровня на единицу. Мне тогда показалось, что такое название хорошо отражает суть. Но, мало ли что мне кажется))
0
|
|
|
|
||
| 17.10.2019, 07:57 | ||
|
1
|
||
| 17.10.2019, 07:57 | |
|
Помогаю со студенческими работами здесь
8
Алгоритм: Трамвайные билеты. Покритикуйте Задачи на числа. Решение. Покритикуйте. (часть №1) Задачи на строки и числа. Решение. Покритикуйте Покритикуйте код моего сокет сервера для игрового чата Покритикуйте пожалуйста резюме Junior Java Developer Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |
|
Новые блоги и статьи
|
|||
|
Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT
bedvit 28.01.2026
Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования.
Часть библиотеки BedvitCOM
Использованы. . .
|
Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 28.01.2026
Содержание блога
SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .
|
Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL3_image
8Observer8 27.01.2026
Содержание блога
SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .
|
влияние грибов на сукцессию
anaschu 26.01.2026
Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .
|
|
Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android
8Observer8 26.01.2026
Содержание блога
SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .
|
Установка Android SDK, NDK, JDK, CMake и т.д.
8Observer8 25.01.2026
Содержание блога
Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip"
Извлеките архив и вы увидите. . .
|
Вывод текста со шрифтом TTF на Android с помощью библиотеки SDL3_ttf
8Observer8 25.01.2026
Содержание блога
Если у вас не установлены Android SDK, NDK, JDK, и т. д. то сделайте это по следующей инструкции: Установка Android SDK, NDK, JDK, CMake и т. д.
Сборка примера
Скачайте. . .
|
Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly
8Observer8 24.01.2026
Содержание блога
Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
|