Nieintuicyjne zachowanie foreach, błąd php czy złe oczekiwania Opcje

[mike]

Witam,

Chciałbym się podzielić pewnym spostrzeżeniem. Kawałek kodu na początek:

[PHP] pobierz, plaintext 
<?php
 
$strColumn = 'surname';
 
$arrColumns = array( 'surname', 'name', 'email' );
 
foreach( $arrColumns as $intId => $strColumn )
{
	echo $intId . ' => ' . $strColumn . '<br />';
}
 
echo '<br />' . $strColumn; // wyświetli `email` a powinno wyświetlić `surname`
 
?>
[PHP] pobierz, plaintext 

Powiedzcie mi czy ja za wiele wymagam, żeby lokalna zmienna $strColumn była faktycznie lokalna?
Dlaczego się czepia wcześniej ustawionej zmiennej?

Czy to jets bug php czy jest to błąd w mojej interpretacji i moich oczekiwaniach?

Już klika lat piszęw php i dopiero teraz to zauważyłem, ...., a może wcześniej nie miałem nigdy takiej sytuacji, ..., ale to raczej wykluczone.

Wersja php na jakiej pracuję to 5.1.2.

(tylko bez głupich komentaży o Przedszkolu (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) )

[pEbE]

No nic dziwnego jak zmieniasz wartosc zmiennej...

[nospor]

