IF spełniony za każdym razem w pętli FOR, trudno opisać... zobaczcie sami Opcje

[tomilipin]

Cześć!
Natrafiłem na bardzo dziwny problem z pętlą FOR... Może ktoś z Was będzie w stanie mi pomóc. Działanie tej pętli mam już opanowane, widzę że nie zrobiłem w niej błędu (lub błędu nie widzę). Poniższy kod to wycinek skryptu do prostej listy sybskrypcyjnej .

Chodzi o to, że za każdym razem, nie ważne który adres e-mail wpiszę- zostaje usunięty ostatni adres! Poszczególne wpisy mam oddzielone poprzez \n . To dziwne zachowanie pętli FOR i IF polega na tym, że warunek zostaje spełniony za każdym razem, w wyniku czego $usunnumer ma wartość taką, ile razy została wykonana pętla- bo za każdym razem warunek w IFie jest spełniony, i za każdym wykonaniem pętli wykonywana jest także zawartość IFa Próbowałem już przestawiać zmienne $adresy i $adres na różne sposoby, efekt ciągle ten sam lub jeszcze gorszy (warunek nie był spełniony).

Czy ktoś mógłby wskazać, gdzie jest błąd? Zapewne chodzi o malutki błąd logiczny bo składniowych nie mam. Bardzo proszę o pomoc...
Jeśli brakuje jakichś danych- proszę pisać, od razu podeślę na forum brakujący kod.

