[PHP] Undefined index, again Opcje

[bigbadcompany]

Witam,

po dłuższym grzebaniu znowu się poddałem i chciałbym zasięgnąć Waszej opinii (może jakieś rady).
W panelu admina zaczęło mi pluć tymi notice'ami na potęgę z jednymi sobie poradziłem z drugimi jest
trochę gorzej.

Błąd o treści takiej jak w temacie. Odnosi się do linijki 1050 której zlepek zamieszczam poniżej.
Mianowicie if($_GET['akcja']=='nowy' && $_POST['tytul'])

[PHP] pobierz, plaintext 
	case 'artykuly':
	// konfiguracja tematow
	if($_GET['akcja']=='nowy' && $_POST['tytul'])
	{
		$_POST['tytul']=trim($_POST['tytul']);
		$_POST['skrot']=trim($_POST['skrot']);
		$_POST['opis']=trim($_POST['opis']);
		$_POST['dodal']=trim($_POST['dodal']);
		$_POST['data']=trim($_POST['data']);
		$_POST['tag1']=trim($_POST['tag1']);
		$_POST['tag2']=trim($_POST['tag2']);
		$_POST['tag3']=trim($_POST['tag3']);
		$_POST['kategoria']=trim($_POST['kategoria']);
 
		if(strlen(trim($_POST['tytul']))>5)
		{
		mysql_query("INSERT INTO `artykuly` (`id`, `tytul`, `skrot`, `opis`, `dodal`, `data`, `tag1`, `tag2`, `tag3`, `id_kat`, `aktywny`, `seo_url`) VALUES (NULL, '".$_POST['tytul']."', '".$_POST['skrot']."', '".$_POST['opis']."', '".$_POST['dodal']."', NOW(), '".$_POST['tag1']."', '".$_POST['tag2']."', '".$_POST['tag3']."', '".$_POST['kategoria']."', '1', '".stringToLink($_POST['tytul'])."')");
		header("location: index.php?s=artykuly&ok=1");
		}
		else header("location: index.php?s=artykuly&err=2");
	}
[PHP] pobierz, plaintext 

Celem zabiegów jest pozbycie się noticów w ogóle i nie interesują mnie półśrodki typu "poprawić php.ini"
Jakieś pomysły ?

[peter13135]

Ogólnie mówiąc, błąd undefined index/variable jest wtedy, gdy chcesz odczytać jakiś element tablicy/zmienną która nie istnieje.

Dlatego takie sprawdzanie :

[PHP] pobierz, plaintext 
if($_GET['foo']=='bar')
{
 
}
[PHP] pobierz, plaintext 

Albo przypisywanie :

[PHP] pobierz, plaintext 
$foo = $_GET['foo'];
[PHP] pobierz, plaintext 

W momencie kiedy $_GET['foo'] nie istnieje, powoduje właśnie ten komunikat.

Dlatego, zanim będziesz porównywał jakieś zmienne, czy cokolwiek z nimi robił, upewnij się, czy one istnieją. A możesz to zrobić za pomocą np. funkcji isset().
Isset zwraca true, jeśli zmienna jest zdefiniowana, false jeśli niezdefiniowana.


Przykład :

Zamiast pisać

[PHP] pobierz, plaintext 
if($_GET['akcja']=='nowy' )
[PHP] pobierz, plaintext 

napisz:

[PHP] pobierz, plaintext 
if(isset($_GET['akcja']) && $_GET['akcja']=='nowy' )
[PHP] pobierz, plaintext 

Ponieważ tu jest koniunkcja w warunku, to jeśli isset($_GET['akcja']) zwróci false, to następna część warunku nie będzie w ogóle sprawdzana.

Następna sprawa, nie używaj takiego czegoś jak

[PHP] pobierz, plaintext 
if($_GET['cośtam'])
{}
[PHP] pobierz, plaintext 

bo zdaje się, że nie do końca wiesz co to oznacza.

jeśli $_GET['cośtam'] będzie miało wartość "0lalala". To cały warunek nie będzie spełniony.
Tutaj uzyj funkcji empty()

Ten post edytował peter13135 4.09.2012, 13:08:48

[PanGuzol]

[PHP] pobierz, plaintext 
if($_GET['cośtam'])
{}
[PHP] pobierz, plaintext 

bo zdaje się, że nie do końca wiesz co to oznacza.

jeśli $_GET['cośtam'] będzie miało wartość "0lalala". To cały warunek nie będzie spełniony.
Tutaj uzyj funkcji empty()

Warunek będzie spełniony. PHP zamieni "0lalala" w boolean co przy stringach innych niż pusty lub "0" daje prawdę, ale zgadzam się iż lepiej skorzystać z empty.

[bigbadcompany]

