Problém zo zobrazením obrázkov – PHP – Fórum – Programujte.com

Zdravím mám problém zo zobrazením obrázkov, na X-weboch mi to funguje a tu je zas nejaký problém :)

obrázky pridávam na FTP, skript na vloženie funguje v poriadku, problém je zo zobrazením...

http://salgovce.esy.es/index.php?page=projekty

v ceste medzi posledným podpriecinkom a suborom mi nechce zobrazit premennu "$cislo" z databazi, ale v ADMINISTRACII (po prihlaseni) link na ZMAZANIE/UPRAVENIE/PRIDANIE OBRAZKA  tu premennu zobrazuje...

 FUNKCIA NA NACITANIE

function CitajDIR($cesta) {
    if (!is_dir ($cesta)) {
    echo "Žiadne nahraté obrázky";
    }
    else
      if ($dir = opendir ($cesta)) { 
        while ($subor = readdir($dir)) {
          if ($subor!='.' and $subor!='..')           
           echo "<div class='obra'><a href='projekty/foto/$cislo/$subor' title='$subor' rel='lightbox'>
                  <div class='obr1'><img src='projekty/thumbs/$cislo/$subor' alt='$subor'/></div>
                 </a></div>";    
        } 
      closedir($dir);
      }      
}

ZOBRAZENIE OBRAZKOV

<h1>Projekty</h1>
<?php
$skratka = $_GET['page'];
$cislo = $tab['id'];
 	
function right_int($arg) {
  $arg = (int) $arg;
  if (is_numeric($arg))
    { return $arg; }
  return 1;
}  
?>

<div id="okraj">
<?php
  define ("ON_PAGE", 10);
  if(!isset($_GET["id"])) {
    $page = 1;
  }
  else {
    $page = right_int($_GET["id"]);
  }
  $by = (ON_PAGE * ($page - 1));
  
$psql = "SELECT * FROM ".$skratka." ORDER BY id DESC LIMIT ".ON_PAGE." OFFSET ".$by;
$presult = mysqli_query($link, $psql);
if (mysqli_num_rows($presult) > 0){
  while ($tab = mysqli_fetch_array($presult)) {  
     $obsah = $tab["text"];
     $nazov = $tab['nadpis'];
     $cislo = $tab['id'];
     $obsah1 = wordwrap($obsah, 25, "\n", true);
     echo "<h1>".$tab['nadpis']."</h1>
           <p>".$obsah1."</p>";
     echo '<div class="gal">';
     CitajDIR("projekty/thumbs/$cislo");
     echo '</div>';
     if ($_SESSION["log"] == "1") {
      echo 
  	   "<a href='index.php?page=zclanok&amp;skratka=".$skratka."&amp;cis=".$cislo."'>Zmazať</a>
        <a href='index.php?page=upravit&amp;skratka=".$skratka."&amp;cis=".$cislo."'>Upraviť</a>
        <a href='index.php?page=uobrazok&amp;skratka=".$skratka."&amp;cis=".$cislo."'>Vložiť obrázok</a>";
  	  }      
   }}      
?>
 
<?php
$sql = "SELECT * FROM ".$skratka." ORDER BY id DESC";
$result = mysqli_query($link, $sql);
$pocetzaznamov = mysqli_num_rows($result);
	if($page != 1) { 
    echo "<a href='index.php?page=".$skratka."&amp;id=1'>[1]</a>";
	}
	if($page > 1) { 
		for($i = 2; $i > 0; $i--) {
			if(($page - $i) > 1) { 
        echo "|<a href='index.php?page=".$skratka."&amp;id=".($page-$i)."'>[".($page-$i)."]</a>";
			}
		}
	echo "|<a href='index.php?page=".$skratka."&amp;id=".($page-1)."'>[&laquo;]</a>|";
	}
	$pocetstran = (ceil($pocetzaznamov / ON_PAGE));	
	if($page < $pocetstran) { 
    echo "|<a href='index.php?page=".$skratka."&amp;id=".($page+1)."'>[&raquo;]</a>";
		for($i = 1; $i < 2; $i++) { 
   			if(($page + $i) < $pocetstran) { 
          echo "|<a href='index.php?page=".$skratka."&amp;id=".($page+$i)."'>[".($page+$i)."]</a>";
      	}
		}
	}
	if($page < $pocetstran) { 
    echo "|<a href='index.php?page=".$skratka."&amp;id=".$pocetstran."'>[".$pocetstran."]</a>";
	}
?>
</div>

Ďakujem veľmi pekne...

