[PHP][MYSQL] Problem ze skryptem Opcje

[shtoc]

Witam mam takie pytanko napisałem skrypt który ma za zadanie przypisać message_id rekordom z taka zależnością że jak w bazie pole references jest puste to identiti = 0 a jak nie to ma wyciagnać wszystkie rekordy o takim samym message_id i przypisać im w message_id ta sama liczbe.

Niestety z pustymi mam ok ale z pełnym caly czas dostaje rozne liczby w takich samych rekordach. sprawdzałem przez echo $sql i mam tak ze znajduje przykładowo 3 takie same rekordy ale aktualizuje message_id tylko dla jednego ktory jest pierwszy 3 razy :/

Czy moze ktos powiedzieć gdzie jest błąd?

[PHP] pobierz, plaintext 
<?php
mysql_connect('localhost', 'admin', 'hasło') or die('BĹ‚Ä…d poĹ‚Ä…czenia: '.mysql_error());
mysql_selectdb('messages') or die('BĹ‚Ä…d wyboru bazy: '.mysql_error());
mysql_query("SET NAMES utf8");
 
$sql="SELECT * FROM messages2";
$dane=mysql_query($sql);
$i=1;
$a=0;
while($row = mysql_fetch_array($dane))
{
 
$references=mysql_escape_string($row['references']);
$id=mysql_escape_string($row['id']);
 
if (empty($references))
{
$sqll="UPDATE messages2 SET message_id='$a' WHERE id='$id'";
$danee= mysql_query($sqll);
 
}
else
{
$sqlr="SELECT * FROM messages2 WHERE `references`='$references' && message_id='0'";
$daner= mysql_query($sqlr);
 
 
    while($roww = mysql_fetch_array($daner))
    {
    $idd=mysql_escape_string($row['id']);
    $ref=mysql_escape_string($row['references']);
    $sqlll="UPDATE messages2 SET message_id='$i' WHERE id='$idd'";
    $daneee= mysql_query($sqlll);
 
    unset($idd,$ref);
    }
}
$i++;
 
}
 
?>
[PHP] pobierz, plaintext 

(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

Ten post edytował shtoc 27.04.2009, 16:34:02

[Darti]

Wiersz 29 to raczej

[PHP] pobierz, plaintext 
<?php
while($row = mysql_fetch_array($daner))
?>
[PHP] pobierz, plaintext 

p.s. nazywaj sensownie te zmienne to Ci się nie będą mylić ...
lepiej poświęcić 2 sekundy na wymyslenie nazwy zmiennej niż później 2 dni na szukanie błędu

Ten post edytował Darti 28.04.2009, 11:18:20

[slawny]

Ten 29 zostaw bez zmian. Zmień to:

[PHP] pobierz, plaintext 
<?php
$idd=mysql_escape_string($row['id']);
    $ref=mysql_escape_string($row['references']);
?>
[PHP] pobierz, plaintext 

na

[PHP] pobierz, plaintext 
<?php
$idd=mysql_escape_string($roww['id']);
    $ref=mysql_escape_string($roww['references']);
?>
[PHP] pobierz, plaintext 

[Darti]

Ten 29 zostaw bez zmian.

racja, to pętla w pętli ...

[shtoc]

Faktycznie dzięki (IMG:http://forum.php.pl/style_emoticons/default/exclamation.gif) wszystko przez literówkę.