Faktycznie pomogło. Jednak nie rozumiem jeszcze jednego notice'a. Problem jest taki, że w podglądzie na podstronie z dodawaniem nowych artykułów lista tematów się wyświetla. Jak wejdę w edycję to już pluje błędem. Szczerze nie wiem gdzie szukać. Gdzieś trzeba ten warunek udowodnić. Jak wstawiam go po if(isset($_GET['id']) && $_GET['akcja']=='edytuj') to nie mogę w ogóle edytować danego artykułu. Nie pobiera mi nic z bazy. Treść komunikatu jest taka o jakiej wcześniej wspomniałeś czyli Notice: Undefined variable: lista_tematow in C:\xampp\htdocs\admin\index.php on line 1265. Linia ta wygląda tak '; (zakończenie linii). W przeklejonym kodzie linia 141

[PHP] pobierz, plaintext 
	// pokaz formularz edycji danego artykulu
    if(isset($_GET['id']) && $_GET['akcja']=='edytuj')
	{
		$artykul_id=$_GET['id'];
		$sq_kat='select * from `kategorie_a`';
		$sql_kat=mysql_query($sq_kat);
		$lista_kategorii='Edycja artykułu';
		$lista_kategorii.='
	<form name="artykuly" method="post" action="index.php?s=artykuly&akcja=zapisz&id='.$artykul_id.'">
	<table>
	<tr>
		<th colspan="3">Nowy artykuł</th>
	</tr>
	<tr>
		<td>Tytul: </td><td><a href="" class="help" rel="dymek_news1">[?]</a></td><td><input type="text" name="tytul" value="'.$artykul_tytul[$artykul_id].'" class="bdlugi"/></td>
	</tr>
	<tr>
		<td>Krótka treść: </td><td><a href="" class="help" rel="dymek_news2">[?]</a></td><td><textarea style="height:100px" name="skrot" class="bdlugi">'.$artykul_skrot[$artykul_id].'</textarea></td>
	</tr>
 
	<tr>
		<td>Długa treść: </td><td><a href="" class="help" rel="dymek_news3">[?]</a></td><td><textarea style="height:400px" name="opis" class="bdlugi">'.$artykul_opis[$artykul_id].'</textarea></td>
	</tr>
 
	<tr>
		<td>Tag 1: </td><td><a href="" class="help" rel="dymek_news3">[?]</a></td><td><input type="text" name="tag1" value="'.$artykul_tag1[$artykul_id].'" class="bdlugi"/></td>
	</tr>	
 
	<tr>
		<td>Tag 2: </td><td><a href="" class="help" rel="dymek_news3">[?]</a></td><td><input type="text" name="tag2" value="'.$artykul_tag2[$artykul_id].'" class="bdlugi"/></td>
	</tr>	
 
	<tr>
		<td>Tag 3: </td><td><a href="" class="help" rel="dymek_news3">[?]</a></td><td><input type="text" name="tag3" value="'.$artykul_tag3[$artykul_id].'" class="bdlugi"/></td>
	</tr>	
 
	<tr>
		<td>Autor: </td><td><a href="" class="help" rel="dymek_news3">[?]</a></td><td><input type="text" name="dodal" value="'.$artykul_dodal[$artykul_id].'" class="bdlugi"/></td>
	</tr>	
 
	<tr>
		<td>Link: </td><td><a href="" class="help" rel="dymek_news3">[?]</a></td><td><input type="text" name="url" value="'.$artykul_url[$artykul_id].'" class="bdlugi"/></td>
	</tr>	
 
	<tr>
		<td>Kategoria: </td><td><a href="" class="help" rel="dymek_news4">[?]</a></td><td>
 
	<select name="kategoria">
	';
 
	while($kat=mysql_fetch_assoc($sql_kat))
	{
		//$selected='';
		$id_kat=$kat['catt_id'];
		$tab_kat[$id_kat]=$kat['catt_name'];
		$kat['catt_id']==$artykul_kategoria[$artykul_id] ? $selected='selected' : $selected='';
		$lista_kategorii.='<option value='.$kat['catt_id'].' '.$selected.'>'.$kat['catt_name'].'</option>';
	}
	$lista_kategorii.='</select></td>
	</tr>
	<tr>
		<td colspan="2"></td><td><input type="submit" value="zapisz artykuł" /><a href="" class="help" rel="dymek_news6">[?]</a></td>
	</tr>
	</table>
	</form>';
 
	}
 
	else
	{
	//listuj kategorie tematow
	$sq_kat='select * from `kategorie_a`';
	$sql_kat=mysql_query($sq_kat);
	$lista_kategorii='
	<form name="artykuly" method="post" action="index.php?s=artykuly&akcja=nowy">
	<table>
	<tr>
		<th colspan="3">Nowy artykuł</th>
	</tr>
	<tr>
		<td>Tytuł: </td><td><a href="" class="help" rel="dymek_news1">[?]</a></td><td><input type="text" name="tytul" class="bdlugi"/></td>
	</tr>
 
	<tr>
		<td>Krótka treść: </td><td><a href="" class="help" rel="dymek_news2">[?]</a></td><td><textarea style="height:100px" name="skrot" class="bdlugi"></textarea></td>
	</tr>
 
	<tr>
		<td>Długa treść: </td><td><a href="" class="help" rel="dymek_news3">[?]</a></td><td><textarea style="height:200px" name="opis" class="bdlugi"></textarea></td>
	</tr>
 
	<tr>
		<td>Autor: </td><td><a href="" class="help" rel="dymek_news3">[?]</a></td><td><input type="text" name="dodal" class="bdlugi"/></td>
	</tr>	
 
	<tr>
		<td>Tag 1: </td><td><a href="" class="help" rel="dymek_news3">[?]</a></td><td><input type="text" name="tag1" class="bdlugi"/></td>
	</tr>	
 
	<tr>
		<td>Tag 2: </td><td><a href="" class="help" rel="dymek_news3">[?]</a></td><td><input type="text" name="tag2" class="bdlugi"/></td>
	</tr>	
 
	<tr>
		<td>Tag 3: </td><td><a href="" class="help" rel="dymek_news3">[?]</a></td><td><input type="text" name="tag3" class="bdlugi"/></td>
	</tr>	
 
	<tr>
		<td>Podstrona: </td><td><a href="" class="help" rel="dymek_news4">[?]</a></td><td>
 
	<select name="kategoria">
	';
 
	while($kat=mysql_fetch_assoc($sql_kat))
	{
		$id_kat=$kat['catt_id'];
		$tab_kat[$id_kat]=$kat['catt_name'];
		$lista_kategorii.='<option value='.$kat['catt_id'].'>'.$kat['catt_name'].'</option>';
	}
 
	$lista_kategorii.='</select></td>
	</tr>
	<tr>
		<td colspan="2"></td><td><input type="submit" value="dodaj artykuł" /><a href="" class="help" rel="dymek_news5">[?]</a></td>
	</tr>
	</table>
	</form>';
 
	$lista_tematow.='
	<div class="odstep"></div>
	<table border=0>
		<tr><th colspan="5" style="width:100%;padding-bottom:10px">Opublikowane artykuły (edytuj, aby przeczytać cały tekst danego artykułu)</th></tr>
		<tr>
			<th class="sredni">Tytuł</th>
			<th class="krotki">Krótka treść</th>
			<th class="sredni">Długa treść</th>
			<th class="kto">Autor</th>
			<th class="krotki">Podstrona</th>
			<th class="mini">Opcje</th>
		</tr>
	';
	$w=0;
	foreach($artykul_tytul as $nr => $art)
	{
		$bg='171717';
		$w++;
		if($w==2) {$w=0;}
		elseif($w==1){$bg='#222';}
 
		$kat_nazwa=$artykul_kategoria[$nr];
		$lista_tematow.='
		<tr>
			<td style="padding:0 5px;border-bottom:solid 0px silver;background:'.$bg.'" class="mini_font">'.substr(htmlspecialchars($artykul_tytul[$nr]),0,255).'</td>
			<td style="padding:0 5px;border-bottom:solid 0px silver;background:'.$bg.'" class="mini_font">'.substr(htmlspecialchars($artykul_skrot[$nr]),0,50).'</td>
			<td style="padding:0 5px;border-bottom:solid 0px silver;background:'.$bg.'" class="mini_font">'.substr(htmlspecialchars($artykul_opis[$nr]),0,50).'</td>
			<td style="padding:0 5px;border-bottom:solid 0px silver;background:'.$bg.'" class="mini_font">'.substr(htmlspecialchars($artykul_dodal[$nr]),0,50).'</td>
			<td style="padding:0 5px;border-bottom:solid 0px silver;background:'.$bg.'" class="mini_font">'.$tab_kat[$artykul_kategoria[$nr]].'</td>
			<td style="border-bottom:solid 0px silver;text-align:center;background:'.$bg.'"><div style="padding-bottom:5px">[<a href="index.php?s=artykuly&akcja=edytuj&id='.$artykul_id[$nr].'">edytuj</a>]</div>
			<div style="padding-bottom:5px">[<a href="index.php?s=artykuly&akcja=usun&id='.$artykul_id[$nr].'" style="color:#C00">usuń</a>]</a></div>
			</td>
		</tr>';
	}
	$lista_tematow.='</table>';
	}
 
	if(DEBUG)
	{
		debuguj($artykul_kategoria);
		debuguj($tab_kat);
		debuguj($_POST);
	}
	echo $header;
	include("inc/top.inc.php");
	include("inc/tip_info.inc.php");
	if(!empty($blad_txt)) echo $blad_txt;
	echo '<tr>
						<td bgcolor="#3D5883">
						<font face="Tahoma" color="#FFFFFF" size="4">Artykuły</font></td>
					</tr>';
	if(isset($_GET['err']) && $_GET['err']>0) tipError($tip_temat[$_GET['err']]);
	if(isset($_GET['ok']) && $_GET['ok']>0) tipInfo($tip_temat[$_GET['ok']]);
	echo $lista_kategorii;
	echo $lista_tematow;
	break;	
