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

kisyaman 36views 更新:2016年8月8日

チケットのレビューで可読性の話が勉強になりました。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"));

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

ありがとうございます。

ログイン / 新規登録してコメントする

このソースコードをストックして後で利用したり、作業に利用したソースコードをまとめることができます。

こちらもお役に立つかもしれません