[MySQL][PHP] Parametry funkcji Opcje

[agentmullder]

Witam. Dzis postanowilem zoptymalizowac moj kod. Z racji, ze sporo kodu sie powtarza, to postanowilem zastosowac funkcje. Na etapie testowania funkcji napotkalem na pewien problem, otoz:

wywolanie funkcji:

[PHP] pobierz, plaintext 
edytuj('pilka_kolor',$jakies_id);
[PHP] pobierz, plaintext 

postac funkcji edytuj:

[PHP] pobierz, plaintext 
function edytuj($a,$b)
{
		$zapytanie=mysql_query("SELECT '".$a."' FROM pilki WHERE pilka_id='".$b."'");
		$zapytanie_wykonaj = mysql_fetch_array($zapytanie);
		$d=$zapytanie_wykonaj['"'.$a.'"']; 	
 
                 echo $d;
 
}
[PHP] pobierz, plaintext 

W taki sposob funkcja nie chce mi wypisac zawartosci zmiennej $d. Podejrzewam, ze problem znajduje sie w sposobie uzycia zmiennej $a=pilka_kolor w tej funkcji, poniewaz wstawiajac w miejsce zmiennej $a w tej funkcji na pilka_kolor wszystko dziala idealnie.

Tutaj kod, ktory dziala tak jak nalezy:

[PHP] pobierz, plaintext 
function edytuj($a,$b)
{
		$zapytanie=mysql_query("SELECT pilka_kolor FROM pilki WHERE pilka_id='".$b."'");
		$zapytanie_wykonaj = mysql_fetch_array($zapytanie);
		$d=$zapytanie_wykonaj['pilka_kolor']; 	
 
                 echo $d;
 
}
[PHP] pobierz, plaintext 

Macie jakies pomysly w jaki sposob uzyc zmiennej $a w tej funkcji dla jej poprawnego dzialania?

Ten post edytował agentmullder 19.05.2010, 13:46:37

[thek]

Nie... Problemem są u Ciebie apostrofy. Sprawdź zamiast

[PHP] pobierz, plaintext 
$zapytanie=mysql_query("SELECT '".$a."' FROM pilki WHERE pilka_id='".$b."'");
[PHP] pobierz, plaintext 

choćby

[PHP] pobierz, plaintext 
$zapytanie=mysql_query('SELECT '.$a.' FROM pilki WHERE pilka_id='.$b );
[PHP] pobierz, plaintext 

Ale zaznaczam... zarówno $a jak i $b w takim wypadku musisz filtrować, bo na chwilę obecną narażasz się na błędy i możliwy atak typu SQL Injection jeśli ktoś dojdzie gdzie można pchnąć ewentualnie zmanipulowane dane.

PS.: Oczywiście przyjąłem, że pilka_id to liczba i stąd darowałem sobie apostrofy. Jak wspomniałem.... FILTRUJ a życie stanie się prostsze (IMG:style_emoticons/default/winksmiley.jpg) Błąd polega w Twoim myśleniu na tym, że nazwa kolumny nie może być ujęta w pojedyncze apostrofy. Może być albo bez, albo w pojedynczych odwróconych czyli `takich`.