記述コードの違いによる意図。※動作に違いはありません。

Question

表題の通り、
どちらも動作自体に変化はないです。

下記コードの②について、
どの様な意図があると思いますか。
※1:書物による勉強の為、このコードを書いた人に意図はきけません
※2:必要のないコードは載せていません。
→全てのコードを記載しろとの指摘はご遠慮願います
※3:私も人間です。横暴と思われる返答文の場合大変恐れ入りますが無視します。

-----【私の認識】-------
取得してくる行
加算する行
をそれぞれ分けた方が可読性が上がる？
という事でしょうか

---

-----【処理の流れ】-------
①
今ある「いいね」の数をget()メソッドで取得
それに1を加えてcount変数に代入
そのカウント変数をset()メソッドでプロパティに入れる

---

		//①いいねを加算
		 int count = siteEV.getLike() + 1;
		 siteEV.setLike(count);

		//②いいねを加算
		 int count = siteEV.getLike();
		 siteEV.setLike(count + 1 );

Comments

[K_3578]

一つお聞きしたい、
①と②が別解のように書いてあったって事ですか？

[m.ts10806]

「回答者の追記修正依頼はすべて排除する」というご自身の横暴は許して良いのでしょうか。
意図を聞くなら本気でコード全体ないと他人には答えられません。
一部だけでどうしろと。
他人を頼ることを決めた以上は必要な情報は自分で決めるものではありません。見る人が判断します。

[m.ts10806]

それに書籍であっても出版社含め作者にアプローチする術は幾らでもあるし、もし間違ってる内容なら間違ってる内容でお金取ってるわけですから指摘する必要がある。
その補足は言い訳を列挙しただけで本気で解決したい人の投稿には見受けられません。

[Aspiration]

>K_3578 様
いえ、別解の様に記載はしておりません。
②が記載されている内容です。

①が私が作成したコードです。

[Aspiration]

>m.ts10806
以前の質問でもお答えした様にあなたからの返答は求めておりません。
質疑応答の中でコードが追加で必要に感じれば勿論記載します。
そして現状、全く必要性は感じません。
ゆえにあなたの言葉遊び(ご自身の横暴は…など)に付き合うつもりもございません。

ブロック機能がないので次回よりあなたの返答は完全に無視いたします。

[TakaiY]

もったいないなぁ というのが感想です。

[K_3578]

この部分だけで考えるなら完全に好みになると思うので既に出ている回答を私は全面的に支持します。

>※2:必要のないコードは載せていません。
ちょっとこれが気になったのでコメントさせてください。

エラーが出ているというような質問なら再現可能ならばミニマムコードで良いと思いますけど、
こういうどちらが良いか、という判断に関しては現状質問者さんが理解されているように、

どちらも同じ結果を出すのならば比較については前後の内容で判断するしかないと思うので、
全部を載せろとは言いませんが、どういう構造とかは書いて頂いた方が良かったのではと
思います。

#と、コメントを書いていたら既にほぼ私の言いたいことをquickquipさんが代弁してました｡

Answer 1

このコードの後ろでcountを使うかどうか
その時使いたいのが加算する前の値か、加算した後の値か

で答えが変わるのだから「必要のないコードは載せていません」なんてありえないでしょう。
質問の前提となる情報がおかしいです。

---

逆に言うと、必要のないコードを省いたらこれだけしか残らなかった、
のであれば、その事実こそがどっちでもいいという結論にいたる証拠だろうと思います。

Comments

[Aspiration]

ご指摘ありがとうございます。
なるほど、そういう事ですね。
であれば、この後countは使う事はございません。
抜粋したコードはLogicクラスによるメソッドの処理で加算処理です。
そしてそのメソッドの処理内容は上記コードで完結します。
呼び出し元はコントローラクラスです。
(MVCモデルです)

----------
なるほど
確かにそういう見方もできますね

[quickquip]

使わないならcount変数に取っていることそのものの是非/よしあしも考えないと意味がなさそうです。

siteEV.setLike(siteEV.getLike() + 1);

で十分では??

Answer 2

掲載コード部分だけなら①の方が私好みですが、
各自お好きにという感じであまり意味ないように思います。

ただ、条件が変わればこっちの方がいいだろうというのはあると思います。
例えばsetterが複数あって、getLike()はいつ呼んでも変わらないが、微妙に値を調整してsetしたい場合は②の方が良いでしょうし

int count = siteEV.getLike();
siteEV.setLike1(count + 1);
siteEV.setLike2(count + 2);
siteEV.setLike3(count + 3);

getLike()が呼ぶたびに値が変わるような場合は、①の方が良いでしょう。

int count1 = siteEV.getLike() + 1;
siteEV.setLike1(count1);
int count2 = siteEV.getLike() + 1;
siteEV.setLike2(count2);
int count3 = siteEV.getLike() + 1;
siteEV.setLike3(count3);

Comments

[Aspiration]

なるほど。
わかりやすいご説明ありがとうございます。
大変勉強になります

Answer 3

countがどの時点のカウントか

        //①いいねを加算
         int countToSet = siteEV.getLike() + 1;
         siteEV.setLike(countToSet);

        //②いいねを加算
         int currentCount = siteEV.getLike();
         siteEV.setLike(currentCount + 1 );