Чтобы бы вы сказали о программисте по представленному коду?

[alekbort]

Есть следующий код (кликабельно):

Comments

[fortunator]

1. С#
2. Приложение по работе с базой данных каких-то контрактов.
3. Тело foreach выделил бы в отдельный метод, назвал бы его GetRow(Contract m_object) например.

[fortunator]

ну и у класса Contract метод GetComparer() сделал бы статическим, а то студия ругается :))

[fortunator]

последнюю строчку записал бы по другому
tbpContract.Text =
либо
String.Format("Договора ({0})", m_contractList.Count);
либо
"Договора ("+m_contractList.Count+")";

[dm_kalashnikov]

Бухгалтерское приложение. Загружает список договоров в таблицу по какому-то фильтру.

Судя по всему в коде ошибка.

[fortunator]

Ну это смотря какое соглашение, мы например в своей фирме не делаем так, а бухгалтера и системщики очень даже часто.

[kolllak]

c#

прога делает следующее:
1)отбирает из некой таблички договора с контрагентами
2)договора помещаются в другую таблицу формата: код контрагента, дата начала, дата окончания, сумма
3)если договор актуален на определенную дату то даты проставляются, если нет то поля оставляются пустые
4)считается общее количество договоров

п. 3 очень странен, ну видимо так хотел заказчег ))

по структуре кода притензий нет, реализация очень простенькая (без изюминки и изъебов) но работающая

[a_suber]

Средненький код для загрузки данных в эксель или на формочку.
C#
Поставить решарпер и следовать его инструкциям :)
Использовать констаны и ресурсы вместо магических чисел.
Формат вместо конкатенации.
В именовании переменных не использовать "_" и "m", "_" в шарпе мешает чтению, а м - вообще не нужно. Просто именовать переменную "contractor".
Отсутствие комментариев. Особенно хмл-ных для метода.
Настораживает еще статический метод при создании фильтра, но это без детализации не обсуждается.