手続き型でよく見る配列に情報を持つ場合(カンマ区切りなども含めて)があります。問題は何番目に何のデータを保持しているのかが分かりにくいということです。これは非常に危険なデータの持ち方でもあります。5.演習・課題-3.リファクタリング変数スコープは出来るだけ短くするとバグの発生を抑えることができます。変数iはメソッドの最初の方に宣言されていますが、実際はfor文でしか使用されていません。5.演習・課題-3.リファクタリングステートメントをネストし過ぎると可読性が著しく低下します。逆に言えばネストが少なければ見やすくなるのです。以下のような数値をチェックするような関数を考えます。以下のような処理を考えます。5.演習・課題-3.リファクタリング制御フラグでの条件分岐はよくある手法の一つですが、これが多くなりすぎると非常に可読性が悪くなります。以下のコードを見てみましょう。以下のコードのどこが問題でしょうか。5.演習・課題-3.リファクタリング私がコードをチェックする時、「これは駄目なコードだ」とリファクタリングの対象とする点がいくつかあります。今回はそれをまとめて紹介しますので、共感できる部分は是非修正してみて下さい。5.演習・課題-9.おまけVB6、VBAからC#、VB.NETへ流れてきた方専用のチェックリストを作ってみました。自分がオブジェクト指向型を理解しているかどうかを17個のチェックで確認することができます。5.演習・課題-9.おまけVB6やVBAは手続き型と呼ばれていますが、別にオブジェクト指向で書けない訳ではないです。「向いていない」のです。関係性としては、サッカーのフィールダーとキーパーみたいな感じですね。ブログを報告するスターをつけました引用をストックしましたストック一覧を見る閉じる引用するにはまずログインしてくださいログイン閉じる引用をストックできませんでした。再度お試しください閉じる限定公開記事のため引用できません。