[PHP] pobierz, plaintext 

[PanGuzol]

Przy pierwszym wykonaniu pętli zmienna $lista_tematow nie istnieje a operator .= wpierw musi pobrać jej wartość aby móc dopisać do niej nową wartość. Innymi słowy wpierw próbuje pobrać zmienną która nie istnieje i masz notice. W tym przypadku musisz zadeklarować zmienną przed pętlą jako pusty string czyli po prostu $lista_tematow = ""; zaraz przed while.

[bigbadcompany]

Okej tamtego błędu nie ma już. Jednak listy w edycji nadal nie wyświetla. Notice: Undefined variable: lista_tematow in C:\xampp\htdocs\admin\index.php on line 1307 próbuje zgonić na wywołanie echo $lista_tematow;

Zastanawiam się też czemu wcześniej tych błędów nie wyświetlało. Tylko po aktualizacji php. Config się raczej nie zmienił.

Ten post edytował bigbadcompany 4.09.2012, 15:05:02

[PanGuzol]

Źle policzyłem nawiasy wcześniej. zmienną $lista_tematow musisz zdefiniować na początku przed if nie przed while.
Listę tą wypełniasz w else więc jeśli warunek jest spełniony to lista tematów będzie pusta.

Inna konfiguracja dla error_reporting.
Wiele osób ignoruje błędy notice, takie aplikacje wciąż będą działać ale moim skromnym zdaniem lepiej pozbyć się jakichkolwiek błędów.

