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

[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+")";

[fortunator]

конкатенация быстрее,
но через стринг формат часто пишут не напрямую строчку, а например
String.Format(StringConstants.Contracts, Count);

И такая запись получается именно удобней с позиции модификации кода, поменять одну константу легче и надежней чем искать слово "Договора" по всему коду.