4チームの振り分けがうまくいかない

とあるシステムでチーム分け機能を作っています。

チームのHPが0で、一番少ないチームを取得しているはずなのですがなぜか$minteamが定義されていないとでます。なにか改善点がありましたらご教示願います

php
1	$this->red = array();
2	$this->green = array();
3    $this->blue = array();
4    $this->yellow = array();
5
6    /* HP 
7
8    rhp,ghp,bhp,yhp = 75;
9
10    */
11
12    public function getMinTeam(){
13
14		if(min($this->red,$this->green,$this->blue,$this->yellow) === $this->red && $this->plugin->game->rhp !== 0){
15
16			$minteam = "red";
17
18		}elseif(min($this->green,$this->blue,$this->yellow) === $this->green && $this->plugin->game->ghp !== 0){
19
20			$minteam = "green";
21
22		}elseif(min($this->blue,$this->yellow) === $this->blue && $this->plugin->game->bhp !== 0){
23
24			$minteam = "blue";
25
26		}elseif($this->plugin->game->yhp !== 0){
27
28			$minteam = "yellow";
29
30		}
31
32		return $minteam;
33
34	}

yambejp

2019/10/08 07:02

クラスで処理しているならそこから明示してください空の配列を比較する意味はあるのでしょうか？それともなにかデータが与えられているのでしょうか？プラグインの仕様もよくわかりません

sakura_hana

2019/10/08 08:59

・minにarrayを入れた時と数値を入れた時では返って来る結果が違うが理解出来ているのか。 https://www.php.net/manual/ja/function.min.php ・前提条件は「チームのHPが0で、」なのにif文では「チームのHPが0ではない時」になっている。・「一番少ないチームのHPが0ではない」時はどうするのか。この辺を確認＆考える必要があるかと思います。

m.ts10806

2019/10/08 11:56

他者がコピペで動かせる（現象確認できる）コードをご提示ください。

行動規範の内容に同意します

回答2件

なぜか$minteamが定義されていない

それはまぁ、初期値を設定していませんし、elseを設置していないので
$minteamが定義されないパターンは存在しますよね。

「～なはず」と言ってもコードは書いた通り、組んだ通りしか動かないので
きちんとデバッグして流れを追ってください。

投稿2019/10/08 11:58

m.ts10806

総合スコア80875

HP0のチームを拾ってからminした方がスマートな気がしたよ

php
1<?php
2public function getMinTeam(){
3  $teams_has_zero_hp=[];
4  foreach(['red'=>'rhp','green'=>'ghp','blue'=>'bhp','yellow'=>'yhp'] as $color=>$hp_key){
5	  if($this->plugin->game->$hp_key==0){$teams_has_zero_hp[$color]=$this->$color;}
6  }
7  if(empty($teams_has_zero_hp)){return null;}
8  return array_keys($teams_has_zero_hp ,min($teams_has_zero_hp));
9}