Ten post edytował PanGuzol 4.09.2012, 15:10:42

[bigbadcompany]

Wszystkie notice'y znikają ale wraz z nimi lista tematów w edycji

[PanGuzol]

Czy korzystasz ze zmiennej $lista_tematow w innym miejscu w kodzie poza tym fragmentem który tutaj udostępniłeś?

[bigbadcompany]

Tak, ale to już inny case. Poza tym po wyrzuceniu tamtego kodu wspomniane wcześniej zabiegi nie pomagają. Jeśli chcesz mogę na mejla wysłać wszystko co mam w pliku index admina. Jak masz czas rzuciłbyś swoim wnikliwym okiem na to. Nie jest to jakiś majstersztyk ale nadal nad tym pracuję.

[PanGuzol]

Dla długich fragmentów lepiej skorzystać z http://pastebin.com/ lub podobnych serwisów.

Rozumiem, że tematy wyświetlały się zanim pozbyłeś się noticów?

[bigbadcompany]

Napiszę szczerze, że wcześniej już nie mogłem z tym do ładu dojść tylko po aktualizacji zaczęły błędy wyskakiwać co mnie trochę bardziej zmotywowało do działania.

Serwisu nie znałem przyda się z całą pewnością.

[PanGuzol]

Mi się wydaje, że trochę namieszałeś z warunkami.

Uporządkuj sobie kod poprzez odpowiednie wcięcia w kodzie tj każdy nowy blok kodu ma jeden więcej tabulator na początku każdej linii. Teraz masz pomieszane czasami robisz wcięcia a czasami nie oraz w przypadku dłuższych bloków pisz komentarze przy zamykających i opcjonalnie również otwierających klamrach abyś mógł łatwo określić gdzie się zaczyna a gdzie kończy(if, pętla, funkcja) bez niepotrzebnego liczenia.

[bigbadcompany]

Wiesz co w sumie masz rację. Najpierw to uporządkuję a dopiero później zacznę rzucać luźne uwagi bo tak szukamy po omacku a nie ma sensu żebyś tracił cenny czas. Mimo to dziękuję bo bardzo dużo pomogłeś. Pozdrawiam i życzę miłego wieczoru

[peter13135]

Warunek będzie spełniony. PHP zamieni "0lalala" w boolean co przy stringach innych niż pusty lub "0" daje prawdę, ale zgadzam się iż lepiej skorzystać z empty.

Faktycznie, myślałem, że dotyczy to wszystkich stringów zaczynających się na 0, ale się myliłem. Dzięki za poprawienie