problem z foreach Opcje

[mb83]

witam,
poniżej przedstawiam kod który ma wyświetlić pojedyncze zdjęcie z galerii.

[PHP] pobierz, plaintext 
$zapytanie = "SELECT id, tytul, galeriaduze, galeriamale FROM `newsy` WHERE id='$id' ";
 
 $wynik = mysql_query($zapytanie);
 while ( $dane = mysql_fetch_row($wynik) ) {
 $tytul = $dane[1];
 $gd = $dane[2];
 $gm = $dane[3];
class PaginateDirectory{ 
    var $sDirectoryPath; 
    var $iCurrentPage; 
    var $iNumberResources; 
    var $iNumberPerPage; 
    var $aImages; 
 
    function PaginateDirectory($path, $page){ 
        $this->resetVars(); 
        $this->_setPath($path); 
        $this->_setCurrentPage($page); 
        $this->aImages = $this->getResources($path);         
    } # end constructor 
 
    function resetVars(){ 
        $this->sDirectoryPath = NULL; 
        $this->iCurrentPage = 0; 
        $this->iNumberResources = 0; 
        $this->iNumberPerPage = NUMBER_PER_PAGE; 
        $this->aImages = array(); 
        return; 
    } 
 
    function _setPath($path){ 
        $this->sDirectoryPath = $path; 
        return; 
    } 
 
    function _setCurrentPage($page){ 
        if( (int)$page < 1 ){ 
            $page = 1; 
        } 
        $this->iCurrentPage = (int)$page; 
        return; 
    } 
 
    function getResources($path){ 
        $container = array(); 
        $total = 0; 
        if (is_dir($path)){ 
            if ($handle = opendir($path)){ 
 
                $begin = ($this->iCurrentPage > 1)  
                            ?     (($this->iCurrentPage - 1) * NUMBER_PER_PAGE) 
                            :    0; 
                $end = $begin + NUMBER_PER_PAGE - 1; 
 
                while (false !== ($file = readdir($handle))){ 
                    if ($file != '.' && $file != '..' && !is_dir($path . $file)){ 
                        if( ($total > $begin) && ($total < $end) ){ 
                            $size = @getimagesize($path . $file); 
                            $container[$total] = array('file' => $file, 
                                                       'dimensions' => $size[3]  
                                                      ); 
                        } 
                        $total++; 
                    }                     
                } 
            } 
        } 
        $this->iNumberResources = $total; 
        return $container; 
    }  
 
    function getImages(){ 
        return $this->aImages; 
    } 
 
    function displayCount(){     
        $fromNumber = ($this->iCurrentPage > 1)  
                        ?     (($this->iCurrentPage - 1) * NUMBER_PER_PAGE) + 1 
                        :    $this->iCurrentPage; 
 
        $toNumber = $fromNumber + NUMBER_PER_PAGE - 1; 
        if ($toNumber > $this->iNumberResources){ 
            $toNumber = $this->iNumberResources; 
        }         
 
        return 'Zdjęcie <b>' . $fromNumber  . '</b> / <b> <b>' . $this->iNumberResources . '</b>'; 
    } 
 
    function displayLinks(){ 
        $numberPages = ceil($this->iNumberResources / NUMBER_PER_PAGE); 
        $output = array(); 
        $previousPage = $this->iCurrentPage - 1; 
        $nextPage = $this->iCurrentPage + 1; 
 
        if( $previousPage !== 0 ){ 
            $output[] = '<a href="/photo.php?id='.$_GET['id'].'&amp;page='.$previousPage.'">[<< Poprzednie]</a>'; 
        } 
 
 
        if ( $nextPage <= $numberPages ){ 
            $output[] = '<a href="/photo.php?id='.$_GET['id'].'&amp;page='.$nextPage.'">[Następne >>]</a>'; 
        } 
 
        return implode('&nbsp;', $output); 
    } 
 
} # end class  
define('NUMBER_PER_PAGE', 1); 
define('PATH_TO_DIR', ''.$dane[2].''); 
$page = isset($_GET['page']) ? (int)$_GET['page'] : 1; 
$list = new PaginateDirectory(PATH_TO_DIR, $page); 
 
?>
<table width="100%"> 
    <tr> 
        <td><?php echo $list->displayCount(); ?></td> 
        <td><?php echo $list->displayLinks(); ?></td> 
    </tr> 
    <tr> 
        <td colspan="2"> 
        <? foreach( $list->getImages() as $index => $image ): ?> 
            <fieldset> 
            <legend><?php echo basename($image['file']); ?></legend> 
            <img src="<?php echo basename($image['file']); ?>" <?php echo $image['dimensions']; ?> style='max-wdth: 600px;' /> 
            </fieldset> 
            <br />
            <? endforeach; ?>  
        </td> 
    </tr> 
    <tr> 
        <td><?php echo $list->displayCount(); ?></td> 
        <td><?php echo $list->displayLinks(); ?></td> 
    </tr> 
</table> 
 
<?php }  ?>
[PHP] pobierz, plaintext 

I wszystko fajnie działa, pokazuje się paginacja dolna i górna, tylko nie wyświetla się zdjęcie czyli pętla foreach.

[PHP] pobierz, plaintext 
        <? foreach( $list->getImages() as $index => $image ): ?> 
            <fieldset> 
            <legend><?php echo basename($image['file']); ?></legend> 
            <img src="<?php echo basename($image['file']); ?>" <?php echo $image['dimensions']; ?> style='max-wdth: 600px;' /> 
            </fieldset> 
            <br />
            <? endforeach; ?>  
