Forum PHP.pl > [PHP] Problem z pętlą

Pomoc - Szukaj - Użytkownicy - Kalendarz

RaVeno

26.02.2012, 01:28:05

Mam problem z pętlą która ma usuwać wpisy z pliku, owszem usuwa, ale pojedynczo, tj. pętla po usunięciu wpisu kończyła by się, zamiast wrócić do początku. Powinno usunąć wszystkie wpisy spełniające warunek.

plik php

[PHP] pobierz, plaintext 
<?php
if(file_exists('vip.list'))
{
	$vip = file('vip.list');
	$x = 0;
	for($i = 0; $i <= count($vip) - 1; $i++)
	{
		@$explode = explode(' || ', $vip[$i]);
		if(isset($explode[0]) && isset($explode[2]) && isset($explode[3]))
		{
			$end2 = round((strtotime(trim($data[3])) - strtotime(date('d.m.Y'))) / 86400);
			if($end <= 0)
			{
				$x = $x + 1;
				unset($vip[$i]);
				$vip = implode($vip);
				$file = fopen('vip.list', 'w+');
				fwrite($file, $vip);
				fclose($file);
			}
		}
	}
	echo 'Usunięto '.$x.' rekordów';
}
?> 
[PHP] pobierz, plaintext

plik vip.list

Kod

Serwer || nick || 22.01.2012 || 25.02.2012
Serwer || nick1 || 22.01.2012 || 25.02.2012
Serwer || test2 || 22.01.2012 || 24.02.2012
Serwer || lalal || 24.01.2012 || 26.02.2012
Serwer || kotek || 24.01.2012 || 24.02.2012
Serwer || piesek || 22.01.2012 || 23.02.2012
Serwer || nick5 || 22.01.2012 || 22.02.2012
Serwer || kotali || 22.01.2012 || 21.02.2012

CuteOne

26.02.2012, 02:41:56

Ja bym to zrobił inaczej - zamiast usuwać coś z pliku stwórz sobie zmienną, która przechowa tylko te dane, które spełnią warunek. Warunek jest odwrotny do tego który masz w kodzie czyli zostawia tylko te wiersze, których ma nie usuwać.

[PHP] pobierz, plaintext 
$content = ''
if($end2 > 0) {
 
   $content .= $vip[$i] . "\n\r"; // albo \r\n, \n juz nie pamietam ;p
}
 
file_put_contents('vip.txt', $content);
[PHP] pobierz, plaintext

ps. hmm if($end <= 0) ? a nie $end2?

RaVeno

26.02.2012, 13:30:32

Tak faktycznie tam jest $end

OK, mam tak jak mi poleciłeś, jednak nie mam pojęcia dlaczego, ale nie dopisuje do pliku, chociaż zmienna przekazuje dane dobrze. Plik jest pusty po wywołaniu.

[PHP] pobierz, plaintext 
<?php
if(file_exists('vip.list'))
{
	$vip = file('vip.list');
	$x = 0;
	for($i = 0; $i <= count($vip) - 1; $i++)
	{
		@$explode = explode(' || ', $vip[$i]);
		if(isset($explode[0]) && isset($explode[2]) && isset($explode[3]))
		{
			$end = round((strtotime(trim($explode[3])) - strtotime(date('d.m.Y'))) / 86400);
			$content = '';
			if($end > 0) {
				$x = $x+1;
			   $content .= $vip[$i] . "\r\n";
 
			}
			echo $content;			 
			file_put_contents('vip.txt', $content);			
		}
	}
	echo 'Usunięto '.$x.' rekordów';
}
?>   
[PHP] pobierz, plaintext

CuteOne

26.02.2012, 16:20:18

[PHP] pobierz, plaintext 
if(file_exists('vip.list'))
{
	$vip = file('vip.list');
	$x = 0;
	for($i = 0; $i <= count($vip) - 1; $i++)
	{
		@$explode = explode(' || ', $vip[$i]);
		if(isset($explode[0]) && isset($explode[2]) && isset($explode[3]))
		{
			$end = round((strtotime(trim($explode[3])) - strtotime(date('d.m.Y'))) / 86400);
			$content = '';
			if($end > 0) {
				$x = $x+1;
			   $content .= $vip[$i] . "\r\n";
 
			}
			echo $content;			 			
		}
	}
file_put_contents('vip.txt', $content);
	echo 'Usunięto '.$x.' rekordów';
}
[PHP] pobierz, plaintext

RaVeno

26.02.2012, 16:47:36

Tak też już próbowałem...

CuteOne

26.02.2012, 18:41:33

[PHP] pobierz, plaintext 
if(file_exists('vip.list'))
{
	$vip = file('vip.list');
	$x = 0;
$content = '';
	for($i = 0; $i <= count($vip) - 1; $i++)
	{
		@$explode = explode(' || ', $vip[$i]);
		if(isset($explode[0]) && isset($explode[2]) && isset($explode[3]))
		{
			$end = round((strtotime(trim($explode[3])) - strtotime(date('d.m.Y'))) / 86400);
 
			if($end > 0) {
				$x = $x+1;
			   $content .= $vip[$i] . "\r\n";
 
			}
			echo $content;			 			
		}
	}
file_put_contents('vip.txt', $content);
	echo 'Usunięto '.$x.' rekordów';
}
[PHP] pobierz, plaintext

RaVeno

26.02.2012, 22:50:12

Dzięki wielkie, teraz działa!

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.