一時変数を避けて可読性を上げる

チケットのレビューで可読性の話が勉強になりました。theDay という変数を定義したかったのですが、変数の種類が合わずに何回も一時的な変数を定義をし直す形で対応しました。

VelocityPortlet portlet = ALEipUtils.getPortlet(rundata, context);
String a = portlet.getPortletConfig().getInitParameter("z1a-rows");
int theDay = Integer.parseInt(a);

しかしktaguroさんにレビューして頂いた時のコメントにあったのですが、 一時変数を避けることで無駄に変数の種類を変えずに済むようです。

int theDay = Integer.parseInt(ALEipUtils
   .getPortlet(rundata, context)
   .getPortletConfig()
   .getInitParameter("z1a-rows"));

非常に見づらかったコードが、「何を」定義して「どう」操作しているのか、分かりやすくなりました。

ありがとうございます。