[PHP] pobierz, plaintext 

bardzo proszę o pomoc co może być nie tak że nie wykonuje się ta pętla.
PATH_TO_DIR jest ścieżką do pliku gdzie są duże zdjęcia.

[KsaR]

A gdzieś dodajesz do własności $aImages elementy? Bo nie widze.
Poza tym używaj <?php zamiast samego <?.

[mb83]

a możesz napisać jak dodawać te elementy?
skrypt znalazłem na jakimś zagranicznym forum, nie za bardzo wiem jak te elementy tam dodać.
Bo chodzi pewnie o to że $aImages nic nie zwraca tak?

[KsaR]

a możesz napisać jak dodawać te elementy?
skrypt znalazłem na jakimś zagranicznym forum, nie za bardzo wiem jak te elementy tam dodać.
Bo chodzi pewnie o to że $aImages nic nie zwraca tak?

Wywołujesz metode getImages(); ktora zwraca wlasnosc $this->aImages.

Jednak to jest puste bo nigdzie do tej tablicy nie dodajesz..

Mozesz np. Tak:

$this->aImages['klucz']='wartość';

Jak się nie myle.

[mb83]

super, ale co jest kluczem a co wartością?
czy to ma być coś na zasadzie:
$this->aImages['file']
w którym miejscu to ma być?

skrypt jest stąd:
http://forums.devshed.com/php-development/...tml#post1279792

Ten post edytował mb83 15.08.2015, 20:44:37

[MESSIAH :)]

a możesz napisać jak dodawać te elementy?
skrypt znalazłem na jakimś zagranicznym forum, nie za bardzo wiem jak te elementy tam dodać.
Bo chodzi pewnie o to że $aImages nic nie zwraca tak?

Jak Ty to robisz? Znalazłeś kod z 2005 roku i chcesz go teraz odpalić? Będziesz musiał go trochę zmodyfikować aby zadziałał i nie zapomnij zastąpić metodę mysql na nowszą.

[mb83]

Jak Ty to robisz? Znalazłeś kod z 2005 roku i chcesz go teraz odpalić? Będziesz musiał go trochę zmodyfikować aby zadziałał i nie zapomnij zastąpić metodę mysql na nowszą.

ale kod działa tylko nie działa jedna pętla.
może inaczej: da się ten kod odpalić nie uwzględniając przepisania wszystkiego od nowa?
czy jest to napisane tak, że kompletnie się nie da?



trochę łopatologicznie ale działa :-)

[PHP] pobierz, plaintext 
<img src="/img/galerie/<?php echo $_GET['id']; ?>/d/<?php echo basename($image['file']); ?>" <?php echo $image['dimensions']; ?>/>
[PHP] pobierz, plaintext 

problem jest gdy chcę wyświetlić jedno zdjęcie a nie 20:

[PHP] pobierz, plaintext 
define('NUMBER_PER_PAGE', 1);
[PHP] pobierz, plaintext 

gdy wyświetlam 10, 12 czy 20 zdjęć to jest tak jak ma być ale gdy chcę jedną albo dwie to już nie działa ta pętla.

pomożecie?

[wasyllinio]

przy jednym albo dwóch zdjęciach nie wyświetlisz nic. Kod który Ci sprawdza zadziała dopiero od ilości 3.
Przeanalizuj sobie metodę getResources

[PHP] pobierz, plaintext 
    function getResources($path){ 
        $container = array(); 
        $total = 0; 
        if (is_dir($path)){ 
            if ($handle = opendir($path)){ 
 
                $begin = ($this->iCurrentPage > 1)  
                            ?     (($this->iCurrentPage - 1) * NUMBER_PER_PAGE) 
                            :    0; 
                $end = $begin + NUMBER_PER_PAGE - 1; 
 
                while (false !== ($file = readdir($handle))){ 
                    if ($file != '.' && $file != '..' && !is_dir($path . $file)){ 
                        if( ($total > $begin) && ($total < $end) ){ 
                            $size = @getimagesize($path . $file); 
                            $container[$total] = array('file' => $file, 
                                                       'dimensions' => $size[3]  
                                                      ); 
                        } 
                        $total++; 
                    }                     
                } 
            } 
        } 
        $this->iNumberResources = $total; 
        return $container; 
    }  
[PHP] pobierz, plaintext 

Załóżmy, że

[PHP] pobierz, plaintext 
$begin = 0;
define('NUMBER_PER_PAGE', 1);
[PHP] pobierz, plaintext 

wtedy

[PHP] pobierz, plaintext 
$end = 0;
[PHP] pobierz, plaintext 

a potem spójrz na if'a w linii 57.

[PHP] pobierz, plaintext 
if( ($total > 0) && ($total < 0))
[PHP] pobierz, plaintext 

Identycznie będzie wyglądać przy

[PHP] pobierz, plaintext 
define('NUMBER_PER_PAGE', 2);
[PHP] pobierz, plaintext 

wtedy

[PHP] pobierz, plaintext 
$end = 1;
if( ($total > 0) && ($total < 1))
[PHP] pobierz, plaintext 

W obu przypadkach warunek nie jest spełniony.

Zmień może warunek na >= i <=

[PHP] pobierz, plaintext 
if( ($total >= $begin) && ($total <= $end) )
[PHP] pobierz, plaintext 

p.s.
jak to ogarniesz, to popraw trochę kod, bo definujesz klasę w pętli while od głównego zapytania...

Ten post edytował wasyllinio 16.08.2015, 15:27:25