プログラムの中身は見てないけど

>>・細かいところだけど、関数の途中で変数宣言するのはイクナイ癖。やるにしても、ブロックを作ってその中にするべき

こういうのは論外ですね
Cがそれしかできなかったからって古い悪習を強要するのはやめろ
こういうやつが開発現場にいると激しく開発効率が下がるってしばしばコラムでも糾弾の対象になる

>>・usingはC#特有とはいえ、せめてtry finallyぐらい指示してあげなさいよ>先生

これも論外