Python+baza mysql Opcje

[blandzi]

Zaczne od tego ze posiada baze mysql z dwoma tabelami uczniowie i frekwenca_dzien. Napisalem skrypt ktory pobiera klucz z czytnika rfid a nastepnie ma sprawdzic czy taki klucz istnieje w tabeli uczniowie pod nazwa "uczen_id". Jezeli klucz taki istnieje skrypt ma wkleic to tabeli frekwencja_uczen wlasnie ten klucz i aktualna godzine w przeciwnym przypadku wypisac informacje ze nie ma takiego klucza. Prosze o sprawdzenie poprawnosci napisanego skryptu. Chodzi mi glownie czy dobrze wykorzystalem komendy, laczylem sie z baza itp. A co do "$klucz" szukam w jaki sposob jest to pobierane z czytnika wiec prosze sie tym w ogole nie przejmowac. Zapomniałbym skrypt pisany w pythonie z wykorzystaniem biblioteki mysql.

[SQL] pobierz, plaintext 
import MySQLdb
$conn = Mysqldb.connect("localhost", "user", "haslo") OR die ('Nie moge połączyć się z Mysql');
mysql_select_db (uczniowie) OR die ('Nie moge wybrać tabeli');
 
$klucz = "pobierany z czytnika rfid";
 
# Sprawdzanie czy istnieje $klucz=uczen_id w bazie
     $w = mysql_fetch_array(mysql_query(SELECT COUNT(uczen_id) FROM uczniowie WHERE uczen_id == '$klucz'));
 
# jeżeli nie -> napis, jeżeli jest -> dodaje uczen_id i obecny czas do tabeli frekwencja_dzien
IF ($w[0]{
INSERT INTO `frekwencja_dzien` (uczen_id, czas) VALUES ('$klucz',CURTIME());
}   else {
 
print "Nie ma takiego ucznia"
}
 
 
 
[SQL] pobierz, plaintext 

[thek]

Ogólnie wygląda na dobre, (ale tylko algorytm, a nie kod - uwagi niżej dałem) tylko nie jest potrzebne aż takie rozbudowane funkcjami sprawdzanie (IMG:style_emoticons/default/winksmiley.jpg) Wystarczy samo

[PHP] pobierz, plaintext 
$w = mysql_query('SELECT uczen_id FROM uczniowie WHERE uczen_id == \''.$klucz.'\'')
[PHP] pobierz, plaintext 

A to dlatego, że jeśli istnieje choć jeden taki uczeń to zwróci tu pewne wartości, jeśli zaś nie to FALSE. I tylko to wystarczy sprawdzać. Nas nie interesuje czy jest konkretna wartość tam, lub ilu takich jest. Ważne jest tylko istnienie, czyli wartość typu bool. A to już nam samo mysql_query zwróci :)No i zwróc uwagę na nawiasy, apostrofy i funkcje, bo o nich zupełnie zapomniałeś (IMG:style_emoticons/default/winksmiley.jpg) Popatrz, że masz choćby insert do bazy... ale to tylko jakiś text. Nie ma bowiem mysql_query (IMG:style_emoticons/default/winksmiley.jpg)
Ostatecznie masz więc:

[PHP] pobierz, plaintext 
import MySQLdb
$conn = Mysqldb.connect("localhost", "user", "haslo") or die ('Nie moge połączyć się z Mysql');
mysql_select_db (uczniowie) or die ('Nie moge wybrać tabeli');
 
$klucz = "pobierany z czytnika rfid";
 
$w = mysql_query('SELECT uczen_id FROM uczniowie WHERE uczen_id == \''.$klucz.'\'');
if( $w ) {
    mysql_query( 'INSERT INTO frekwencja_dzien (uczen_id, czas) values ( \''.$klucz.'\', CURTIME() )');
} else {
    echo 'Nie ma takiego ucznia';
}
[PHP] pobierz, plaintext 

Ten post edytował thek 18.09.2009, 14:27:39

[blandzi]

dzieki wszystko jasne temat do zamkniecia

[Riklaunim]

[PHP] pobierz, plaintext 
import MySQLdb
$conn = Mysqldb.connect("localhost", "user", "haslo") or die ('Nie moge połączyć się z Mysql');
mysql_select_db (uczniowie) or die ('Nie moge wybrać tabeli');
 
$klucz = "pobierany z czytnika rfid";
 
$w = mysql_query('SELECT uczen_id FROM uczniowie WHERE uczen_id == \''.$klucz.'\'');
if( $w ) {
    mysql_query( 'INSERT INTO frekwencja_dzien (uczen_id, czas) values ( \''.$klucz.'\', CURTIME() )');
} else {
    echo 'Nie ma takiego ucznia';
}
[PHP] pobierz, plaintext 

To nie jest kod Pythona, a raczej jakiś PHP - $, nawiasy klamrowe, mysql_query itd. To nie ma prawa działać ani w PHP ani w Pythonie (IMG:style_emoticons/default/winksmiley.jpg)