オブジェクト指向の導入

5．演習・課題 5．演習・課題-3．リファクタリング

課題手続き型でよく見る配列に情報を持つ場合（カンマ区切りなども含めて）があります。一番の問題は何番目に何のデータを保持しているのかが分かりにくいということです。これは非常に危険なデータの持ち方でもあります。 string[] person = new string[2]…

2019-06-08

無意味なスコープ

5．演習・課題 5．演習・課題-3．リファクタリング

変数スコープは出来るだけ短くするとバグの発生を抑えることができます。課題① 変数iはメソッドの最初の方に宣言されていますが、実際はfor文でしか使用されていません。 int i; //処理① for (i = 0; i <= 10; ++i) { //処理② } Dim i As Integer '処理① Fo…

2019-05-27

ガード節によるネスト防止

5．演習・課題 5．演習・課題-3．リファクタリング

ステートメントをネストし過ぎると可読性が著しく低下します。逆に言えばネストが少なければ見やすくなるのです。課題以下のような数値をチェックするような関数を考えます。 private int test1(int x, int y, int z) { int res = 0; if (x < 0) { res = 1…

2019-05-06

三項演算子の有効活用

5．演習・課題 5．演習・課題-3．リファクタリング

課題例えば以下のような処理を考えます。 bool syoriFlg = true; string naiyo = ""; if (syoriFlg ) { naiyo = "hoge"; } else { naiyo = "piyo"; } Dim syoriFlg As Boolean = True Dim naiyo As String = "" If syoriFlg Then naiyo = "hoge" Else naiyo…

2019-03-27

不要な制御フラグ

5．演習・課題 5．演習・課題-3．リファクタリング

制御フラグでの条件分岐はよくある手法の一つですが、これが多くなりすぎると非常に可読性が悪くなります。課題① 例えば以下のコードを見てみましょう。 int x = 1; int y = 2; bool flg = true; while (flg) { if (x * y > 10) flg = false; ++x; ++y; } D…

2019-02-24

マジックナンバーはシンボリック定数に置き換える

5．演習・課題 5．演習・課題-3．リファクタリング

課題以下のコードのどこが問題でしょうか。 string[] colorNames = {"桃", "黒", "黄", "青", "赤" }; for (int i = 2; i <= colorNames.Length - 1; i++) { Console.WriteLine(colorNames[i]); } Dim colorNames As String() = {"桃", "黒", "黄", "青", "…