実装パターン

読んで気になったところをメモ

メソッドの名前

public class Employee
{
  public int EmpNo;
  public String Name;
  ...
}

public class EmployeeManager
{
  //どっちも社員を取得するメソッド
  public Employee getEmployeeInfo(int emp_no){ ... }
  public Employee get(int emp_no){ ... }
}

無理に名前にすべてを入れなくても、パラメタも名前の一部と見なすと短い名前でも十分通じる。

コメント

コードからは読み取れないことに対してだけ書くようにする。
→コードをメンテしてもコメントのメンテをし忘れる場合が多々発生する為。

//EmployeeManager.getメソッドの例
//何でこんな書き方しているのかをコメントとして残しておくと読む人の理解の助けになる。
public Employee get(int emp_no)
{  
  Employee ret_emp = null;

  foreach(Employee emp in this.emp_list)
  {
    if(emp.EmpNo.ToString() == emp_no.ToString())
    {
      ret_emp = emp;
      break;
    }
  }

  return ret_emp;
}

メソッドオブジェクト

行数が多いメソッドが存在する場合は、それをクラスにしてしまう。