Зачем тебе первый if, если судя по подсказке continue не будет вызвано? Тесты лучше как-то называть, чтобы было понятно, что там просходит, также можно сделать негативные сценарии
Насчёт if'ов, то это для того, чтобы '\n' не шёл в else, а сразу отсеялась. Иначе она попадёт в else, и будет добавлена в sb.
Плюс у меня установлен Sonarlint, и она любит выдавать подобные предупреждения.
Ошибается он довольно редко, надо заметить.
Сейчас попробую поправить.
Сонарлинт не говорит, что надо убирать ветку. Он говорит, что в ветке не нужен continue
Теперь Идея ругается о том, что if пустой.
Но зато тесты проходит, и это уже хорошо.
Этот код можно переписать несколькими способами :)
У тебя в логике ошибка как минимум, что будет если входная строка {3} в твоём варианте и варианте с реплейс?
Вот это я даже и не знаю, как делать правильно. В этом и проблема.
Если у тебя условие попадает в if, он уже не войдет в else
Исправил. Тесты проходят. Код: https://pastebin.com/nd3DxPWT Тесты: https://pastebin.com/tsUdqpuh
Обсуждают сегодня