teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

3

指摘の通り、改行を追加しただけのコードを書いた

2020/01/28 01:05

投稿

miyabi_pudding
miyabi_pudding

スコア9570

answer CHANGED
@@ -1,5 +1,21 @@
1
1
  個人的な意見です。
2
2
  可読性云々考えても同じ条件をifに入れるなら、
3
3
  **可読性自体あまり変わらない**と思います。
4
- つまり、そのままでいんじゃね?ってことです。
4
+ ~~つまり、そのままでいんじゃね?ってことです。~~
5
- ただ、if使うにしろ、比較演算で直接論理値返すにしろ、コメントアウトで何と何をどう比較しているかは記載した方が、他の人にもわかりやすいものになるかと。
5
+ ただ、if使うにしろ、比較演算で直接論理値返すにしろ、コメントアウトで何と何をどう比較しているかは記載した方が、他の人にもわかりやすいものになるかと。
6
+ また、構文そのままでも、改行を入れるだけでも可読性はあげれるかと思います。
7
+ ```php
8
+ function Test1() {
9
+ $A = true;
10
+ $B = 1;
11
+ $C = 2;
12
+ $D = 3;
13
+ $E = 4;
14
+
15
+ return (
16
+ ($A && ($B !== $C)) ||
17
+ (!$A && ($D !== $E))
18
+ );
19
+ }
20
+ ```
21
+ (構文そのままだと、これ以上入れると、逆に可読性悪くなるかもしれない)

2

用語間違っていたため修正

2020/01/28 01:05

投稿

miyabi_pudding
miyabi_pudding

スコア9570

answer CHANGED
@@ -2,4 +2,4 @@
2
2
  可読性云々考えても同じ条件をifに入れるなら、
3
3
  **可読性自体あまり変わらない**と思います。
4
4
  つまり、そのままでいんじゃね?ってことです。
5
- ただ、if使うにしろ、三項演算で直接返すにしろ、コメントアウトで何と何をどう比較しているかは記載した方が、他の人にもわかりやすいものになるかと。
5
+ ただ、if使うにしろ、比較演算で直接論理値返すにしろ、コメントアウトで何と何をどう比較しているかは記載した方が、他の人にもわかりやすいものになるかと。

1

質問者のコードままで別に問題ないことを追記

2020/01/27 20:02

投稿

miyabi_pudding
miyabi_pudding

スコア9570

answer CHANGED
@@ -1,4 +1,5 @@
1
1
  個人的な意見です。
2
2
  可読性云々考えても同じ条件をifに入れるなら、
3
3
  **可読性自体あまり変わらない**と思います。
4
+ つまり、そのままでいんじゃね?ってことです。
4
5
  ただ、if使うにしろ、三項演算子で直接返すにしろ、コメントアウトで何と何をどう比較しているかは記載した方が、他の人にもわかりやすいものになるかと。