回答編集履歴

2

サンプルコード修正

2017/06/21 07:13

投稿

m.ts10806
m.ts10806

スコア80852

test CHANGED
@@ -10,9 +10,11 @@
10
10
 
11
11
  $updatemethod = "_mountain_";
12
12
 
13
+ $idstr = substr($req['id'], 0, 3);
14
+
13
15
  //呼び出し1
14
16
 
15
- if(substr($req['id'], 0, 3) != 'XXX'){
17
+ if($idstr != 'XXX'){
16
18
 
17
19
  if($this->is_time_change($req)){
18
20
 
@@ -22,7 +24,7 @@
22
24
 
23
25
  $updatemethod .= "sch";
24
26
 
25
- }elseif(substr($req['id'], 0, 3) == 'XXX'){
27
+ }else{
26
28
 
27
29
  //呼び出し2
28
30
 

1

編集

2017/06/21 07:13

投稿

m.ts10806
m.ts10806

スコア80852

test CHANGED
@@ -1,4 +1,4 @@
1
- 特に変とは思いませんが、2つのメソッドが同時に実行されるようではないので、
1
+ 2つのメソッドが同時に実行されるようではないので、ifが1回1回切れるのは無駄かなと思います。
2
2
 
3
3
  いっそのことメソッド名を変数に入れるのはどうでしょうか。
4
4