Помогите с циклом.

[komyak]

Пишу себе граббер(загрузка страниц, распасивание и подготовка sql-дампа). Всё получилось, кроме:
Моя строка:

$text = trim("VALUES (".$y.", '', '', '".$match2[2][$k]."', '', '', '".$match2[1][$k]."', '', '', '1', '".$y."');");

Должно получиться:

VALUES (1, '', '', 'text', '', '', 'text', '', '', '1', '1')
VALUES (2, '', '', 'text', '', '', 'text', '', '', '1', '2')
VALUES (3, '', '', 'text', '', '', 'text', '', '', '1', '3')
и т.д. Нужно, чтоб с каждой записью массива, значения id и post увеличивались на один.

Мой

for($k=0;$k<=count($match2[1]);$k++){
for($y=0;$y<=100000);$y++){
$text = trim("VALUES (".$y.", '', '', '".$match2[2][$k]."', '', '', '".$match2[1][$k]."', '', '', '1', '".$y."');");
fwrite($fh, $text."\n");}}}}
fclose($fh);

Я что-то перемудрил и в итоге на серваке за 4-6 секунд оказался файлик на 5 гигов... А должен был на 6 метров...
Подскажите...

 

[Eihwaz]

При каждой итерации вашего первого цикла, второй 100000 раз записывает строку в файл.

$y = 1;
for($k=0;$k<=count($match2[1]);$k++){
    $text = trim("VALUES (".$y.", '', '', '".$match2[2][$k]."', '', '', '".$match2[1][$k]."', '', '', '1', '".$y."');");
    fwrite($fh, $text."\n");
    $y++;
}
fclose($fh);

 

[komyak]

Не проходит, $y остаётся = 1:

VALUES (1, '', '', 'text', '', '', 'text', '', '', '1', '1') 
VALUES (1, '', '', 'text', '', '', 'text', '', '', '1', '1') 
VALUES (1, '', '', 'text', '', '', 'text', '', '', '1', '1') 
.....

Попробовал так:

$y = 1;
for($k=0;$k<=count($match2[1]);$k++){
$text = trim("VALUES (".$y.", '', '', '".$match2[2][$k]."', '', '', '".$match2[1][$k]."', '', '', '1', '".$y."');");
$y++;
fwrite($fp, $text."\n");
}

Но в итоге получается:

VALUES (1, '', '', 'text', '', '', 'text', '', '', '1', '1') 
VALUES (2, '', '', 'text', '', '', 'text', '', '', '1', '2')
VALUES (1, '', '', 'text', '', '', 'text', '', '', '1', '1') 
VALUES (2, '', '', 'text', '', '', 'text', '', '', '1', '2')
.....

В $match2 ~ 100к строк, может надо указывать максимальное значение - $y <= 1; ?

 

[Eihwaz]

У вас там выше по коду где-то есть циклы (хотя по-идее их быть не должно) — смотрите в них

 

[komyak]

У вас там выше по коду где-то есть циклы (хотя по-идее их быть не должно) — смотрите в них

Есть ешё 2 цикла. Без них не получается выдёргивать.

 

[Sanuch]

и получается, что для каждой страницы, откуда вы берете урлы, вы "обнуляете" значение переменной $y, вынесите $y = 1; за пределы циклов.

 

[komyak]

Всё парни спасибо! Теперь и парсит и записывает и считает. Спасибо!
Добавлено через 4 минуты

ПС: странно, что у вас preg_match_all на регулярное выражение не ругается с неэкранированным слешом

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

<div>(.*?)<\/div>

он не хотел ничего выдавать. Также и через print_r смотрел.

 

[ShaDeRzz]

Хорошо что всё получилось..
Ещё один совет:
Когда используете такую конструкцию:

for($k=0;$k<=count($match2[1]);$k++)

Выносите

count($match2[1])

за скобки, ато при каждой итерации у вас заново просчитывается количество элементов в массиве, а при 100К итераций - это большая лишняя нагрузка на сервер.
Тоесть в итоге надо писать так:

$myparam = count($match2[1]);
for($k=0;$k<=$myparam;$k++){
......
}