ランダムに3回違った文字を表示

Question

yahoo知恵袋をみて、少しきになってコードを実現させようとしましたが
うまくいかないので、質問してみました。

$a = "a";
$b = "b";
$c = "c";
を、ランダムに3回表示させたいです。重複はダメです。

そこで、以下のように組んでみましたが、うまくいきません。

<?php
$a = "a";
$b = "b";
$c = "c";

$array = array();

$i = 0;
while($i<3){
	$rand = rand(0,2);

	if($rand == 0 && !array_search($a, $array)){
		array_push($array, $a);	
		echo $a;
	}elseif($rand == 1 && !array_search($b, $array)){
		array_push($array, $b);
		echo $b;
	}elseif($rand == 2 && !array_search($c, $array)){
		array_push($array, $c);
		echo $c;
	}else{
		$i--;
	}

	$i++;
}


echo "<pre>";
print_r($array);
echo "</pre>";

どこのロジックが間違っているでしょうか。

Answer 1

array_search

array_searchを使うのではなく、in_arrayを使ってみてはどうでしょうか。

例:↓↓
if($rand == 0 && !in_array($a, $array, true))

Comments

[退会済みユーザー]

in_arrayでできました！
ありがとうございます。

Answer 2

array_search

needle が見つかった場合に配列のキー、 それ以外の場合に FALSE を返します。

これ、ちょっと厄介で、配列の最初の値と同じものを引くと0が返ります。
配列の中に値が存在するのにvar_export(!0);がtrueになり、意図した動作をしません。
＊配列の一番戦闘に入った値は重複を許してしまいます。

記述にあるコードの意図を最大限尊重すると、array_searchとなっている箇所でin_arrayを使用すると良いです。

追記
!array_search($x, $array)===falseってのもありですね。

Comments

[退会済みユーザー]

なるほど！
array_searchにはそんな落とし穴があったとは....

in_array()でできました！
ありがとうございます！！！！

Answer 3

shuffleしてはだめなのでしょうか？

$a = "a";
$b = "b";
$c = "c";
$array = array($a,$b,$c);
shuffle($array);
print_r($array);

Comments

[退会済みユーザー]

suffleという組み込み関数があったのですね。
今知りました。ありがとうございます。

Answer 4

どこのロジックが間違っているでしょうか。

重複はダメです、の意味が同じ文字が複数回出現してはだめの意味なら、そのロジックがないからです。間違っているのではなく、かけらもないのです。

私なら6パターンしかないので、６個の配列を持ってランダムで６個のうちの一つを返すロジックにします

Comments

[papinianus]

もっと要素が多いならfisher-yatesでシャッフルします

[退会済みユーザー]

先ほど気がつきました。ありがとうございます。