PHP debuggaamista?

[Macpoika]

Ajattelin kirjoittaa lyhyen skriptin, mutta kun se ei toimi, enkä oikein osaa debugata.
Kyse on siis kuvagalleria skriptistä, joka ottaa kuvat palvelimelta, tekee niille koodin, ja tallentaa koodinpätkät vektoriin.
Kuvat tulevat olemaan 20 kuvan sarjoissa ja vektorista sitten tulostetaan sivun mukaan.
Aikaisempi skriptini toimi hyvin, mutta se laittoi kaikki kuvat yhdelle sivulle. Javascript muuten on sitä varten, että linkkien kohteet muutetaan funktioksi, joka vaihtaa suurta kuvaa (joka tulee sivun yläreunaan) ilman uudelleenlatausta.

Ongelma siis on tod. näk. silmukoissa.

Koodi: Valitse kaikki

<?php
	$dir = opendir('images/thumbnails/');
	
	$file_name = readdir($dir);
	
	$image_check = false;
	
	$page_index[0] = "";
	
	for($i = 0; $file_name; $i++) {
		if(substr($file_name, -4) == '.jpg' || substr($file_name, -5) == '.jpeg' || substr($file_name, -4) == '.gif' || substr($file_name, -4) == '.png') {
			$image_check = true;
			for($int = 0; $int < 20; $int++) {
				$id = 'id' . $int;
				$file_path = 'images/thumbnails/' . $file_name;
				$img_size = getimagesize($file_path);
				$big_img_path = 'images/photos/big_' . $file_name;
				$big_img_size = getimagesize($big_img_path);
				$width = $big_img_size[0];
				$height = $big_img_size[1];
				$page_index[$i] = $page_index[$i] . "<a class=\"thumbnail\" id=\"{$id}\" href=\"?target=galleria&page=1&img={$file_name}\" title=\"{$file_name}\" name=\"{$id}\"><img src=\"{$file_path}\" " . $img_size[3] . " alt=\"{$file_name}\" /></a>" . "<script type=\"text/javascript\">switchToJs('{$id}', '{$file_name}', {$width}, {$height});</script>";
			}
			$img_count++;
		}
	}
	
	if($image_check == false) {
		$page_index[1] = "<p>Ei kuvia.</p>";
	}
	
?>

Alkuperäinen skripti:

Koodi: Valitse kaikki

<?php
	$dir = opendir('images/thumbnails/');
	
	$file_name = readdir($dir);
	
	$image_check = false;
	
	for($int = 0;$file_name; $int++) {
		if(substr($file_name, -4) == '.jpg' || substr($file_name, -5) == '.jpeg' || substr($file_name, -4) == '.gif' || substr($file_name, -4) == '.png') {
			$image_check = true;
			$id = 'id' . $int;
			$file_path = 'images/thumbnails/' . $file_name;
			$img_size = getimagesize($file_path);
			$big_img_path = 'images/photos/big_' . $file_name;
			$big_img_size = getimagesize($big_img_path);
			$width = $big_img_size[0];
			$height = $big_img_size[1];
			echo "<a class=\"thumbnail\" id=\"{$id}\" href=\"?target=galleria&page=1&img={$file_name}\" title=\"{$file_name}\" name=\"{$id}\"><img src=\"{$file_path}\" " . $img_size[3] . " alt=\"{$file_name}\" /></a>";
			echo "<script type=\"text/javascript\">switchToJs('{$id}', '{$file_name}', {$width}, {$height});</script>";
		}
		$file_name = readdir($dir);
	}
	
	if($image_check == false) {
		echo "<p>Ei kuvia.</p>";
	}
	
?>

[Macpoika]

Oikeastaan osaan debugata, mutta aikaisemmin olin varmaan sen verran väsynyt, ettei ajatus kulje tai ei jaksa pohtia. Löysin montakin ongelmaa.

Kiitos ja anteeksi.

[Matti Niemelä]

Ihan vinkkinä, käytä html-stringeissä '-lainausmerkkejä. Eli $str = '<a href="urli">nimi</a>'; Niin ei tarvi hajoilla noiden escapejen kanssa.

Lisäksi kannattaisi tutustua esimerkiksi pathinfo() -funktioon tuon tiedostopäätteen tutkimisen osalta. (pathinfo($polku, PATHINFO_EXTENSION);)

Itseasiassa, itse tekisin näin

Koodi: Valitse kaikki

if(in_array(strtolower(pathinfo($file_name, PATHINFO_EXTENSION)), array('png', 'jpg', 'jpeg', 'gif')) === true) { ...

Noh. Onko tuo nyt sitten yhtään parempi

Array alustetaan muuten näin: $page_index = array(); (Ihan vaan noin yhtenäisyyden nimissä.)

Ja if($var === true) tarkoittaa sitä että arvojen vertauksen lisäksi testataan että tietotyyppi on sama. Itse suosin tuota aina bool arvojen kanssa.

[Macpoika]

Ihan vinkkinä, käytä html-stringeissä '-lainausmerkkejä. Eli $str = '<a href="urli">nimi</a>'; Niin ei tarvi hajoilla noiden escapejen kanssa.

Lisäksi kannattaisi tutustua esimerkiksi pathinfo() -funktioon tuon tiedostopäätteen tutkimisen osalta. (pathinfo($polku, PATHINFO_EXTENSION);)

Itseasiassa, itse tekisin näin

Koodi: Valitse kaikki

if(in_array(strtolower(pathinfo($file_name, PATHINFO_EXTENSION)), array('png', 'jpg', 'jpeg', 'gif')) === true) { ...

Noh. Onko tuo nyt sitten yhtään parempi

Array alustetaan muuten näin: $page_index = array(); (Ihan vaan noin yhtenäisyyden nimissä.)

Ja if($var === true) tarkoittaa sitä että arvojen vertauksen lisäksi testataan että tietotyyppi on sama. Itse suosin tuota aina bool arvojen kanssa.

Tiedän tuon '-jutun, mutta en jaksanut alkaa muuttelemaan, kun sen olin siinä jossain välissä keksinyt. Kiitti vinkistä. Kiitos myös, että selvitit mitä merkkaa kolme onyhtäkuinmerkkiä eikös sen variaatiot sitten ole: !== ja !!=. Onko olemassa =!= ?
Voisin käyttää tuota pathinfoa. Kiitos siitäkin. Vektorithan voi myös alustaa niinkuin minulla on. Niiden alustamiseen on useampi eri keino. PHPn hyvä puoli on, että se on aika vapaata tietotyyppien ym. suhteen. Tuon tavan ajattelin olevan paras/yksinkertaisin tähän.

Kiitos.

[Matti Niemelä]

eikös sen variaatiot sitten ole: !== ja !!=. Onko olemassa =!= ?

Huh? Ihan vaan === ja !==.

Niiden alustamiseen on useampi eri keino. PHPn hyvä puoli on, että se on aika vapaata tietotyyppien ym. suhteen.

Toki. PHP:n hyvä ja huono puoli (nyt kauan ja paljon tehneenä) on juuri tuo vapaus tehdä asiat juuri kuten ahteria kyseisellä hetkellä kutittaa. Kuitenkin ei kannata miettiä mistä tavasta itse tykkää eniten, vaan mitä tapaa käytetään mailmalla eniten. Näin vältyt myöhemmin aiheuttamasta kamalia päänsärkyjä toisille ihmisille jotka joutuvat koodejasi ronkslaamaan.

Muista aina KISS.

Edit: Muista aina sekin, että PHP Manuaali on ehdottomasti paras ystäväsi. Mistään et saa niin nopeasti ja hyvää tietoa kuin manuskasta.