169 похожих чатов

Что можно улучшить? Пробовал через array, но там надо юзать

copyOf для того, чтобы избавиться от \u0000 в конце. Плюс надо иметь дополнительное поле count для того, чтобы символы нормально добавлялись в массив.

15 ответов

6 просмотров
Egor-Gavrilov Автор вопроса
Egor Gavrilov
screenshot

Зачем тебе первый if, если судя по подсказке continue не будет вызвано? Тесты лучше как-то называть, чтобы было понятно, что там просходит, также можно сделать негативные сценарии

Egor-Gavrilov Автор вопроса
Михаил
Зачем тебе первый if, если судя по подсказке conti...

Насчёт if'ов, то это для того, чтобы '\n' не шёл в else, а сразу отсеялась. Иначе она попадёт в else, и будет добавлена в sb.

Egor-Gavrilov Автор вопроса
Михаил
Зачем тебе первый if, если судя по подсказке conti...

Плюс у меня установлен Sonarlint, и она любит выдавать подобные предупреждения.

Egor Gavrilov
Плюс у меня установлен Sonarlint, и она любит выда...

Ошибается он довольно редко, надо заметить.

Egor-Gavrilov Автор вопроса
Egor-Gavrilov Автор вопроса
Egor Gavrilov
Сейчас попробую поправить.

Вот что происходит, если убрать эту ветку.

Egor Gavrilov
screenshot Вот что происходит, если убрать эту ветку.

Сонарлинт не говорит, что надо убирать ветку. Он говорит, что в ветке не нужен continue

Egor-Gavrilov Автор вопроса
Nikita Fedkin
Сонарлинт не говорит, что надо убирать ветку. Он г...

Теперь Идея ругается о том, что if пустой.

Egor-Gavrilov Автор вопроса
Egor Gavrilov
Теперь Идея ругается о том, что if пустой.

Но зато тесты проходит, и это уже хорошо.

Egor Gavrilov
Теперь Идея ругается о том, что if пустой.

Этот код можно переписать несколькими способами :)

Egor Gavrilov
screenshot

У тебя в логике ошибка как минимум, что будет если входная строка {3} в твоём варианте и варианте с реплейс?

Egor-Gavrilov Автор вопроса
Олег Гуреев
У тебя в логике ошибка как минимум, что будет если...

Вот это я даже и не знаю, как делать правильно. В этом и проблема.

Egor Gavrilov
Насчёт if'ов, то это для того, чтобы '\n' не шёл в...

Если у тебя условие попадает в if, он уже не войдет в else

Egor-Gavrilov Автор вопроса
Egor Gavrilov
Вот это я даже и не знаю, как делать правильно. В ...

Исправил. Тесты проходят. Код: https://pastebin.com/nd3DxPWT Тесты: https://pastebin.com/tsUdqpuh

Похожие вопросы

Обсуждают сегодня

Если у меня есть такой класс: Object = {} function Object:new(a_name, a_transform, a_color, a_mesh, a_material, a_shader, a_textures) local private = {} private.n...
Cuarno Vile
4
я не магистр хаскеля, но разве не может лейзи тип конвертнуться в не-лейзи запросив вычисление содержимого прям при инициализации?
deadgnom32 λ madao
100
зачем же переименовывать ? чтобы кол-во участников возросло или вдруг IBM от этого снова на свифте начнет кодить ? Я не понимаю что страшного в том что свифт гавно, если это т...
Oleh Nerzh
10
здравствуйте. совершаю вот такую вещь: strcpy(line, (char)current_number); где current number — неподписанный шорт, line — массив чаров. ругань следующая: main.c:29:30: error...
Roberto's Ширгозиев
13
@MrMiscipitlick А можешь макрос написать, который будет вычислять смещение относительно переданных меток? Просто .label1-.label2, и вернуть значение.
КТ315
35
А еще в перле можно уже @arr1 + @arr2?
Sergei Zhmylove
53
@NikaBelurgHR Бот? Ассемблер?
КТ315
11
юзеры Jetpack Compose тут?
Qtless Qtless
7
Добрый день! Подскажите, пожалуйста: какими компетенциями нужно обладать, чтобы претендовать на работу эрланг (отдельная благодарность, если про элексир тоже подскажете) разр...
via ☸️ led
20
Привет всем. появился вопрос. Разрабатываю сайт, в данный момент он запущен. Хостинг beget. Добавляю на сайт яндекс метрику с помощью полей client-settings (взято отсюда http...
Andrew
2
Карта сайта