Как исправить проблему с циклами foreach и массивом php?

Question

[Orange___XD]

Всем привет. Есть у меня вот такой вот код:

$textareaValue = 'aaa@aaa|aaa/nbbb@bbb|bbb/nggg@ggg|ggg';
$smth = explode('/n',$textareaValue);
foreach ($smth as $aa => $bb) {
list($emails, $passwords) = explode('|', $bb);
foreach ($emails as $a1 => $b1) {
    echo $b1.':';
}
foreach ($passwords as $a2 => $b2) {
     echo $b2.'<br>';
     } 
}

Код делит сначала строку на строки вида "aaa@aaa|aaa" и добавляет их в массив, а затем каждую строку в массиве делит на еще на две части, но они записываются в два разных массива. С ключами в них творится вообще непонятно что: рандомно выставляется 0 или 1. Подскажите пожалуйста, как объединить их после деления в один массив вида $key => $item, где $key будет равен "[e-mail]" (Пример), а $item будет равен "test" (Пример)? Или хотя бы как сделать так, чтобы ключи в массивах после деления шли от 0 и до n?

Есть подозрения, что проблема в list(), но как исправить не знаю.

Answer 1

[yanchumak]

/n это новая строка? если да,то это не новая строка,вот символ новой строки \n

Comments

[Orange___XD]

За это спасибо, но остальное все равно не работает.

Answer 2

[Arman]

$textareaValue = 'aaa@aaa|aaa/nbbb@bbb|bbb/nggg@ggg|ggg';

$explode_1 = explode('/n', $textareaValue);

foreach ($explode_1 as $value_1)
    {
    $explode_2 = explode('|', $value_1, 2);

    echo $explode_2[0] .': '. $explode_2[1] .'<br />';
    }

Comments

[Orange___XD]

Вот спасибо огромное)

Answer 3

[Константин Чухломин]

Для начала, ваш код бросает исключение «Invalid argument supplied for foreach()» когда вы пытаетесь пройтись по переменной $emails, которая строка.

Далее, что будет, если в пароле встречается символ |?

Далее, при добавлении в конец $textareaValue /n, код тоже валится — плохо.

Вообще, вам следует больше уделить внимания вашему code-style, а также посмотреть в сторону xDebug.

Рабочий код:

$textareaValue = 'aaa@aaa|a|aa/nbbb@bbb|bbb/nggg@ggg|ggg/n';
$explodedTextarea = explode('/n', $textareaValue);

$result = array();
foreach ($explodedTextarea as $line) {
	list($email, $password) = array_pad(explode('|', $line, 2), 2, null);
	
	if ($email) { 
		$result[$email] = $password;
	}
}

print_r($result);

Comments

[Orange___XD]

По поводу исключения - да, мой косяк, скобочки квадратные забыл, в оригинальном коде были квадратные скобки. В пароле символ | не встречается, вот =) Про добавление в конец $textareaValue /n не понял, что вы имели ввиду. Про code-style - Phpstorm сам выставляет tab'ом =) А за xDebug спасибо, щас почитаю про него и попробую. Ну и за код тоже, щас будем разбираться.

[Константин Чухломин]

> В пароле символ | не встречается
Если пароли придумают сами пользователи, такой сценарий стоит учесть.

> Про добавление в конец $textareaValue /n не понял, что вы имели ввиду
Я не знаю, откуда берется строка в $textareaValue, но представьте, что она может оканчиваться на /n, как в моем примере.

> Про code-style - Phpstorm сам выставляет tab'ом
Тогда в PHPStorm выберите в меню Code > Reformat Code на весь файл.

[Orange___XD]

Понял, запомнил =)