zmienne lokalne dotyczą funkcji, a nie kawalkow kodu objętych {}. No przykro mi, ale to trafia na przedszkole (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
http://pl.php.net/manual/pl/language.variables.scope.php

[mike]

No kurcze,

aż dziw bierze że dopiero teraz mi się to przytrafiło (IMG:http://forum.php.pl/style_emoticons/default/laugh.gif)

Człowiek uczy się całe życie.

[dr_bonzo]

Stane w obronie mike_mecha: patrzcie na to (C++ nie C):

Kod

#include <stdio.h>

int main()
{
        int i = 2323;
        for ( int i = 0; i < 10; i++ )
        {
                printf( "i = %d\n", i );

        }

        printf( "\ni = %d", i );
        return 0;
}

Kod

$ ./C.bin
i = 0
i = 1
i = 2
i = 3
i = 4
i = 5
i = 6
i = 7
i = 8
i = 9

i = 2323

Skoro php jest bazowane na C/C++ mozna bylo oczekiwac takiego zachowania (no chyba za na jakiejs stronie manuala 4pt czcionka jest napisane ze to dziala inaczej (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) )

[mike]

Dokładnie coś takiego miałem na myśli skoro w konstrukcji

Kod

foreach(wyrażenie_tablicowe as $klucz => $wartość)

Podczas każdej iteracji $klucz otrzymuje wartość aktualnego klucza a $wartość otrzymuje wartość aktualnego elemantu tablicy i są to zmienne jakoby lokalne dla pętli foreach to nie powinny się czepiać tego co na zewnątrz.

Ja to uważam za niekonsekwencję, bo skoro jest tak jak jest do niby dlaczego nie jest od nas wymagane pisanie:

[PHP] pobierz, plaintext 
<?php
 
$arrColumns = array( 'surname', 'name', 'email' );
 
$intId	 = null; // wcześniejsza deklaracja zmiennych, to oczywiście nie ma sensu
$strColumn = null; // ale jeśli wartość byłaby tablicą to już tak
foreach( $arrColumns as $intId => $strColumn )
{
	echo $intId . ' => ' . $strColumn . '<br />';
}
 
?>
[PHP] pobierz, plaintext 

Ja uważam zachowanie php w tym miejscu za nieintuicyjne.

[nospor]

Ja to uważam za niekonsekwencję, bo skoro jest tak jak jest do nieby dlaczergo nie jest od nas wymagane pisania:

Ale czemu? php nie wymaga inicjacji (zadeklarowania) zmiennej, przed przypisaniem jej wartości

ps: przenosze na php bo mnie zywcem zjecie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

edit: no i dla mnie to nie jest nieintuicyjne. normalka. lokale w funkcjach, wszystko inne nie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[mike]

Ja to uważam za niekonsekwencję, bo skoro jest tak jak jest do nieby dlaczergo nie jest od nas wymagane pisania:

Ale czemu? php nie wymaga inicjacji (zadeklarowania) zmiennej, przed przypisaniem jej wartości

Oj, no wiesz. Tego to mi mówić nie musisz.

Chodziło mi w tym przykładzie (który jest trochę bardziej hipotetyczny niż realny), że skoro to co jest wewnątrz pętli foreach wpływa na resztę to powinno to być powiedziane.

A po za tym. Zmienne tablicowe trzeba deklarować wcześniej bo dostaniesz E_NOTICE. I dokładnie o to mi chodziło (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Nieintuicyjne. Może nie aż tak bardzo. Po prostu miałem taką sytuację, że mnie to zaskoczyło. I uświadomiłem sobie że nie jest to najlepiej rozwiązane.

[nospor]

A po za tym. Zmienne tablicowe trzeba deklarować wcześniej bo dostaniesz E_NOTICE.

zmienne tablicowe tak. ale od kiedy index tablicy to tablica? pozatym foreach inicjuje to sam w sobie.
No i pozatym tablice napewno nie inicjuje się przez przypisanie jej null'a (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[scanner]

Jesli dobrze pamiętam ze szkoły (dawno nie praktykowałem C++) to tam można zrobić lokalny fragment kodu poprzez samo postawienie klamer - wszystko co wewnątrz nich jest jest lokalne i już.

[MaKARON]

Sto lat temu to robilem, ale przegladalem zrodla kernela linuksa i wydaje mi sie, ze w C tez mozna cos takiego zrobic, ale uzywalo sie haka:

do {
// tutaj kod
} while( 0 );

[mike]

(...) zmienne tablicowe tak. ale od kiedy index tablicy to tablica? pozatym foreach inicjuje to sam w sobie.

Tu chodziło mi o wartości, które tablicami już mogą być :-)

A właśnie o to "sam w sobie" mi chodzi. Skoro on robi do własnych celów to powinno być to o zasięgu lokalnym.

[kszychu]

MIke, zupełnie nie rozumiem, dlaczego takie zachowanie foreach jest dla Ciebie dziwne. Przecież dokładnie tak samo działają inne pętle (for).
Poza tym, napisałeś, że tablice trzeba wcześniej deklarować.... niekoniecznie:

[PHP] pobierz, plaintext 
<?php
 
$tablica[] = $wartosc; // to wywali notice
$tablica[$indeks] = $wartosc; // ale to już nie
 
?>
[PHP] pobierz, plaintext 

[anopak]

z jednej strony mike ma rację, bo np jeżeli includujemy jakiś kod z pętlą forech i nie wiemy jakie tam są zmienne, to może nam to nieźle namieszać w kodzie...

co do tablice, to:

[PHP] pobierz, plaintext 
<?php
$tablica[]=1;
$tablica[$in]=2;
 
?>
[PHP] pobierz, plaintext 

nie wyrzyca mi błędów E_NOTICE, ani pierwsze ani drugie...

Ten post edytował anopak 22.03.2006, 00:36:18

[nospor]

bo np jeżeli includujemy jakiś kod z pętlą forech i nie wiemy jakie tam są zmienne, to może nam to nieźle namieszać w kodzie...

Ale co to za argumentacja? Rownie dobrze mozemy includowac kod bez pętli i nie wiedziec jakie ma zmienne i tez moze nam "namieszac w kodzie" (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[bigZbig]

mike_mecha zaskoczylo takie zachowanie poniewarz przenosi swoje doswiadczenia z innych jezykow na php, ale w php jest to wlasnie w ten sposob ustalone, co mnie akurat wydaje sie calkiem logiczne bo jesli klamry mialyby okreslac granice zasiegu zmiennych to przy pelnej konsekwencji powinnismy otrzymac cos takiego:

[PHP] pobierz, plaintext 
<?php
$aA = array('kicha', 'micha', 'rycha');
$aB = array('bleble');
foreach($aA as $sKey => $sValue) {
  //powinna nastapic inicjalizacja zmiennej lokalnej
  $aB[$sKey] = 'my_'.$sValue;
}
print_r($aB);
//powinno zwrocic array('0' => 'bleble');
?>
[PHP] pobierz, plaintext 

Zauwazcie ze w php nie trzeba inicjalizowac zmiennych dlatego gdyby klamry wyznaczaly zasieg zmiennych php musialoby podjac decyzje czy tablica $aB jest kontynuacja zmiennej zainicjalizowanej poza petla, czy nalezy zainicjalizowac nowa zmienna.

W ActionScripcie jest to rozwiazane dzieki instrukcji var inicjalizujacej zmienna

Kod

i:Number = 10;
A_array = new Array(1, 2, 3);

for(var i:Number = 0; i < A_array.length; i++) {
  // zwroci kolejno 0 1 2
  trace(i);
}

// zwroci 10
trace(i);

for(i = 0; i < A_array.length; i++) {
  // zwroci kolejno 0 1 2
  trace(i);
}

// zwroci 2
trace(i);

[anopak]

Ale co to za argumentacja? Rownie dobrze mozemy includowac kod bez pętli i nie wiedziec jakie ma zmienne i tez moze nam "namieszac w kodzie"

yh, faktycznie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Zauwazcie ze w php nie trzeba inicjalizowac zmiennych dlatego gdyby klamry wyznaczaly zasieg zmiennych php musialoby podjac decyzje czy tablica $aB jest kontynuacja zmiennej zainicjalizowanej poza petla, czy nalezy zainicjalizowac nowa zmienna.

no niby tak jest, ale mogłoby to być rozwiązane np tak:

[PHP] pobierz, plaintext 
<?php
 
foreach($tablica){
 echo $this->key .'=>'.$this->value.'<br>';
}
 
?>
[PHP] pobierz, plaintext 

i wtedy nie było by problemów z nadpisaniem zmiennych, tylko byłby kolejny problem z wykorzystaniem takiej konstrukcji w klasach (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

Ten post edytował anopak 22.03.2006, 12:26:41

[bigZbig]

@anopak - sam sobie odpowiedziales

[anopak]

@bigZbig - no niby tak, ale to mogły być poprostu zmienne zarezerwowane (tak jak $this)... no cóż może w któreś wersji ogólnie będzie to inaczej rozwiązane (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[bigZbig]

@anopak - no OK, ale potrzebne Ci to? Moim zdaniem wystarczy izolowanie zmiennych w funkcjach i klasach.

Ten post edytował bigZbig 22.03.2006, 15:02:46