VBAわかりますけど(キリッ)みたいな人が作ったマクロを直すのが苦痛すぎる。
なんでもエクセルでやろうとすんな。
マスタのデータをエクセルに貼り付けたものをつかってVBA組むな。
変数はご丁寧に一番先頭で宣言祭り、コントロールの名前が連番、無意味な処理、データの件数を取得するためだけに同じSQLをCOUNTにして実行、無意味なループに、ifの4段ネスト、メソッド名が不適切(checkXXX)、スコープは全部Public、定数の概念無し(マジックナンバー多すぎ問題)、型変換の概念無し(文字列を数字にぶっこむ)、例外処理なし、その他突っ込みどころ多数