Forum PHP.pl

Witam

Zrobiłem sobie galerię internetową (nie ukrywam, że na podstawie jakiegoś tam skryptu). Otóż offline, gdy testuje stronkę na swoim kompie wszystko elegancko chodzi, natomiast po wrzuceniu na stronkę nie działają mi przyciski "dalej" i "wstecz", czy ktoś mógłby rzucić okiem co jest nie tak na poniższy kod. Bardzo proszę o pomoc w tej sprawie i z góry dziękuję za wszelkie sugestie.

[PHP] pobierz, plaintext 
<?php
 
$pic_info_file="/naw_galer/pics1.txt";
 
 
 
 
print "<CENTER>";
 
 
 
  if ( !file_exists($pic_info_file))
  {
        $dh = opendir( "galeria/absolut" );
        while( $file = readdir( $dh ) )
        {
                if (    ereg("jpg$",$file) || ereg("JPG$",$file) ||
                        ereg("jpeg$",$file) || ereg("JPEG$",$file) ||
                        ereg("Jpg$",$file) || ereg("Jpeg$",$file) )
                {
                        $pic_info[] = $file;
                }
        }
  }
  else $pic_info=file("/naw_galer/pics.txt");
 
 
  $number_of_pics = count ($pic_info);
  if (($picture > $number_of_pics) || ($picture == $number_of_pics) || !$picture)
    $picture = '0';
  $item = explode (";", $pic_info[$picture]);
 
  
  print "<b>$item[1]</b>";
 
 
  $next = $picture + 1;
  if ($picture > 0 ) $back = $picture - 1;
  else $picture = '0';
 
  
  print "<a href=$PHP_SELF?picture=$back><u>Wstecz</u></a>";
  print "          ";
  print "<a href=$PHP_SELF?picture=$next><u>Dalej</u></a><br>";
 
  
  print "<br><img src='galeria/absolut/$item[0]' border='3' width='610px' height='410px'><br> <br>";
 
 
 
print "</CENTER></BODY></HTML>";
?>
[PHP] pobierz, plaintext 

Ten post edytował simon111 11.03.2009, 20:46:12

[PHP] pobierz, plaintext 
<?php
print "<a href=$PHP_SELF?picture=$back><u>Wstecz</u></a>";
?>
[PHP] pobierz, plaintext 

$PHP_SELF już się nie powinno używać.

PS. Dodaj tag do tematu.

Chyba równie dobrze mógłbyś dać link bez $PHP_SELF

[HTML] pobierz, plaintext 
<a href="?picture=$back">
[HTML] pobierz, plaintext 

i chyba byłby ten sam efekt...

Ten post edytował Lion_87 11.03.2009, 22:46:16

Witajce

Usunąłem całkowicie $PHP_SELF - nic to nie dało, wpisałem mu też na sztywno nazwę pliku galerii (czyli galeria2.php):

[PHP] pobierz, plaintext 
<?php
print "<a href=galeria2.php?picture=$back><u>Wstecz</u></a>";
?>
[PHP] pobierz, plaintext 

a także przypisałem zmiennej $PHP_SELF="galeria2.php" i dalej nic :/

Co wy na to? Podam wam stronkę na której to jest zamieszczone, a sami zobaczycie działanie:
http://www.mechanika.ip.pwsz.edu.pl/log/galeria2.php

PS. Najdziwniejsze, że offline wszystko chodzi, a na serwerze nic nie chce...

A pobierasz gdzieś z tablicy $_GET numer zdjecia? Nie.

Czy mógłbyś mi napisać jak mam pobrać wartość z tablicy $item funkcją GET?
Spróbuję wówczas w ten sposób to zapisać na stronie.

$_GET to nie funkcja. Tylko tablica.
Przejrzyj ja http://php.net/print_r
http://pl.php.net/manual/pl/language.varia...uperglobals.php

Przejrzałem linki które podałeś "wookieb", ale niejak mogłem to odnieść do mojego przykładu jedyne co mi przyszło do głowy to coś takiego:

[PHP] pobierz, plaintext 
<?php
...
$_GET = explode (";", $pic_info[$picture]);
...
...
...
 
print "<br><img src='galeria/absolut/$_GET[0]' border='3' width='610px' height='410px'><br>&nbsp;<br>";
...
...
...
?>
[PHP] pobierz, plaintext 

reszta taka sama. Ale działa mi tak samo - offline chodzi, a na serwerze nie chce :/. Czy dobrze sformułowałem powyższy kod?

Sformułowałeś go dość niefortunnie, próbując do tablicy superglobalnej przypisać jakąś tablicę, ale załóżmy że to będzie działać.

Lepiej byłoby nazwać tę zmienną nie $_GET, a jakkolwiek inaczej. $_GET jest praktycznie zarezerwowane dla zmiennych szczególnego typu. Twój kod zadziała, ale nie ma sensu tak pisać, nazwij zmienną inaczej. Poza tym jest ok, ale nie rozumiem, do czego Twój kod ma służyć. Podobnie jak nie rozumiem, co to znaczy `nie działają przyciski dalej i wstecz`. Uściślij to jakoś, napisz co takiego się dzieje.

Dziękuję bardzo "wookieb" twój post bardzo mi pomógł.
Wkońcu galeria działa tak jak mi chodziło, zrobiłem to wprawdzie dość niezgrabnie (mam na myśli postać skryptu) i na pewno dałoby się to jeszcze zoptymalizować, no ale w końcu cel uświęca środki. Ostatecznie mój skrypt wygląda tak:

[PHP] pobierz, plaintext 
<?php
 
$pic_info_file="/naw_galer/pics1.txt";
 
 
 
print "<CENTER>";
 
 
 
  if ( !file_exists($pic_info_file))
  {
        $dh = opendir( "galeria/absolut" );
        while( $file = readdir( $dh ) )
        {
                if (    ereg("jpg$",$file) || ereg("JPG$",$file) ||
                        ereg("jpeg$",$file) || ereg("JPEG$",$file) ||
                        ereg("Jpg$",$file) || ereg("Jpeg$",$file) )
                {
                        $pic_info[] = $file;
                }
        }
  }
  else $pic_info=file("/naw_galer/pics.txt");
 
 
  $number_of_pics = count ($pic_info);
  if (($picture > $number_of_pics) || ($picture == $number_of_pics) || !$picture)
    $picture = '0';
  $item = explode (";", $pic_info[$picture]);
 
 
$picture=$_GET['picture'];
 
 
  $next = $picture + 1;
  if ($picture > 0 ) $back = $picture - 1;
  else $picture = '0';
 
 
 
 
$results=$picture.'.jpg';
 
 
 
 
 
  print "<a href=?picture=$back><u>Wstecz</u></a>";
  print "&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; ";
  print "<a href=?picture=$next><u>Dalej</u></a><br>";
 
 
 
 
 
  print "<br><img src='galeria/absolut/$results' border='3' width='610px' height='410px'><br>&nbsp;<br>";
 
 
 
 
 
print "</CENTER></BODY></HTML>";
?>
[PHP] pobierz, plaintext