![[personal profile]](https://www.dreamwidth.org/img/silk/identity/user.png)
Нет, в целом так всё неплохо, как я уже писал. Но есть моменты...
Весь, АБСОЛЮТНО весь код, написанный моим начальником, ужасен. Просто ужасен. И дело не в оформлении, не в пристрастии к пустым местам и к вложенным блокам, возникающим на ровном месте.
Разделения логики и представления нет. Как класса. Элементарнейшая вещь - но нет, весь вывод идёт прямо внутри скрипта, print и пошло-поехало. Дорабатывать такое просто нереально. В результате я переписываю всё, до чего дотягиваются мои руки, на HTML::Template. Начальник кривится, но в целом не возражает.
Методы работы с информацией. Это же ужасно! Выберем из базы кучу записей. По какому-нибудь простому условию. Запихнём всё это в хэш. Затем второй раз пройдёмся циклом по этому хэшу (for (keys %hash), при этом ключи копируются во временный массив!), отфильтруем записи. Да, неплохо если при этом ключи сортируются. Запишем всё ещё в один хэш. Если записи связаны с другой таблицей, значит из неё тоже всё выберем, и в цикле будем сравнивать с первым хэшем и создавать третью структуру. Хорошо хоть не делает декартова произведения, а то я и такое однажды видел (заносим в память две выборки на несколько тысяч записей, и потом проходимся по декартову произведению этих выборок, занося это в третий массив. Ограничение сегмента данных в 64 килобайта очень сильно обижалось). Запись с максимальной датой выбирается как ORDER BY date DESC LIMIT 1. Ужасно. Просто ужасно. Нет, я всякое в жизни видел, но ТАКОЙ индусский код - всё-таки впервые.
Отношение к работе. Во всём московском офисе есть три человека, которые занимаются делом. Мой начальник, гендиректор, и директор по маркетингу. Все остальные непонятно зачем тут нужны, и непонятно чем занимаются. Их требования на автоматизацию их работы необоснованны по определению. Техподдержка хочет получать статистику по нескольким типам запросов сразу? Не по одному или всем вообще, а по нескольким, то есть возможность выбора? Мало ли что она хочет. "Там столько всего можно дорабатывать, увязнешь". Ужасно. Просто ужасно. Как вообще получается, что такие люди оказываются в руководстве?...
Весь, АБСОЛЮТНО весь код, написанный моим начальником, ужасен. Просто ужасен. И дело не в оформлении, не в пристрастии к пустым местам и к вложенным блокам, возникающим на ровном месте.
Разделения логики и представления нет. Как класса. Элементарнейшая вещь - но нет, весь вывод идёт прямо внутри скрипта, print и пошло-поехало. Дорабатывать такое просто нереально. В результате я переписываю всё, до чего дотягиваются мои руки, на HTML::Template. Начальник кривится, но в целом не возражает.
Методы работы с информацией. Это же ужасно! Выберем из базы кучу записей. По какому-нибудь простому условию. Запихнём всё это в хэш. Затем второй раз пройдёмся циклом по этому хэшу (for (keys %hash), при этом ключи копируются во временный массив!), отфильтруем записи. Да, неплохо если при этом ключи сортируются. Запишем всё ещё в один хэш. Если записи связаны с другой таблицей, значит из неё тоже всё выберем, и в цикле будем сравнивать с первым хэшем и создавать третью структуру. Хорошо хоть не делает декартова произведения, а то я и такое однажды видел (заносим в память две выборки на несколько тысяч записей, и потом проходимся по декартову произведению этих выборок, занося это в третий массив. Ограничение сегмента данных в 64 килобайта очень сильно обижалось). Запись с максимальной датой выбирается как ORDER BY date DESC LIMIT 1. Ужасно. Просто ужасно. Нет, я всякое в жизни видел, но ТАКОЙ индусский код - всё-таки впервые.
Отношение к работе. Во всём московском офисе есть три человека, которые занимаются делом. Мой начальник, гендиректор, и директор по маркетингу. Все остальные непонятно зачем тут нужны, и непонятно чем занимаются. Их требования на автоматизацию их работы необоснованны по определению. Техподдержка хочет получать статистику по нескольким типам запросов сразу? Не по одному или всем вообще, а по нескольким, то есть возможность выбора? Мало ли что она хочет. "Там столько всего можно дорабатывать, увязнешь". Ужасно. Просто ужасно. Как вообще получается, что такие люди оказываются в руководстве?...