[PHP] Skrypt na usuwanie wpisów z pliku tekstowego Opcje

[sloenthran]

Witam serdecznie!

Otóż mam pewien problem a mianowicie napisałem sobie skrypt usuwający wpisy z pewnego pliku... Plik zostaje prawidłowo pobrany jednakże wpis nie zostaje mi z niewiadomych przyczyn usunięty... Jeżeli ktoś mógłby mi z tym pomóc to byłbym wdzięczny...

nick.php

[PHP] pobierz, plaintext 
<?php
 
	require_once('funkcje/MySQL.php');
	require_once('funkcje/Zakupy.php');
	require_once('funkcje/Pliki.php');
 
	OtworzMySQL();
 
	$Zapytanie = @mysql_query("SELECT * FROM serwery");
 
	while($Pobieranie = @mysql_fetch_array($Zapytanie))
	{
 
		$Serwer = $Pobieranie['nazwa'];
 
		$ZapytanieDwa  = @mysql_query("SELECT * FROM $Serwer");
 
		while($PobieranieDwa = @mysql_fetch_array($ZapytanieDwa))
		{
 
			$Waznosc = $PobieranieDwa['waznosc'];
			$UserID  = $PobieranieDwa['id'];
			$Login   = $PobieranieDwa['login'];
			$Haslo   = $PobieranieDwa['haslo'];
			$Flaga   = $PobieranieDwa['flaga'];
 
			if($Waznosc < 0 || $Waznosc == 0)
			{
 
				$ZapytanieTrzy  = @mysql_query("SELECT * FROM serwery WHERE nazwa='$Serwer'");
				$PobieranieTrzy = @mysql_fetch_array($ZapytanieTrzy);
 
				$FTPHOST  = $PobieranieTrzy['ftp_host'];
				$FTPLOGIN = $PobieranieTrzy['ftp_login'];
				$FTPHASLO = $PobieranieTrzy['ftp_haslo'];
				$SCIEZKA  = $PobieranieTrzy['sciezka'];
 
				$SumowanieFlagJeden = @mysql_query("SELECT * FROM $Serwer WHERE id='$UserID' and login='$Login'");
 
				while($SumowanieFlagDwa = mysql_fetch_array($SumowanieFlagJeden))
				{
 
					$SumowanieFlagTrzy = $SumowanieFlagDwa['flaga'];
 
					$Admin = $SumowanieFlagDwa['haslo'];
 
					if($Admin = '-') { }
 
					else 
					{ 
 
						$SumowanieFlagCztery = $SumowanieFlagCztery.$SumowanieFlagTrzy;
 
					}
 
				}
 
				$FlagaZ = 'z';
				$SumowanieFlagPiec = $SumowanieFlagCztery.$FlagaZ;
 
				PobierzUsers($FTPLOGIN, $FTPHASLO, $FTPHOST, $SCIEZKA);
 
				$LinijkaUsers = SprawdzUsers($Login, $Haslo, $SumowanieFlagPiec);
 
				UsunUsers($LinijkaUsers);
 
				WrzucUsers($FTPLOGIN, $FTPHASLO, $FTPHOST, $SCIEZKA);
 
				@mysql_query("DELETE FROM $Serwer WHERE flaga='$Flaga' AND id='$UserID' AND login='$Login' AND haslo='$Haslo'");
 
				$WYNIK = @mysql_query("SELECT * FROM $Serwer WHERE login='$Login' AND id='$UserID'");
 
				if(@mysql_num_rows($WYNIK) == 1)
				{
 
					$DwaSumowanieFlagJeden = @mysql_query("SELECT * FROM $Serwer WHERE id='$UserID'");
 
					while($DwaSumowanieFlagDwa = mysql_fetch_array($DwaSumowanieFlagJeden))
					{
 
						$DwaSumowanieFlagTrzy = $DwaSumowanieFlagDwa['flaga'];
 
						$DwaAdmin = $DwaSumowanieFlagDwa['haslo'];
 
						if($DwaAdmin = '-') { }
 
						else 
						{ 
 
							$DwaSumowanieFlagCztery = $DwaSumowanieFlagCztery.$DwaSumowanieFlagTrzy;
 
						}
 
					}
 
					$ZsumowaneFlagi = $DwaSumowanieFlagCztery.$FlagaZ;
 
					PobierzUsers($FTPLOGIN, $FTPHASLO, $FTPHOST, $SCIEZKA);
 
					DodajUsers($Login, $Haslo, $ZsumowaneFlagi);	
 
					WrzucUsers($FTPLOGIN, $FTPHASLO, $FTPHOST, $SCIEZKA);
 
				}
 
			}
 
		}
 
	}
 
	ZamknijMySQL();
 