[PHP] pobierz, plaintext 
<?
//sprawdzam co wciśnięto i czy coś wpisano
if ($_POST['usu'] and $_POST['adres']) {
$adres=$_POST['adres']; //dla późniejszego uproszczenia
$adresy = file('lista.adr'); //traktuję plik jako tablicę
 
	if (!eregi(&#092;"^.+@.+..+$\", $_POST['adres'])) {
	exit(&#092;"Niepoprawny adres e-mail...\");
	}
		  for ($index=&#092;"0\"; $index < count($adresy); $index++) {
				  if($adresy[$index]=$adres) {
				  echo &#092;"linijka $index \".$adresy[$index].\"<br>\"; //tylko dla testów
				  $usunnumer=$index;
				  }
 
		  }
 
$plik = file('lista.adr');
unset($plik[$usunnumer]);
$f = fopen('lista.adr', 'w');
	 foreach($plik as $linia){
	 fputs($f, $linia);
	 }
fclose($f);
echo &#092;"Podany adres e-mail został usunięty z listy.\";
?>
[PHP] pobierz, plaintext 

[bendi]

musi byc podwoje rowna sie na porownanie zamiasat pojedynczego (przypisanie)

[PHP] pobierz, plaintext 
<?php
if($adresy[$index]==$adres) {
?>
[PHP] pobierz, plaintext 

[tomilipin]

Niestety to nie to...
Po zmianie, o której piszesz, warunek nie jest spełniony i żadna linijka z pliku nie jest usuwana. Informacja "Podany adres e-mail został usunięty z listy." pojawia się, ale wynika to ze złego kodu, po prostu zawsze pojawi się ten napis jeśli ktoś wciśnie przycisk Usuń, poda jakiś adres i ten adres będzie poprawny.

Czy może gdzieś indziej jest błąd?

--EDIT--
gdy mam taki kod (z jednym znakiem równości):

[PHP] pobierz, plaintext 
<?
if($adresy[$index]=$adres) {
echo &#092;"linijka $index \".$adresy[$index].\"<br>\";
$usunnumer=$index;
}
 ?>
[PHP] pobierz, plaintext 

i podam "adres3@serwer3.pl" do usunięcia to wyświetla mi:

Kod

linijka 0 adres3@serwer3.pl
linijka 1 adres3@serwer3.pl
linijka 2 adres3@serwer3.pl
linijka 3 adres3@serwer3.pl
linijka 4 adres3@serwer3.pl

...podczas gdy powninny być kolejne adresy zaczynając od 1 do 5. Potem usuwa ostatni wpis- adres 5. Dobrze że wogóle coś wyświetla, ale ciągle usuwa ostatni wpis! A w moim pliku wpis "adres3@serwer3.pl" znajduje się w linijce 3 (licząc od 1, dla php to będzie linijka 2) bo liczy od 0). Jestem normalnie zdesperowany i zdezorientowany, bardzo proszę o pomoc...

Ten post edytował tomilipin 16.11.2004, 18:15:38

[rogrog]

ale Ty chcesz sprawdzić czy ten adres odpowiada adresowi w tablicy a nie przypisać.

Nie możesz tak myśleć "dobrze że coś się wyświetla" tylko pomyśl co Ty chcesz zrobić.

Najlepiej do przejścia po tablicy użyj foreach" title="Zobacz w manualu PHP" target="_manual. Nie wiem co jest źle tutaj (brzydki kod nie zachęca do czytania ) ale może przy foreach będzie działało.

A jeśli chcesz używać for to pamiętaj żeby nie używać count() i w ogóle funkcji w warunku (o ile to mozliwe). Najlepiej przypisać długość tablicy do zmiennej i sprawdzać czy index jest mniejszy od tej zmiennej, bo funkcja jest wtedy wywoływana tylko raz. A jeśli umieścisz ją w warunku pętli to jest wykonywana za kazdym jej przebiegiem.

Dodatkowo $index jest liczbą a nie stringiem więc przypisuj $index=0 a nie $index="0"

[tomilipin]

Rozumiem. Zmieniłem = na ==.
Teraz moja pętla ma postać:

[PHP] pobierz, plaintext 
<?
$adres=$_POST['adres'];
$adresy = file('lista.adr');
$asdf=count($adresy);
 
for ($index=0; $index < $asdf; $index++) {
   if($adresy[$index]== $adres) {
   echo &#092;"linijka $index \".$adresy[$index].\"<br>\";
   $usunnumer=$index;
   }
 
}
 
$plik = file('lista.adr');
unset($plik[$usunnumer]);
$f = fopen('lista.adr', 'w');
	 foreach($plik as $linia){
	 fputs($f, $linia);
	 }
fclose($f);
echo &#092;"Podany adres e-mail został usunięty z listy.\";
?>
[PHP] pobierz, plaintext 

Przepraszam za brzydki kod... jeśli taki faktycznie jest...
Nie zmienia to jednak faktu że nie działa Żadna linia z pliku nie zostaje usunięta...

[silent]

A nie łatwiej zrobić to tak:

[PHP] pobierz, plaintext 
<?php
	$adres  = $_POST['adres']; // to wiesz
	$adresy = file( 'lista.adr' ); // to też wiesz
 
	// obcina białe znaki we wszystkich wartościach w tablicy $adresy
	$adresy = array_map( 'trim', $adresy );
 
	// zwraca do zmiennej $index indeks pod którym w tablicy $adresy znajduje się ciąg
$adres
	$index = array_search( $adres, $adresy );
 
	// usuwa z tablicy $adresy wartość pod indeksem $index (czyli adres który chcieliśmy usunąć)
	unset( $adresy[$index] );
 
	// zapisuje do pliku zawartość tablicy $adresy (już bez adresu który chcieliśmy usunąć)
	$f = fopen( 'lista.adr', 'w' );
	foreach( $adresy as $linia )
	{
		 fputs( $f, $linia );
	}
	fclose( $f );
 
	echo &#092;"Podany adres e-mail został usunięty z listy.\";
?>
[PHP] pobierz, plaintext 

Ten post edytował silent 18.11.2004, 00:16:17

[kszychu]

Funkcja file() rzeczywiście zwraca plik do tablicy, jednak każdy element tablicy zawoiera na końcu znak nowej linii. Podejrzewam, że tu właśnie tkwi problem. Polecam porównywać to tak:

[PHP] pobierz, plaintext 
<?php
if(rtrim($adresy[$index]) == $adres)
?>
[PHP] pobierz, plaintext 

[tomilipin]

Dzięki Kszychu! Działa
Niech Ci Bozia w dzieciach wynagrodzi

Silent- nie rozumiem kodu, który podaleś. Jeszcze jestem w tym za słaby, ale dzięki za pomoc! Napewno to sobie przyswoję

Jeszcze raz: dzięki Wam za pomoc!

--EDIT--
dodam, że efekt końcowy można zobaczyć na mojej stronie domowej, do której link znajduje się poniżej- w podpisie.

Ten post edytował tomilipin 17.11.2004, 15:51:39

[silent]

Dodałem komentarze i poprawkę kszycha, może teraz Ci się rozjaśni