Работает, но не принимается валидатором как минимум по пунктам 3, 5
Ирина Савенкова
20 уровень
Работает, но не принимается валидатором как минимум по пунктам 3, 5
Решен
Комментарии (12)
- популярные
- новые
- старые
Для того, чтобы оставить комментарий Вы должны авторизоваться
Сергеев ВикторMaster
15 марта 2018, 20:08
нет, не работает
И не используйте статические переменные
0
Ирина Савенкова
15 марта 2018, 21:13
Последовала вашему совету. Теперь код обрабатывает предложенный вами случай, статическую переменную перенесла в main в качестве локальной. Ответ от валидатора прежний.
0
Сергеев ВикторMaster
15 марта 2018, 21:46
потому, что ваш код все так же не работает как надо.
Создайте файлы из моего примера, заполните их тестовыми данными и посмотрите на результат.
"Если вы используете регулярные выражения для решения проблемы, то теперь у вас 2 проблемы"
0
Ирина Савенкова
15 марта 2018, 22:00
0
Ирина Савенкова
15 марта 2018, 22:00
Перепроверила, оказалось, что неверно сортировал из-за того, что я забыла, что строки сортируются в алфавитном порядке. :) Но это я уже исправила, а валидатор все так же недоволен. П.С. Вроде регулярки норм работают, и у меня нет идей, как без них.
0
Ирина Савенкова
15 марта 2018, 23:51
0
Ирина Савенкова
15 марта 2018, 23:52
Немного разделила функциональность, чтобы оптимизировать работу с памятью и облегчить чтение кода, но проблема с валидатором остается.
0
Roman Java Developer Expert
16 марта 2018, 14:13решение
Что происходит в этой строке?
+4
Ирина Савенкова
16 марта 2018, 14:32
Жаль, нельзя поставить больше плюсов! Роман, спасибо!!! Слона-то я и не заметила.))
Кстати, если кому-то интересно будет, мой код с этой правкой (и другими, которые я делала в надежде пройти валидатор) теперь выглядит так (см. след. сообщение).
0
Roman Java Developer Expert
16 марта 2018, 14:37
А вот это уже лишнее - выкладывать правильные решения запрещено.
0
Ирина Савенкова
16 марта 2018, 14:46
Жалко, придется удалить. :(
Опишу тогда так изменения: в вызов конструктора TreeSet добавлен анонимный компаратор по строкам (после чего idea сама предложила использовать лямбды, что и было сделано), использованы Buffered Output/Input Stream'ы вместо буффера byte[] bytesFromPart. А регулярки для получения имени объединенного файла заменены на fileName.substring(0, fileName.lastIndexOf(".")).
0
Roman Java Developer Expert
16 марта 2018, 14:48
Твое старое решение, которое было выложено перед моим комментом тоже прошло бы проверку, если из него убрать ту строку, о которой я написал.
0