Forum PHP.pl

mam problem z pobieraniem zmiennej z linku w wyniku czego nie pokazuje wszystkich rekordow.
Po kliknieciu na link ktory wyglada tak:

[PHP] pobierz, plaintext 
<?php
<a href="http://www.strona.pl/index.php?p=obiekt&amp;o='.$wiersz['id'].'">'.$wiersz['nazwa'].'</a>
?>
[PHP] pobierz, plaintext 

link odczytuje ten skrypt:

[PHP] pobierz, plaintext 
<?php 
$o = $_GET['o'];
if ($_GET['p'])
    {
          $q="SELECT * FROM `obiekty` WHERE id='$o'";
            $r=mysql_query($q); 
            while ($wiersz = mysql_fetch_array($r))
    {        
echo
?>
[PHP] pobierz, plaintext 

czy jesto to prawidlowo napisane?

Konstrukcja linka jest poprawna tylko jeżeli jest wyświetlany za pomocą echo, czyli tak:

[PHP] pobierz, plaintext 
<?php
echo '<a href="http://www.strona.pl/index.php?p=obiekt&o='.$wiersz['id'].'">'.$wiersz['nazwa'].'</a>';
?>
[PHP] pobierz, plaintext 

W przeciwnym razie powienien wyglądać tak:

[HTML] pobierz, plaintext 
<a href="http://www.strona.pl/index.php?p=obiekt&o=<?php echo $wiersz['id'];?>"><?php echo $wiersz['nazwa'];?></a>
[HTML] pobierz, plaintext 

Natomiast w zapytaniu nie powinieneś się odwoływać do zmiennej $o tylko do o jako elementu tablicy $_GET.
Poza tym inaczej należy zagnieżdżać zmienne php w zapytaniu sql.

[PHP] pobierz, plaintext 
<?php
$q="SELECT * FROM `obiekty` WHERE id='".$_GET['o']."'";
?>
[PHP] pobierz, plaintext 

dzieki, wpisy sie pokazuja ale wystapil jakis dziwny blad nowe wpisy ktore teraz dodaje to wogole nie pokazuje. Tzn znajduje je ale nie wyswietla szczegolow wpisu.
Naprawde dziwne to.

Bez odpowiednich fragmentów kodu nic nie wskóramy...

Mógłbyś rozwinąć? Dlaczego odwoływanie się bezpośrednio do elementu tablicy $_GET jest lepsze? Moim zdaniem jest wręcz odwrotnie. Przed wykorzystaniem danych z $_GET powinno się je odpowiednio przefiltrować, aby być pewnym, że zawierają to czego się spodziewamy.

Jeżeli dane pochodzą z tablic typu $_GET lub $_POST, a tak jest w przypadku mariuszzzzzz, to tak czy inaczej trzeba je przefiltrować, obojętnie czy odwołasz się do nich tak: $_GET['o'] czy tak: $o. Natomiast druga metoda jest nieużywana w nowszych wersjach php. Jeśli chcesz szczegółów to poszukaj tematów o register_globals. Teraz jeżeli w forularzu masz pole o nazwie "login" to dostęp do niego odbywa się przez tablicę $_POST['login'] lub $_GET['login'].

Ten post edytował Foxx 13.03.2009, 00:26:37

W kontekście register_globals oczywiście się zgadzam. Chyba przeoczyłeś, że mariuszzzzzz na początku swojego kodu użył

[PHP] pobierz, plaintext 
<?php
$o=$_GET['o'];
?>
[PHP] pobierz, plaintext 

Najprawdopodobniej stąd wynika nieporozumienie

Masz rację, przeoczyłem to

co zauwazylem jest to ze moj inny skrypt ktory ma wyswietlac 10 ostatnich wpisow tez nie pokazuje tych najnowszych tzn te ktore zostaly dodane po wprowadzeniu zmiany na serwerze (mianowicie ustawienie register_globals na off). Moze ten problem ma do czynienia z sama baza danych? Mieliscie juz takie przypadki?
Jesli chodzi o skrypt to jest dobry. Ale i tak popatrzcie:

[PHP] pobierz, plaintext 
<?php
 
    $wynik=mysql_query("SELECT * FROM `obiekty` WHERE ok='yes' ORDER BY id DESC LIMIT 0, 5") or die('Blad zapytania');
    while ($wiersz = mysql_fetch_array($wynik))
    {
    if ($wiersz['id']!=$tablica[0])
    {
    echo '
    <div class="statniododane">
<p class="ostatniododanetekst"><a href="index1.php?p=obiekt&amp;o='.$wiersz['id'].'">
                                '.$wiersz['nazwa'].'</a><br />'.$wiersz['data'].'<br />
                            
 
                        
'.$wiersz['miasto'].'</p><br /><br style="line-height:20px" /></div>';
    $tablica[0]=$wiersz['id'];
    }
    }
    
?>
[PHP] pobierz, plaintext