В чем проблема при записи в файл на php?

Question

[Кирилл Горелов]

Есть строка:
---------------------------------------
>> Москва: [phone]
>> Санкт-Петербург: [phone]
>> Регионы: [phone]

Магазин tara-tovara.ru, [e-mail]
Москва: [phone]
Санкт-Петербург: [phone]
Регионы: [phone]
---------------------------------
Получаю номер регулярными выражениями:

preg_match_all('/(Москва:|Санкт-Петербург:|Регионы:)\s*([\+\-\d\(\)\s]+)/m', $textt, $r);
$resultphone = array_unique($r);
$phone = array(); // массив для телефонов
foreach ($resultphone[0] as $v) { // перебираю телефоны для заполнения массива
$vv = trim($v);
 // echo($v.'<br>'); // для вывода
 $phone[] = $vv;// для записи
}
//удаляю дубликаты
$resultphone = array_unique($phone);

В результате получаю массив в нужном виде:
Array
(
[0] => Москва: [phone]
[1] => Санкт-Петербург: [phone]
[2] => Регионы: [phone]
)
Перебираю эти номера для записи

foreach($resultphone as $value) // запись хорошего массива
	{
	    echo $value. "\n";
	$str2.= "\n". $value; //"\n".
	}

Переменная $str2 выводит в нужном мне виде:
Москва: [phone]
Санкт-Петербург: [phone]
Регионы: [phone]
Делаю var_dump этой переменной

var_dump($str2);
/home/ubuntu/workspace/gm3.php:117:
string(421) "
Array
Москва: [phone]
Санкт-Петербург: [phone]
Регионы: [phone]
Москва:
Санкт-Петербург:
Регионы:
Москва: [phone]
Санкт-Петербург: [phone]
Регионы: [phone]
Москва: [phone]
Санкт-Петербург: [phone]
Регионы: [phone]"

И соответственно в файл записывается такая же ерунда. Почему так происходит?

Comments

[OVK2015]

У вас в коде не понятно откуда берется $resultphone[0]
Это $r из preg_match_all?

[Кирилл Горелов]

OVK2015: Да, верно. Просто я там для теста еще раз удалял дубликаты
$resultphone = array_unique($r); И забыл вставить!

[Кирилл Горелов]

OVK2015: Я пока делаю в c9.io
На самом деле. Это не весь код. я беру эти номера из файла. файлы перебираю в цикле. И каждый файл прохожу через регулярные выражения. Сейчас попробую еще. Натолкнули вы меня на одну хорошую мысль. Я переменную str2 не обнулял при просмотре каждого файла. Может она копила в себе это все. Сейчас еще раз попробую!

[OVK2015]

Кирилл Горелов: Что копила,- вполне логично.

[Кирилл Горелов]

OVK2015: Просто красавчик. Помогло. Я не обнулял этот цикл.

foreach($resultphone as $value) // запись хорошего массива
    {
       // echo $value. "\n";
   	 $str2.= "\n". $value; //"\n".
    }

И эта переменная в себе копил все номера. Спасибо!

[Кирилл Горелов]

OVK2015: Напиши в "ответах" я поставлю +. Тему можно будет закрыть!

Answer 1

[OVK2015]

$testStr = '>> Москва: [phone]
>> Санкт-Петербург: [phone]
>> Регионы: [phone]

Магазин tara-tovara.ru, [e-mail]
Москва: [phone]
Санкт-Петербург: [phone]
Регионы: [phone]';

preg_match_all('/(Москва:|Санкт-Петербург:|Регионы:)\s*([\+\-\d\(\)\s]+)/m', $testStr, $resultphone);
$phone = array(); // массив для телефонов
foreach ($resultphone[0] as $v) { // перебираю телефоны для заполнения массива
$vv = trim($v);
// echo($v.'
'); // для вывода
$phone[] = $vv;// для записи
}
//удаляю дубликаты
print_r($resultphone);
$resultphone = array_unique($phone);

$str2 = "";
foreach($resultphone as $value) // запись хорошего массива
{
// echo $value. "\n";
$str2.= "\n". $value; //"\n".
}


echo "****************\n";
echo $str2."\n";
die();
Да, кстати, насчет самой регулярки. В исходных строках ":" разграничивает название "владельца" и номер телефона? Если да,- ее упростить можно

Comments

[Кирилл Горелов]

С кодом разобрался, спасибо.
А вот на счет разграничений в регулярках не понял, что имелось в виду!

[OVK2015]

Кирилл Горелов:
Что-то вроде вот этого:
preg_match_all("#(\w+):\s*([\+\-\d\(\)\s]+)#siu", $testStr, $resultphone);
Если телефоны будут не только в "Москва", "Санкт-Петербург" и "Регионы"

[Кирилл Горелов]

OVK2015: Аааа про это. Да я об этом думал. Я сделаю по условию. Если мое условие: /(Москва:|Санкт-Петербург:|Регионы:)\s*([\+\-\d\(\)\s]+)/m Пустое, то искать телефоны без города. Спасибо.

[OVK2015]

Не за что, вроде

Answer 2

[Александр Аксентьев]

У вас все переменные по разному называются.

preg_match записывает результат в $r, а затем вы работаете уже с вообще другими названиями.

Вы вообще обрабатываете, выводите не то что надо.

Либо код кривой выложили.

Comments

[Кирилл Горелов]

Да, косяк, поправил!

[Александр Аксентьев]

Кирилл Горелов: намутили что-то, код нормальный. Никак не должен дублироваться вывод. Вычистите все тестовые варианты, оставьте рабочий. Как в вопросе.

[Кирилл Горелов]

Александр Аксентьев: Не все, разобрался. Спасибо!