Как решить правильно простенькую задачку по php?

Question

[Exair]

Сразу скажу,что сравнительно недавно начал изучать php, и решаю задачи из сайта Вот

Условие задачи

Создайте строку из 6-ти случайных маленьких латинских букв так, чтобы буквы не повторялись. Нужно сделать так, чтобы в нашей строке могла быть любая латинская буква, а не ограниченный набор.

Мой план решения

1)Создать строку с 26 буквами латинского алфавита
2)Инициализировать пустую переменную
3)Из строки с буквами сделать массив через explode
4)В цикле for пройтись по этому массиву(нужно 6 итераций)
5)Вывести новую строку на экран

Решение

$str = "a,b,c,d,e,f,g,h,i,j,k,l,
        m,n,o,p,q,r,s,t,u,v,w,x,y,z";
$newStr = "";
$arr = explode(",", $str);

for($i = 0;$i < 6;$i++) {
    $rand = mt_rand(0, 25);
    $newStr .= $arr{$rand};
}
echo $newStr;

Проблема заключается в том, что спустя n-ое количество рандомных строк показываются строки с пробелом,хотя, как вы видите,его у меня в строке $str нет.
И еще вопрос, как решить эту задачу с использованием str_shuffle?

Answer 1

[Игорь Воротнёв]

// По вашему коду:
$source_string = 'abcdefghijklmnopqrstuvwxyz';
$string_randomized = str_shuffle($source_string); // Перетасовали буквы внутри строки
echo substr($string_randomized, 0, 6); // Вывели первые 6, без повторений и каждый раз после str_shuffle - уникально-рандомно (относительно, но задача именно этого и требует)

// По красоте:
$characters = 'abcdefghijklmnopqrstuvwxyz';
echo substr(str_shuffle($characters), 0, 6);

// Тестируем в psych:
>>> $characters = 'abcdefghijklmnopqrstuvwxyz';
=> "abcdefghijklmnopqrstuvwxyz"
>>> echo substr(str_shuffle($characters), 0, 6);
fzpvad
>>> echo substr(str_shuffle($characters), 0, 6);
bzeiyj
>>> echo substr(str_shuffle($characters), 0, 6);
pgkuaq
>>> echo substr(str_shuffle($characters), 0, 6);
otjszx
>>> echo substr(str_shuffle($characters), 0, 6);
zjwaox
>>> echo substr(str_shuffle($characters), 0, 6);
rxuhnv
>>> echo substr(str_shuffle($characters), 0, 6);
rihoec
>>> echo substr(str_shuffle($characters), 0, 6);
uwdfgn
>>>

Comments

[Виталий Хоменко]

Тс, когда за вас делают решения вы ничему не научитесь. Уверен, вы не понимаете, что именно происходит в этом коде

[Игорь Воротнёв]

Виталий Хоменко, для этого я снабдил код комментариями. А так то да, согласен. У меня вот жена Swift сейчас изучает (до этого с программированием не имела ничего общего), она такие задачки сама пилит, хотя иногда приходится посидеть и потупить пару часов. Но на выходе у нее всегда мощный прилив мотивации и реально усвоенный урок. Раз и на всю жизнь. А на Тостере спросить - это читерство самого себя :)

[Exair]

Спасибо вам, на такое решение и смотреть приятно)

[Дмитрий Чижов]

Игорь Воротнёв, не согласен. На тостере обьяснят как правильно и почему именно так. А сам наговнокодит чего нибудь и будет рад что работает. И знать не будет что вместо сотни строк все можно одной сделать. Опытный товарищ всегда полезен

[Игорь Воротнёв]

Дмитрий Чижов, разумеется. Но сначала нужно написать самому в нескольких разных формах. Когда проходишь основы программирования / базу ЯП то подобные задачки обычно идут одна за другой, и люди достаточно быстро учатся с ними справляться. И часто сами задания предусматривают, что ученик напишет говнокод, чтобы потом увидеть элегантное алгоритмическое решение, которое переведет его на следующий уровень обучения. Быстро сдаться и пойти на Тостер - самый легкий путь, имхо. Но у каждого свой путь, объем терпения и усидчивости, поэтому я не осуждаю никого и поэтому дал человеку ответ.

Answer 2

[Рональд Макдональд]

Буквы будут рано или поздно повторяться, если отталкиваться от текущего решения, то я бы загонял шесть случайных чисел в отдельный массив и проверял бы новое рандомное на принадлежность одному из предыдущих.

А на кой чёрт вообще все эти движения с explode, если можно было просто создать строку со всеми буквами и обращаться к отдельным буквам просто по индексу?

Comments

[Игорь Воротнёв]

В задаче нету ограничения на то, что буквы/комбинации начнут повторяться по мере накопления итераций. Это простая задачка на базу ЯП, здесь нужно каждый раз получать уникальное значение, в котором эти буквы не повторяются, и здесь автор пошел в принципе правильным путем. Но explode и работа с массивом (даже по индексу) - это неправильный путь.

Answer 3

[Stimulate]

$letters = range('a', 'z');
shuffle($letters);
$newStr = "";
for($i = 0;$i < 6;$i++) {
    $newStr .= $letters[array_rand($letters)];
}
echo $newStr;

"чтобы буквы не повторялись." - этот пункт не выполнен

Answer 4

[FanatPHP]

Проблема заключается в том, что спустя n-ое количество рандомных строк показываются строки с пробелом

Это очень забавный вопрос :)

хотя, как вы видите,его у меня в строке $str нет.

Как мы видим, его у тебя там есть :)

Comments

[Exair]

\n?

[Дмитрий Чижов]

Да, и то что вокруг него. Строки склеивать через точку если перенос нужен.

Answer 5

[Андрей]

напишу свой вариант, ради спортивного интереса:

function randString($length = 6) {
	$out = "";
	$chars = range("a", "z");
	while (strlen($out) < $length) {
		$rand = rand(0, count($chars) - 1);
		$out .= $chars[$rand];
		unset($chars[$rand]);
	}
	return $out;
}

Comments

[Марат Гарифуллин]

Жесть