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

回答編集履歴

2

サンプルコード修正

2017/06/21 07:13

投稿

m.ts10806
m.ts10806

スコア80888

answer CHANGED
@@ -4,13 +4,14 @@
4
4
 
5
5
  ```PHP
6
6
  $updatemethod = "_mountain_";
7
+ $idstr = substr($req['id'], 0, 3);
7
8
  //呼び出し1
8
- if(substr($req['id'], 0, 3) != 'XXX'){
9
+ if($idstr != 'XXX'){
9
10
  if($this->is_time_change($req)){
10
11
  $req["rain_flag"]= 0;
11
12
  }
12
13
  $updatemethod .= "sch";
13
- }elseif(substr($req['id'], 0, 3) == 'XXX'){
14
+ }else{
14
15
  //呼び出し2
15
16
  $req['day_clim_flg'] = 0;
16
17
  $updatemethod .= "reg";

1

編集

2017/06/21 07:13

投稿

m.ts10806
m.ts10806

スコア80888

answer CHANGED
@@ -1,4 +1,4 @@
1
- 特に変とは思いませんが、2つのメソッドが同時に実行されるようではないので、
1
+ 2つのメソッドが同時に実行されるようではないので、ifが1回1回切れるのは無駄かなと思います。
2
2
  いっそのことメソッド名を変数に入れるのはどうでしょうか。
3
3
  (エラーはありませんが動作自体は未検証です)
4
4