Отзывы о собеседовании в Контур
Технический писатель в Контур.Логистику, middle+
Менеджер по продажам/Эксперт
Начинающий специалист по продажам
XML / XSLT-разработчик
В замечаниях упоминается «дублирование кода» и «спагетти-код», при этом не приведено ни одного конкретного примера архитектурного нарушения, ни объяснения, почему методы Add_Data1_Pay и Add_All_Sum — это дублирование, а не логически разные операции с разными контекстами. В реальных проектах, особенно в финансовых и аналитических системах, такие имена часто используются для ясности и соответствия бизнес-терминологии — они не являются нарушением ООП, если логика внутри них различна. Называть их «дублями» — признак поверхностного понимания предметной области, а не глубокого анализа кода.
Также критикуются имена переменных вроде inputXmlPath или filesDirectory. Эти имена полностью соответствуют стандартам C# (camelCase/pascalCase), семантически точны и легко читаемы. То, что они не совпадают с личными предпочтениями ревьюера, не является основанием для оценки как «нарушения сигнатуры кода». Это субъективное мнение, а не правило. Если бы ревьюер действительно разбирался в архитектуре, он бы обратил внимание на инкапсуляцию, SOLID, разделение ответственности, а не на стиль именования, который в C# строго регламентирован и не вызывает вопросов у опытных разработчиков.
Отсутствие упоминания о том, что код работает, корректно обрабатывает XML, отвечает требованиям, и был написан в рамках реалистичных сроков — говорит о том, что оценка проводилась не по результату, а по формальным, часто произвольным критериям.
Я не могу не отметить: HR-менеджеры, принимающие решение на основе таких оценок, явно не компетентны в оценке технических навыков. Если ревьюер не способен отличить корректный, читаемый и рабочий код от «спагетти», и при этом критикует имена переменных как «нарушение сигнатуры» — это либо непрофессионализм, либо попытка маскировать отсутствие понимания предметной области под видом «строгих стандартов».
Профессиональное ревью — это анализ архитектуры, производительности, масштабируемости, безопасности и читаемости. Не поиск «не того стиля», который не соответствует личному вкусу проверяющего.
Надеюсь, в вашей компании есть возможность пересмотреть процессы технического отбора — иначе вы рискуете потерять толковых разработчиков из-за непрофессиональной оценки.
Контент-мейкер в команду внутрикорпоративных коммуникаций
Персональный менеджер в направление Контур Банк
Интернет-маркетолог
Senior/Lead C# developer
Менеджер по продажам в IT-компанию
BI-разработчик, senior
Ведущий специалист по тендерам
Отзыв соискателя
Менеджер по продажам IT-продуктов и решений для бизнеса
порядок действий в той или иной ситуации
мотивация
Специалист по работе с клиентами (удаленно)
Обрисовка разных ситуаций: как вы поступите в этом случае или в этом? Что выберете - это или это?
Каким должен быть идеальный руководитель?
Менеджер по продажам
Менеджер по работе с ключевыми клиентами (Контур Логистика)
DevOps-инженер, middle+
Специалист в группу исследований
Специалист поддержки
Аналитик продуктовый
Специалист по продлению действующих договоров
Ведущий специалист по тендерам
Менеджер по продажам в IT-компанию
Менеджер по работе с корпоративными клиентами
Отзыв соискателя
Руководитель отдела корпоративных продаж
Начинающий специалист
Менеджер по работе с корпоративными клиентами (проект: Контур.Диадок)
Менеджер по продажам (без холодного обзвона)
Менеджер IT-проектов внедрения в Налоговый мониторинг
Информация о Контур
Количество сотрудников: более 10000
Категория: Информационные технологии, системная интеграция, интернет