Вопрос к более квалифицированным программистам

Вопрос по задаче

Обсуждается

https://github.com/matvej2000/JavaRush/commit/4007ac5cbd638e45d4490b69affda88eb0f36536 Знаю что нельзя прикреплять но хочу чтобы кто-то оценил мою белиберду написал её сам без какой-либо помощи и свои ошибки поправлял через Debug. Так вот может кто-нибудь прокомментировать мой код и указать на места где можно сократить а где вообще можно что-то убрать потому , что у меня такое чувство будто бы я сделал всё очень не корректно...

Комментарии (2)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Justinian Judge в Mega City One

Master

7 ноября 2019, 12:56

синиоры и мидлы тебе будут нужны когда будешь писать свой код на продакшене, даже ревью crud на спринге не потребует навыков мидла и выше для того чтобы оценить или подсказать. А простые алгоритмические задачи и подавно.

for (int j = i; !(check3.equals("&")); ) {

если не шах три равен и прям скороговорка, в теории такое допустимое конечно, но имя чек3 вообще ничего не говорит. Здесь можно написать лучше.

} catch (Exception e){
       strList.add(result2);
}

так никогда не пиши. Блок эксепшен - catch для обработки исключительных ситуаций и точка. Ты представляешь чтобы по такому принципу работали вещи? Машина не переключит передачу, пока ей молотком не разбить окно. Велосипед руль не повернет вправо, пока не оторвать педаль. Не проткнул ногу гвоздем, не попил воды. Прям фильм Пила получается. Ты не можешь закладывать бизнес-логику в ситуации, которые являются исключительными и нежелательными. Если ты решил задачу, и хочешь знать как лучше - открывай Обсуждение задачи, там народ активно делится кто и какими способами решал. Не все способы корректные и нормальные, но идей там много хороших можно взять. И забудь слово "сократить". Короче не значит лучше. Правильный вопрос "как сделать лучше, как сделать корректней, лаконичней". Очень часто, лаконичней и лучше значит больше кода, а не меньше. По употреблению слова "короче код лучше" можно безошибочно выявлять людей только ставших на путь джава программирования, и их нельзя винить, у них просто ориентиров еще нет, что такое качественный код, самое простое что приходит в голову, это короче значит лучше. Нет, лучше это хороший код. А каким он будет, зависит от условий задания

Матвей Software Developer в Aucsys

7 ноября 2019, 14:42

Понял , спасибо большое за разъяснение . Просто когда уже задача прошла , мне это показалась очень сомнительным решением так как не видел я в примерах как ты выделил :

catch (Exception e){
       strList.add(result2);
}

И сама идея работы программы через исключение уже само по себе кажется не верным в данной задаче . Поэтому и задал такой вопрос. По поводу короче значит лучше , полностью согласен по крайней мере новичкам уж точно для практики лучше будет писать больше кода чтобы понимать всю картину в целом.