[MySQL][PHP][PDO] Zapobieganie podwójnym rekordom. Opcje

[mlawnik]

Witam.
rać
Mam tabelę z Id i wartością. id używane jest w innej tabeli. Mój kod dodaje poprawnie rekordy, ale gdy ma dodać zduplikowany to zamiast pominąć wyrzuca błąd:

SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'subjectId' cannot be null

Fragment kodu:

[PHP] pobierz, plaintext 
<?php
        try {
            //put into database
            $queryHandleTwo = $pdo->prepare('INSERT INTO subjects SET subjectName=:subjectName');
            $queryHandleTwo->bindParam(':subjectName', $subjectName);
            $queryHandleTwo->execute();
            $subjectId = $pdo->lastInsertId('subjectId');
        } catch (PDOException $e) {
            if ($e->errorInfo[1] == 1062) {
                // duplicate entry, do something else
                $queryHandleOne = $pdo->prepare('SELECT subjectId FROM subjects WHERE subjectName = :subjectName');
                $queryHandleOne->bindParam(':subjectName', $subjectName);
                $subjectIdQuery = $queryHandleOne->fetch(PDO::FETCH_BOTH);
                $subjectId = $subjectIdQuery[0];
            } else {
                // an error other than duplicate entry occurred
                echo $e->getMessage();
            }
        }
?>
[PHP] pobierz, plaintext 

Chciałbym poprosić o nakierowanie na rozwiązanie.

Pozdrawiam,
mlawnik

[nospor]

Widocznie ten kod:

[PHP] pobierz, plaintext 
// duplicate entry, do something else
 
                $queryHandleOne = $pdo->prepare('SELECT subjectId FROM subjects WHERE subjectName = :subjectName');
 
                $queryHandleOne->bindParam(':subjectName', $subjectName);
 
                $subjectIdQuery = $queryHandleOne->fetch(PDO::FETCH_BOTH);
 
                $subjectId = $subjectIdQuery[0];
[PHP] pobierz, plaintext 

Nie jest wykonywany lub jest wykonywany źle. Ciezko sprawdzić co zwraca?

[sowiq]

Kod

Column 'subjectId' cannot be null

Czytanie ze zrozumieniem się kłania. W zapytaniu INSERT musisz ustawiać również wartość kolumny 'subjectId', bo Ci krzyczy, że nie może być NULL

[mlawnik]

@nospor Racja, już sprawdzam
@sowiq, ale jeżeli to jest AUTO_INCREMENT to chyba nie muszę?

[nospor]

@sowiq, ale jeżeli to jest AUTO_INCREMENT to chyba nie muszę?

oczywiscie ze nie muisz To dla sowiq czytanie ze zrozumieniem się klania

Powód edycji: [nospor]:

[sowiq]

oczywiscie ze nie muisz To dla sowiq czytanie ze zrozumieniem się klania

No chyba nie chcesz mi powiedzieć, że błąd "SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'subjectId' cannot be null" został wypluty po wykonaniu SELECT'a

Ten post edytował sowiq 10.06.2013, 13:53:09

[nospor]

sowiq ale czytaj ze zrozumieniem. Wyraźnie napisał że blad pojawia się dopiero po dodawaniu duplikatu, a nie za każdym razem. Wyraźnie wiec widać, że kod który pluje mu błedem nie jest tu podany..... tu jest podany kod na pobieranie id....

[sowiq]

Wyraźnie wiec widać, że kod który pluje mu błedem nie jest tu podany..... tu jest podany kod na pobieranie id....

To po co mu napisałeś, żeby sprawdził co zwraca podany kod, skoro błąd jest gdzie indziej?

@sowiq, ale jeżeli to jest AUTO_INCREMENT to chyba nie muszę?

No nie musisz, ale nie podałeś tego drobnego szczegółu

[nospor]

To po co mu napisałeś, żeby sprawdził co zwraca podany kod, skoro błąd jest gdzie indziej?

[Skup się]
ten własnie kod co kazalem mu sprawdzić zwraca mu złe ID, a konkretnie puste ID przez co kod, którego nie podał, wali komunikatem, ze nie podano ID....
[/koniec skupienia]
.....

[mlawnik]

To po co mu napisałeś, żeby sprawdził co zwraca podany kod, skoro błąd jest gdzie indziej?


No nie musisz, ale nie podałeś tego drobnego szczegółu

@sowiq, wybacz, wydało mi się oczywiste że id to AUTO_INCREMENT UNIQUE, mój błąd.

@nospor, już doszedłem, czemu nie działało. Zapytanie trzeba po prostu wykonać:

[PHP] pobierz, plaintext 
        } catch (PDOException $e) {
            if ($e->errorInfo[1] == 1062) {
                // duplicate entry, do something else
                $queryHandleOne = $pdo->prepare('SELECT subjectId FROM subjects WHERE subjectName = :subjectName');
                $queryHandleOne->bindParam(':subjectName', $subjectName);
                $queryHandleOne->execute();// o w ten sposób
                $subjectIdQuery = $queryHandleOne->fetch(PDO::FETCH_ASSOC);
                $subjectId = $subjectIdQuery['subjectId'];
            } else {
                // an error other than duplicate entry occurred
                echo $e->getMessage();
            }
        }
[PHP] pobierz, plaintext 

Czy jest jakiś inny ładniejszy sposób na to?

Pozdrawiam,
mlawnik

[nospor]

. Zapytanie trzeba po prostu wykonać:

Nom, czyli błąd był dokładnie tam gdzie kazałem szukać. Na przyszłość sam sprawdzaj tak oczywiste rzeczy

[sowiq]

No OK, w takim razie jesteś lepszą wróżką niż ja. Ja szukałem problemu w podanym przez autora kodzie. IMO było bardzo prawdopodobne, że błąd był generowany przez pierwsze zapytanie.

To chyba przyzwyczajenie, że jak ktoś pisze, że problem sprawia mu cytowany kod, to szukam problemu w cytowanym kodzie. A nie, czekaj... to nie przyzwyczajenie tylko normalność

Pozdrawiam.

[nospor]

No OK, w takim razie jesteś lepszą wróżką niż ja. Ja szukałem problemu w podanym przez autora kodzie. IMO było bardzo prawdopodobne, że błąd był generowany przez pierwsze zapytanie.

Ale tu nie było co wróżyć.... wyraźnie napisał że bląd pojawia się dopiero przy duplikacie, a to wyjasniało wszystko.
Fakt, jakby autor raczył podać dalszą cześć kodu, to pewnie byłoby to jeszcze bardziej czytelne

Powód edycji: [nospor]:

[mlawnik]

Pomógł leci.

Pozdrawiam,
mlawnik

Ten post edytował mlawnik 10.06.2013, 14:14:02