źle działający skrypt wyświetlający zdjęcia Opcje

[maxer]

Witam, zrobiłem na swojej stronie mini galerię. Po lewej stronie wyświetlają się miniaturki, a po prawej duże zdjęcie. Chciałbym aby duze zdjecie ktore jest aktualnie wyswietlane zmienialo obrazek miniaturki (np. na czerwony kwadrat).

napisalem taki kod, ale on nie dziala jak nalezy, znaczy sie miniaturka aktualnie wyswietlana nie zmienia swoich "barw":

[PHP] pobierz, plaintext 
<td valign="top"><div style="margin-top: 90px;">
					 <p style="text-align: center">
			   <?
					 $dir = opendir("gfx/kolekcja/czarne");
	  if($dir) {
	  while($img = readdir($dir)) {
 
	  @list($width, $height, $type, $attr) = getimagesize("gfx/kolekcja/czarne/$img");
 
if($img=='' or $img=='.' or $img=='..')
	  {
	  }
	  else if($img=='$fotka')
	  {
	  print"<a href=\"kolekcja.php?fotka=$img\"><img style=\"margin-top: 3px; margin-left: 2px; border-color: White;\" border=\"1\" src=\"gfx/kolekcja/kolorowe/$img\" width=\"26\" height=\"40\"></a>";
	  }
	  else
	  {
	  print"<a href=\"kolekcja.php?fotka=$img\"><img style=\"margin-top: 3px; margin-left: 2px; border-color: White;\" border=\"1\" src=\"gfx/kolekcja/czarne/$img\" width=\"26\" height=\"40\"></a>";
   
	  }
	  }
	  closedir($dir);
	  }
		 ?>
					 </p>
					 
				  </div></td>
				  <td valign="top">
				  <p style="text-align: right">
				  <?
				  print"<img border='0' src='gfx/kolekcja/duze/$fotka'>";
				  ?></p></td>
[PHP] pobierz, plaintext 

Pozdrawiam

edit: wywaliłem problem ;-)

Ten post edytował maxer 27.02.2006, 14:19:33

[Jojo]

Rozumiem że w zmiennej $fotka masz aktualnie wyświetlane zdjęcie. Jeśli tak to błąd masz prawdopodobnie w linii 13 powinno być:

[PHP] pobierz, plaintext 
<?php
 
else if($img==$fotka)
 
?>
[PHP] pobierz, plaintext 

[maxer]

Niestety wywala błąd :/

Parse error: parse error, unexpected T_ELSE in /home/espee/ftp/od_maxa/kolekcja.php

w lini w której zrobiłem

[PHP] pobierz, plaintext 
<?php
else if($img==$fotka)
?>
[PHP] pobierz, plaintext 

:/

[Jojo]

A pewien jesteś że dobrze to wkleiłeś? Bez znaczników <?php i ?> i nie skasowałeś przy wklejaniu żadnej klamry? (spójrz linijkę wyżej czy wszystko się zgadza).

[nospor]

@jojo nie wprowadzaj ludzi w błąd. w instrukcji if..else moze byc tylko jeden else.
tak wiec kod:

[PHP] pobierz, plaintext 
<?php
 
if ...
else if...
else ...
 
?>
[PHP] pobierz, plaintext 

jest bledny.
Poprawny to:

[PHP] pobierz, plaintext 
<?php
 
if ...
elseif...
else ...
 
?>
[PHP] pobierz, plaintext 

@maxer wywal ten problem z tytulu. tu kazdy ma problem

[Jojo]

nospor: mylisz się...
http://pl.php.net/manual/pl/control-structures.elseif.php
"else if" działa w php dokładnie tak samo jak w C. Czyli jak napiszesz:

[PHP] pobierz, plaintext 
<?php
 
if( warunek )
{
	//kod
}
else if( warunek )
{
	//kod
}
else
{
	//kod
}
 
?>
[PHP] pobierz, plaintext 

jest równoważne z tym jakbyś napisał:

[PHP] pobierz, plaintext 
<?php
 
if( warunek )
{
	//kod
}
else
{
	if( warunek )
	{
		//kod
	}
	else
	{
		//kod
	}
}
 
?>
[PHP] pobierz, plaintext 

[nospor]

@Jojo najmocniej Cię przepraszam. Zmylil mnie ten komunikat błędu, coś mi się jeszcze ubzduralo i tak mi się napisalo. A do linku co podales to ja wcześniej i tak zaglądalem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Co nie zmienia faktu że lepiej jest pisać elseif niz else if (a przynajmniej tak mi kiedys wpojona na torturach i teraz się tego trzymam). Nawet w linku co podales masz elseif (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

edit: "Moderator nie popelnia błędów - on sprawdza czujność użytkowników"

[maxer]

Taki kod mam teraz:

[PHP] pobierz, plaintext 
<?
							$dir = opendir("gfx/kolekcja/czarne");
		if($dir) {
		while($img = readdir($dir)) {
 
		@list($width, $height, $type, $attr) = getimagesize("gfx/kolekcja/czarne/$img");
 
if($img=='' or $img=='.' or $img=='..')
		{
		}
		else
		{
		print"<a href=\"kolekcja.php?fotka=$img\"><img style=\"margin-top: 3px; margin-left: 2px; border-color: White;\" border=\"1\" src=\"gfx/kolekcja/czarne/$img\" width=\"26\" height=\"40\"></a>";
		}
	else if($img==$fotka)
		{
		print"<a href=\"kolekcja.php?fotka=$img\"><img style=\"margin-top: 3px; margin-left: 2px; border-color: White;\" border=\"1\" src=\"gfx/kolekcja/kolorowe/$img\" width=\"26\" height=\"40\"></a>";
		}
 
		}
 
		closedir($dir);
		}
		   ?>
[PHP] pobierz, plaintext 

i to wywala ten błąd :/

[kszychu]

linie 9 i 10: co to ma być?

[Jojo]

nospor: Ja też piszę "łącznie" ale ten fragment kodu był po prostu skopiowany z listingu nad moim postem.

kszychu: Sądzę że to jest if w którym nic się nie wykonuje (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

maxer: Masz teraz złą kolejność najpierw powinien być if potem elseif a na koniec dopiero else.

[kszychu]

@Jojo, w takim razie wytłumacz mi, do którego ifa odnosi się linia 15.

[maxer]

@Jojo serdecznie dziękuję (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Działa jak należy (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
@kszychu linie 9 i 10 są po to aby skrypt nie wyświetlał katalogu jako zdjęcia ;-)

Pozdrawiam

[kszychu]

@maxer, w tym celu, przed while daj sobie:

[PHP] pobierz, plaintext 
<?php
readdir($dir);
readdir($dir);
 
?>
[PHP] pobierz, plaintext 

oszczędzi Ci to sprawdzania za każdym przebiegiem warunku, który występuje tylko dwa razy na początku.

[Kuziu]

To zamiast linii 9 i 10 możesz napsiać tak

[PHP] pobierz, plaintext 
<?php
if($img!='' && $img!='.' && $img!='..')
?>
[PHP] pobierz, plaintext 

I nie bedziesz miał takich dziwactw w kodzie choć owczywiście Twój kod zadziała tak samo.

Ten post edytował Kuziu 27.02.2006, 14:39:58