Comments 15
Этот трэшак же написал джун-аналитик незнакомый с регулярными выражениями, да?
// Шаблон № ИП:
// цифры, «/»,две цифры, символ «/»,
// четыре цифры, символ «-»,
// далее два символа в верхнем регистре русской раскладки –или ИП, или СВ, или СД
// Пример:542145345/23/5468-ИП
/^\d+\/\d{2}\/\d{4}-(ИП|СВ|СД)$/
Главное, чтобы работало. Функция выше, как я понял, соберет нужную маску, вот только условию: "или ИП, или СВ, или СД" - не соответствует: пропустит любые две буквы в верхнем регистре.
Главное, чтобы работало.
Ну-да, ну-да. А потом мы жалуемся, что одна страничка жрёт гигабайты памяти.
Если бы строители строили здания так же, как программисты пишут программы, первый залетевший дятел разрушил бы цивилизацию.
Тестирование было проведено, любые не проходят
А любые две кириллические (в верхнем регистре)?
да, две любые. Но у нас требование было такое, чтобы две буквы на русской раскладке (в верхнем регистре) . В данной ситуации разработчик закомментировал строку, выбрав образец данный в ТЗ. Собственно, это моя первая статья и я не думала, что будут читать код и закомментированные строки. Учту на будущее. спасибо.
Забыл написать, что буквы должны быть кириллическими. Но из комментария ниже понял, что две любые кириллические буквы в верхнем регистре не проходят. Значит, реализовали лучше, чем в коде в статье.
Благодарю за комментарий. Во-первых- это фрагмент кода, а не ТЗ. Нужно понимать, что аналитик делает свою часть работы, разработчик свою.
Елена, вот не хочется быть предвзятым к вашему материалу, но на основании ваших же слов, "грамотный подход аналитиков" - это потраченное вникуда время аналитиков и команды разработки на совещания, а также на выдачу очень примитивного решения, которое могло бы прийти в голову и джуну с опытами курса практикума. Извините, но полезного тут ничего не увидел. Минус.
Интересный способ делать ревъю кода - выкладывать его на хабр 😁
Как грамотный подход аналитиков может сэкономить бюджет компании за счет изменения подхода к реализации проекта