Java - это та ещё развлекуха, особенно если ваши разработчики Java не умеют.
Сколько времени делалось PHP приложение? Сколько разработчиков знают, что творится в коде и какая бизнес-логика с какими исключениями реализовывалась? Документация актуальна? Приложение уже в проде?
Если у вас та же команда, которая знает весь код, и все нестандартные решения, и все куски кода типа if skud == 42 //Так сказал Иван Иваныч, но он уже не помнит, почему мы завязываемся именно на 42, да ещё при этом у вас актуальная документация - то вы в хороших условиях. В противном случае - вы попали. Особенно, если приложение в проде.
Наверное, в вашем случае, есть смысл начинать потихоньку заменять куски PHP кода на Java. Сначала развязываете PHP подули через REST или ещё что-нибудь, а потом заменяете на Java отрезанный кусок.
Про 42, вообще такой код просто не пройдет ревью. Даже на самом жутком проекте
Да? Ну, ОК. Не пройдет - и не пройдет.
Если полноценное ревью на проекте уже было в то время, когда этот код писался
Это да, но в тырпрайзе обычно такое все же документируется. Даже в самом жутком Легаси которое я видел все расписано в дизайнах джиры. Ну и кодстайл подразумевает какую то адекватную оболочку для этой бизнес логики: наименование переменных/методов/etc
Я вам завидую. Вам везло с проектами. Без шуток и сарказма.
Код может пройти код ревью если код ревью нету =)
Как можно пройти что-то, чего нет? 😁
пфффф....легко, потому что как пример из другой системы вам приходит массив...ну я хз, например акций для пользователя и скажем для акции с id=42 вам не надо писать какоенибудь соообщение или например, надо написать что-то дополнительной в итоге это 42 у вас гдето всё равно будет вбито, константой,настройкой...массивом) ведь вы умный и сделали с заделом на будущее, ведь бизнес уже завтра после выкатки в прод прибежит и скажет что они забыли что ещё одна почти такая же акция есть
в чем проблема не писать if (id==42) а вынести в метод, с читающимся названием Плюс внести в доку (но это у кого как принято)
аа..вам не нравится кокнкретно if (id=42)..ну это конечно дичь бесспорно я просто к тому что гдето вы всёравно напишите blabla=42 (и именно там надо сделать явный комментарий почему 42, а не 68), о чём и написал выше)
Обсуждают сегодня