?>
 
[PHP] pobierz, plaintext 

pliki.php

[PHP] pobierz, plaintext 
<?php
 
	function PobierzUsers($FTPLOGIN, $FTPHASLO, $FTPHOST, $SCIEZKA)
	{
 
		$Download = curl_init('ftp://'.$FTPLOGIN.':'.$FTPHASLO.'@'.$FTPHOST.'/'.$SCIEZKA.'');
		$Open = fopen('users.ini', 'w');
		curl_setopt($Download, CURLOPT_FILE, $Open);
		curl_exec($Download);
		fclose($Open);
		curl_close($Download);
 
	}
 
	function WrzucUsers($FTPLOGIN, $FTPHASLO, $FTPHOST, $SCIEZKA)
	{
 
		$Upload = curl_init('ftp://'.$FTPLOGIN.':'.$FTPHASLO.'@'.$FTPHOST.'/'.$SCIEZKA.'');
		$Open = fopen('users.ini', 'r');
		curl_setopt($Upload, CURLOPT_INFILE, $Open);
		curl_setopt($Upload, CURLOPT_UPLOAD, true);
		curl_exec($Upload);
		fclose($Open);
		curl_close($Upload);
 
		unlink('users.ini');
 
	}
 
?>
[PHP] pobierz, plaintext 

zakupy.php

[PHP] pobierz, plaintext 
<?php	
 
	function SprawdzStatusHosta($IP, $Port)	
	{			
 
		if($Query = @fsockopen("$IP", "$Port", $errno, $errstr, 30))		
		{					
 
			$SprawdzStatusHosta = 1;						
			fclose($Query);					
 
		}				
 
		else		
		{					
 
			$SprawdzStatusHosta = 0;				
 
		}				
 
		return $SprawdzStatusHosta;			
 
	}		
 
	function DodajUsers($Login, $Haslo, $Flagi)
	{			
 
		$OtworzPlik = fopen('users.ini', "a");		
 
		if($OtworzPlik)		
		{				
 
			$DodajPremke = fwrite($OtworzPlik, '"'.$Login.'" "'.$Haslo.'" "'.$Flagi.'" "a"');		
 
			if($DodajPremke)		
			{							
 
				$NowaLinia = fwrite($OtworzPlik, "\r\n");	
 
				if($NowaLinia)			
				{						
 
					fclose($OtworzPlik);							
				}							
 
			}					
 
		}		
 
	}		
 
	function SprawdzUsers($Login, $Haslo, $Flagi)	
	{			
 
		$Tablica = file('users.ini', FILE_IGNORE_NEW_LINES);     		
		$PozycjaRekordu = array_search('"'.$Login.'" "'.$Haslo.'" "'.$Flagi.'" "a"', $Tablica);		
 
		return $PozycjaRekordu;		
 
	}		
 
	function UsunUsers($Linijka)	
	{			
 
		$Dane = file('users.ini');				
 
		if($Linijka AND $Dane)		
		{					
 
			unset($Dane[$Linijka--]);				
 
			$OtworzPlik = fopen($Plik, 'w');					
 
			if($OtworzPlik)			
			{						
 
				$NadpiszPlik = fwrite($OtworzPlik, join('',$Dane));	
 
				if($NadpiszPlik)		
				{						
 
					fclose($OtworzPlik);	
 
				}						
 
			}		
 
		}		
 
	}	
 
	function DodajAdmina($Login, $Flagi)	
	{			
 
		$OtworzPlik = fopen('users.ini', "a");	
 
		if($OtworzPlik)		
		{					
 
			$DodajPremke = fwrite($OtworzPlik, '"'.$Login.'" "" "'.$Flagi.'" "ce"');	
 
			if($DodajPremke)			
			{								
 
				$NowaLinia = fwrite($OtworzPlik, "\r\n");				
 
				if($NowaLinia)				
				{								
 
					fclose($OtworzPlik);							
 
				}							
 
			}					
 
		}		
 
	}		
 
	function SprawdzAdmina($Login, $Flagi)	
	{			
 
		$Tablica = file('users.ini', FILE_IGNORE_NEW_LINES);     		
 
		$PozycjaRekordu = array_search('"'.$Login.'" "" "'.$Flagi.'" "ce"', $Tablica);				
 
		return $PozycjaRekordu;		
 
	}	
 
