Q&A
むしろ convertStr2Int という関数名と month という引数の方が問題ありだと思いますが、そこはいいのですか？
お忙しいところ失礼します！
現在Javaを勉強しているのですが、可読性を高めるために命名やコメントを徹底的に考えてプログラミングをしています！
そこでよく悩むのですが、
Java
1// 引数monthに対して翌月を返す 2public String addMonth(String month){ 3 final String dec = "12"; 4 if(dec.equals(month)){ 5 month = "1"; 6 }else{ 7 int 「①」= Integer.valueOf(month);// int型に変換したmonthを代入 8 month = String.valueOf(「①」+1); 9 } 10 return 「month」; 11}
このような場合、「①」に何と命名するのがふさわしいでしょうか？
1行でreturnする場合や引数をint型場合は考えず、型変換後の値を変数に代入するとしたらどういう命名が適切なのか教えていただけますでしょうか？
よろしくお願いいたします！！
回答6件
0
可読性は, 書く側読む側の共通認識(いわゆる”文化”)によって異なります. ですので, 「命名やコメントを徹底的に考えて」も全てにおいての「適切」は無いものと考えます.
あえて申し上げれば, 一般的なほうを一般的な名前にし, (比較的)一般的でないほうを特殊な名前にするとよいのではないでしょうか.
件の場合, 「月」は数値のほうが利用し易いと思いますので, int のほうを month とし, String のほうを monthStr 等としては如何でしょうか.
投稿2019/11/12 03:08
総合スコア12646
0
このような場合、「①」に何と命名するのがふさわしいでしょうか？
「変換以外の処理をどうするか」による、としか言いようがないです。
純粋に変換しかしないなら、変数自体が不要でしょう。
投稿2019/11/12 03:01
総合スコア145183
質問文のコードを修正致しました。
可能でしたら修正内容の場合、どのようにするかお答えいただきたいです！
修正後のコードでも変数自体不要という議論は成立しますね。
public String addMonth(String month){
final String dec = "12";
if(dec.equals(month)){
return "1";
}else{
return String.valueOf(Integer.valueOf(month) + 1);
}
}
みたいな。
こちらの方が可読性が高いという主張もあり得るでしょう（個人的にもそう思います）。
0
オレがするなら、a
たかだかローカル変数の命名にそこまでかかずらわってられまへん
投稿2019/11/12 03:18
総合スコア87774
0
ベストアンサー
文字列から数値への変換であれば
引数：str
ローカル変数：num
とかでいい気がします。
質問文のコードでは変数定義不要としか見れないので、
他にも似たような変数が必要なのであれば
他に必要な変数と考慮して名前を考えます。
[質問文修正後を受けて追記]
わざわざ月を文字列で扱っている状況がわかりませんが、
私なら以下のいずれかでやると思います。
（採用度高い順に並んでます）
案１：
そもそも数値の月を文字列で扱う方が特殊なので、
引数をmonthStr、該当箇所をmonthにする
案２：
本当にその箇所だけで使うならtmpという一時変数を使う
案３：
monthIntのように後ろに型名をつける
投稿2019/11/12 03:11
編集2019/11/12 04:28
総合スコア5158
質問文のコードを修正致しました。
可能でしたら修正内容の場合、どのようにするかお答えいただきたいです！
追記しました。
0
[元の変数名]as[型名] を使う場合があります。
monthAsLocalDate
投稿2019/11/12 03:20
総合スコア12011
0
y_waiwaiさんと同意見d^^
ローカル変数の名前まで、あまり細かく考えません。sTmpとかiTmpとかsInxとかcPos・・・ｗ
・・・60行を超える関数（メッソド）は作らないから、見通しが悪くなることはない・・・
投稿2019/11/12 05:48
総合スコア6851
ドットインパクトの時代に、６６行で収まるように書く癖が出来たｖ＾＾
流石に、２４行（昔のモニタの行数）は無理ｗ
名前の重要性はスコープの大きさに比例しますよね。
ライブラリのパブリックな名前は一度つければまず変えられませんし、逆にブロックにローカルな変数は変えても他の場所には影響しません。
あなたの回答
tips
プレビュー
