[php] Porównanie tablicy z wpisami w bazie danych., Problem natury logicznej Opcje

[kartun11]

Witam, już 3 dni męczę się z takim prostym kodem i nie mam pojęcia co jest nie tak.

[PHP] pobierz, plaintext 
<?php
//...
$repeat_num = 0; // Liczba powtarzających się kodów
$added_num = 0; // Liczba dodanych kodów
    
    for($i = 0; $i < $arrsize; $i++) { //Zmienna $arrsize to ilość elementów w tablicy
    
        $isindb = FALSE; //Zmienna informuje o tym, czy dany kod już istnieje w bazie
        
        while($row = mysql_fetch_row($result)) { //$result to wynik zapytania do bazy
            if($code[$i] == $row[0]) { //$code to tablica z kodami utworzona z pliku
                $isindb = TRUE; //Kod istnieje w bazie
                $repeat_num++; //Zwiększam ilość powtórzeń
                break;
            }
        }
 
        if($isindb === FALSE) { //Jeżeli kodu nie ma w bazie
            $query = "INSERT INTO codes VALUES('$code[$i]');"; // Zapytanie do bazy danych
            mysql_query($query) or die("<h1><span style='color: red;'>Blad w zapytaniu</span></h1>"); // Dodaje kod
            $added_num++; //Zliczamy dodany kod
        }
    }
//...
?>
[PHP] pobierz, plaintext 

Za pierwszym razem dodaje mi wszystkie kody z pliku i jest ok. Gdy próbuję dodać kody drugi raz, też wszystko działa i zlicza powtórzenia.
Problem się pojawia, gdy usunę jakiś kod w bazie i będę chciał dodać kody jeszcze raz. Teoretycznie powinno dodać mi tylko jeden kod.
Wtedy wyglądało by to tak, że dodano jeden kod, a reszta się powtarza. Mi jednak działa to tak, że gdy natrafi na kod, którego nie ma w bazie dodaje go, ale później dodaje już wszystkie kody. Nie mam pojęcia co jest nie tak.
Proszę o pomoc.

Ten post edytował kartun11 11.06.2009, 21:04:17

[daniel1302]

Spróbuj dodać w tym if'ie
if($isindb === FALSE){}

na końcu $isindb = TRUE;

[Crozin]

Nie ma sensu wykonywać n zapytań dodających kod do tabeli.

Zrób tak:

[PHP] pobierz, plaintext 
<?php
 
//lista kodów do dodania
$codes = array(
  123, 321, 432, 234,
  542, 423, 132, 534
);
 
$exists = array();
 
//oczywiście jeżeli kody nie są numeryczne, powinieneś zadbać o zbudowanie poprawnego zapytania
$query = sprintf('SELECT * FROM tbl_name WHERE code IN (%s);', implode(', ', $codes));
$result = mysq_...
while(...){
  $exists[] = $row['code'];
}
 
$insert = array_dif($codes, $exists);
foreach($insert as &$i){
  $i = '(' . $i . ')';
}
 
$query = sprintf('INSERT INTO tbl_name VALUES %s;', implode(', ', $insert));
mysql_query($query);
?>
[PHP] pobierz, plaintext 

Ten post edytował Crozin 11.06.2009, 19:53:35

[kartun11]

Spróbuj dodać w tym if'ie
if($isindb === FALSE){}

na końcu $isindb = TRUE;

Nie pomogło.

A co do tego innego sposobu, to pewnie jest bardziej wydajny, ale jak dla mnie bardziej skomplikowany.
Skrypt będę odpalał może raz na miesiąc, więc nie potrzebuje wydajności.
Mam już napisane to w trochę inny sposób, ale chodzi mi bardziej o to, dlaczego mój kod nie działa tak jak chcę.