?>
[PHP] pobierz, plaintext 

[mlawnik]

Najpierw wywal zagłuszanie błędów i daj error_reporting na E_ALL, może coś wyskoczy.
Sprawdź także prawa zapisu do plików.

[b4rt3kk]

Jednego jestem pewien - masz źle skonstruowaną bazę danych. Co widać po kodzie pierwszego pliku, który zamieściłeś. Tyle zapytań w pętlach, które można by śmiało zredukować do jednego. Nie prościej mieć tak?

Tabela: Serwery - id, nazwa, ftp, itd...
Tabela: Users - id, id_serwer, login, haslo, itd...

To co realizujesz tam poprzez miliard zapytań mógłbyś zrobić jednym:

[SQL] pobierz, plaintext 
SELECT * FROM Serwery JOIN Users ON Serwery.id = Users.id_serwer
[SQL] pobierz, plaintext 

[sloenthran]

Najpierw wywal zagłuszanie błędów i daj error_reporting na E_ALL, może coś wyskoczy.
Sprawdź także prawa zapisu do plików.

Nic nie wyskakuje...

Jednego jestem pewien - masz źle skonstruowaną bazę danych. Co widać po kodzie pierwszego pliku, który zamieściłeś. Tyle zapytań w pętlach, które można by śmiało zredukować do jednego. Nie prościej mieć tak?

Tabela: Serwery - id, nazwa, ftp, itd...
Tabela: Users - id, id_serwer, login, haslo, itd...

To co realizujesz tam poprzez miliard zapytań mógłbyś zrobić jednym:

[SQL] pobierz, plaintext 
SELECT * FROM Serwery JOIN Users ON Serwery.id = Users.id_serwer
[SQL] pobierz, plaintext 

Może i prościej jednakże w PHP "siedzę" dopiero od jakiegoś czasu i nie potrafię inaczej konstruować baz danych...

[b4rt3kk]

Ta funkcja ma usuwać ową linijkę z pliku, prawda?

[PHP] pobierz, plaintext 
function UsunUsers($Linijka)	
	{			
 
		$Dane = file('users.ini');				
 
		if($Linijka AND $Dane)		
		{					
 
			unset($Dane[$Linijka--]);				
 
			$OtworzPlik = fopen($Plik, 'w');					
 
			if($OtworzPlik)			
			{						
 
				$NadpiszPlik = fwrite($OtworzPlik, join('',$Dane));	
 
				if($NadpiszPlik)		
				{						
 
					fclose($OtworzPlik);	
 
				}						
 
			}		
 
		}		
 
	}
[PHP] pobierz, plaintext 

Jakbyś przeanalizował krok po kroku ową funkcję (np. wyświetlając kawałek po kawałku wynikową tablicę, to byś zauważył, że ten warunek nie ma sensu:

[PHP] pobierz, plaintext 
if($Linijka AND $Dane)
[PHP] pobierz, plaintext 

Po pierwsze, chcesz tutaj porównać tablicę (powstałą przez otwarcie pliku) ze (tak zakładam) stringiem. Wyświetlałeś jedno i drugie? A po drugie do porównania nie służy AND tylko ==, względnie ===. Jak powinna wyglądać ta funkcja?

[PHP] pobierz, plaintext 
function UsunUsers($Linijka) {			
 
  $dane = file('users.ini');				
 
  foreach ($dane as $key => $value) {
 
    // dla pewności wyświetl oba porównywane stringi!!!
    var_dump($value);
    echo ' ?= ';
    var_dump($Linijka);
 
    if ($value == $Linijka) {
      $dane[$key] = '';
      unset($dane[$key]);
    }
 
  }
 
  // teraz możesz zapisać wynik do pliku
  file_put_contents($Plik, $dane);
}
[PHP] pobierz, plaintext 

Tak na marginesie, nie przesadzasz z tymi zmiennymi globalnymi?

Zbudowanie ergonomicznej i wygodnej bazy danych nie wymaga znajomości PHP, wystarczy troszkę logiki.

Ten post edytował b4rt3kk 27.06.2013, 18:49:34