#1 Tomáš
Ve funkci CitajDIR nemáš deklarovánu proměnnou $cislo, nemůžeš ji tedy používat.

#2 Kit
Neviem či ťa správne chápem do funkcie CitajDIR stačí keď doplním $cislo = 0; ?

Skúšal som to doplniť vo funkcií za ELSE bez úspechu...

Skúšal som vo funkcií vytiahnuť znova z DATABÁZY a taktiež bez úspechu... 

Môžeš byť konkrétnejší?

#3 Tomáš

Máš tam na jednom řádku ve funkci CitajDIR

<img src='projekty/thumbs/$cislo/$subor'

a hodnotu $cislo nemáš nikde definovánu. Místo ní je tedy dosazen prázdný řetězec a vypsáno "PHP Notice:  Undefined variable: $cislo". To by tě mělo dostatečtě trknout, že tam něco chybí.

#4 Kit
Vyriešené! :) Ďakujem veľmi pekne, dosť mi pomohla http://www.zive.sk/clanok/7842/php-od-a-po-z-cast-iv-praca-s-premennymi

riešenie namiesto samotnej premennej $cislo stačilo definovať globalnu premennu o LVL vyššie a teda $GLOBALS['cislo']

#5 Tomáš
To je to nejhorší, co jsi mohl udělat. Fuj!

#6 Kit
Skús to okrem komentovania aj vysvetliť prípadne hodiť sem riešenie, to že si tu vyjadríš názor nikomu nepomôže a v druhom rade ani nikoho nezaujíma... Každý sem ide nájsť riešenie, mne to ide, síce ma možno zaujíma prečo je to zle ale k podstate veci si sa s tým "Fuj!" asi nedostal, takže si to nabudúce nechaj doma pri obede...

#7 Tomáš
Stačilo přece funkci CitajDIR přidat jeden parametr.

Globální proměnné jsou jedno velké zlo a je nutné se jich zbavit.

#7 Tomáš
Jde to samozřejmě i s jedním parametrem: 

function CitajDIR($cislo) {
    $thumbs = "projekty/thumbs/$cislo";
    $foto = "projekty/foto/$cislo";
    if (!is_dir ($thumbs)) {
        echo "Žiadne nahraté obrázky";
    } else if ($dir = opendir ($thumbs)) { 
        while ($subor = readdir($dir)) {
            if ($subor!='.' and $subor!='..')           
                echo "<div class='obra'><a href='$foto/$subor' title='$subor' rel='lightbox'>
                      <div class='obr1'><img src='$thumbs/$subor' alt='$subor'/></div>
                      </a></div>";    
        } 
        closedir($dir);
    }      
}

Jako parametr při volání však nepoužiješ $cesta, ale $cislo.

Mam jiny nazor. Globalnim promennym je dobre se vyhybat, ale neodsuzuji je.

Pouzivam je, kdyz chci do jine class zavest sql nebo lng. Tak si to definuji na zacatku a do construct si nacpu prirazeni globalnich promennych nekam. Stale lepsi nez do kazde funkce pak psat dalsi parametr navic.

funkce($sql, $lng, $a, $b, $c)

function __construct($sql) {$this->$sql = $sql;}
funkce($a, $b, $c) {$this->$sql ...}

Treba mam funkce, kde bych takto musel zbytecne opisovat 10 parametru, ktere budou stale stejne. To mi prijde nelogicke. To radeji berlicku s global.
funkce($a, $b, $c) {global $sql, $lng;}
 

#10 peter
Pokud máš stále stejných 10 parametrů, udělej si třídu a ty parametry strč do konstruktoru instance.

Dik za nepridany priklad :) Treba, kdyz to delam takhle, tak to jinak neumim, ne? Do kontruktoru je prave davam, jak jsem naznacil.

function __construct($sql) {$this->sql = $sql;}
function funkce($a, $b, $c) {$this->sql ...}

Je to trochu krkolone to furt volat pres this, ale snazim se tak vyhybat global, za ktere tu lidi stale karas :) Ikdyz obcas pouzivam zamerne i kod:

function __construct() {global $sql; $this->sql = $sql;}
function funkce($a, $b, $c) {$this->sql ...}

Jo, a mluvis na mne cinsky, mam kurz informatiky je SS a uz vubec ne na OOP, ikdyz s tim pracuji a chvalim stvoritele, protoze mam pak lepsi kontrolu nad promennymi :) Zkus cestinu a vynech ty odborne vyrazy :)

resp, pouzivam velka pismena, tim odlisuji global od toho uvnitr.
function __construct() {global $SQL; $this->sql = $SQL;}