[PHP] foreach met if statement

[Tweaqer]

Beste allemaal, ik heb een probleempje met php en ik kom er niet uit. Hopelijk kunnen jullie mij uitleggen waarom het fout gaat. Ik wil namelijk graag een website maken waarbij ik een lijst heb met alle films die ik gezien heb en die wil ik mijn eigen score meegeven. Om deze lijst te maken copy-paste ik de filmnamen uit een tekst bestandje in een html-formulier en die verstuur ik naar onderstaand php-script. Tot zover gaat het goed, maar ik wil voorkomen dat hij films dubbel in de database zet vandaar dat ik hier een check op wil uitvoeren. Echter voert hij de if-functie niet uit. Kijk ik ergens over heen? Wat begrijp ik verkeerd?
Ik hoop dat jullie mij kunnen helpen

PHP:

<?
include("dbconnect.php");
if ($_POST["films"] != "") {
    // filmlijst splitsen per regel
    $films = explode("\n", $_POST['films']);
    foreach ($films as $film) {
        
        // check database of film er al in staat
        $filmcheck = mysql_query("SELECT * FROM films WHERE titel LIKE '$film'");
        $num_rows = mysql_num_rows($filmcheck);
        
        // tijdelijke echo - hier gaat het nog wel goed, bijvoorbeeld: The Matrix = 0 of Away We Go = 1
        echo "$film = $num_rows<br />";

        // maar toch werkt deze if niet
        if ($num_rows = 0) {
                mysql_query("INSERT INTO films (gezien, titel, score)". "VALUES ('nee', '$film', '0')");
                echo "+ $film<br />";
        } 
        
    }
    mysql_close();
}
?>

[marco_balk]

PHP:

if ($num_rows == 0)

Verder zou je ook eens kijken naar een "INSERT ON DUPLICATE KEY" binnen MySQL.
Dan hoef je niet te checken of de film al bestaat, maar INSERT je gewoon alles, maar komt er geen dubbele records in de DB (mits je KEYS goed staan).

[RobIII]

PHP:

 if ($num_rows = 0) {

PHP:

 if ($num_rows == 0) {

Zoek de verschillen. Overigens is dit wel héél basic debugwerk...

[ Voor 35% gewijzigd door RobIII op 29-03-2010 21:03 ]

[Tharulerz]

het is ==, niet =

te laat

[ Voor 29% gewijzigd door Tharulerz op 29-03-2010 21:02 ]

[Tweaqer]

RobIII schreef op maandag 29 maart 2010 @ 21:02:

Zoek de verschillen. Overigens is dit wel héél basic debugwerk...

Je hebt helemaal gelijk, domme fout

marco_balk schreef op maandag 29 maart 2010 @ 21:02:
Verder zou je ook eens kijken naar een "INSERT ON DUPLICATE KEY" binnen MySQL.

Dat is inderdaad een mooie alternatieve oplossing waar ik mij even op ga inlezen Thanks ook voor het oplossen van de fout die ik gemaakt heb!

[naam]

een tip voor iedereen die dat soort fouten wel eens maakt met ifjes, keer het statement om, dus niet

PHP:

if ($myVar == 0)

maar

PHP:

if(0 == $myVar)

in het tweede geval zal php een fout geven aangezien je niet aan een 0 kan assignen omdat het geen variabele is.

[Tharulerz]

marabunta2048 schreef op maandag 29 maart 2010 @ 21:58:
een tip voor iedereen die dat soort fouten wel eens maakt met ifjes, keer het statement om, dus niet

PHP:

if ($myVar == 0)

maar

PHP:

if(0 == $myVar)

in het tweede geval zal php een fout geven aangezien je niet aan een 0 kan assignen omdat het geen variabele is.

Dat is pas onlogisch programmeren... In je hoofd zeg je toch ook "Als myvar gelijk is aan 0 dan..." en niet "Als 0 gelijk is aan myvar dan..."

[Janoz]

Hoezo? Beiden zijn toch volkomen equivalent? Het is een heel gebruikelijke conding standaard. Je ziet het vooral bij mensen die (vroeger) veel C gedaan hebben, omdat het probleem daar ook al veel voorkwam. (Tegenwoordig geeft de compiler vaak een warning)

Dus ja, je kunt in je hoofd heel goed zeggen "Als constante gelijk is aan variabele, dan..", en het is ook erg handig om dat aan te leren wanneer je in een taal werkt waar een toekenning als resultaat van een booleaanse expressie geinterpreteerd kan worden.

[Rvanlaak]

Waarom gebruik je een LIKE statement in je query als je niet eens '%...%' toevoegd? Nu is het een gewone vergelijking. Performance technisch is het vervangen van LIKE door = veel beter.

[Grijze Vos]

performance technisch resulteert het als het goed is in hetzelfde query executie pad.

[Fuzzillogic]

Gebruik gewoon een goede IDE. Die tekent gewoon een geel kringeltje onder dit soort dubieuze statements en meer. Netbeans bevalt me uitstekend, voor zowel PHP als Java.

@TS: ik neem aan dat dit gestripte voorbeeldcode is? Zo nee, kijk dan uit met SQL injection en andere security-zaken. Ik raad je aan om geen mysql_* functies te gebruiken. Dat is archaïsch. Tegenwoordig is PDO met prepared statements de betere oplossing.

[RobIII]

Fuzzillogic schreef op dinsdag 30 maart 2010 @ 00:41:
Tegenwoordig is PDO met prepared statements de betere oplossing.

offtopic:
Ik ben niet zo heel erg thuis in PHP maar PDO of MySQLi? Ik hoor steeds PDO maar wat is er mis met MySQLi (die ook prepared statements kent). Voor het kleine beetje php/mysql werk dat ik heb gedaan heb ik die laatste gebruikt en me dus (nog) nooit verdiept in PDO noch de voor/nadelen. Dat de procedural mysql_* eruit moet ben ik met je eensch. Ik vind hier weinig echt overtuigends tussen staan en PDO schijnt nogal buggy te zijn naar ik verneem her-en-der. Als ik de feature comparison er naast pak heeft MySQLi mijn voorkeur.

[ Voor 23% gewijzigd door RobIII op 30-03-2010 00:51 ]

[Fuzzillogic]

Je zegt het zelf al, PDO is veel meer OO-gericht. Voor mij is dat al voldoende om dat te gebruiken ipv mysqli. Buggy.. Mja, het zal vast, maar waar wij het voor gebruiken is dat niet echt een probleem gebleken. En natuurlijk kun je makkelijker van DB switchen dan met mysqli.

[Avalaxy]

marco_balk schreef op maandag 29 maart 2010 @ 21:02:

PHP:

if ($num_rows == 0)

Verder zou je ook eens kijken naar een "INSERT ON DUPLICATE KEY" binnen MySQL.
Dan hoef je niet te checken of de film al bestaat, maar INSERT je gewoon alles, maar komt er geen dubbele records in de DB (mits je KEYS goed staan).

Wat zijn de voor- en nadelen van beide methodes?

[_Thanatos_]

Waarom gebruik je een LIKE statement in je query als je niet eens '%...%' toevoegd? Nu is het een gewone vergelijking. Performance technisch is het vervangen van LIKE door = veel beter.

performance technisch resulteert het als het goed is in hetzelfde query executie pad.

FWIK gebruikt de LIKE vergelijking *nooit* indexen, dus dat is in dit geval het performanceverschil

[.oisyn]

Ook niet als je match-string niet met een wildcard begint? Voor iets als "aap%" zou gewoon een index gebruikt kunnen worden.

.edit: blijkbaar doet MySQL dat ook

[ Voor 19% gewijzigd door .oisyn op 30-03-2010 02:37 ]

[pedorus]

.oisyn schreef op dinsdag 30 maart 2010 @ 02:14:
Ook niet als je match-string niet met een wildcard begint? Voor iets als "aap%" zou gewoon een index gebruikt kunnen worden.

.edit: hmm blijkbaar niet dus

Hoezo die edit?

quote: http://dev.mysql.com/doc/refman/5.5/en/mysql-indexes.html

The index also can be used for LIKE comparisons if the argument to LIKE is a constant string that does not start with a wildcard character. For example, the following SELECT statements use indexes:

SQL:

SELECT * FROM tbl_name WHERE key_col LIKE 'Patrick%';
SELECT * FROM tbl_name WHERE key_col LIKE 'Pat%_ck%';

[.oisyn]

Omdat ik dom was en een test deed met EXPLAIN op een kolom waar geen index op stond

[HuHu]

RobIII schreef op dinsdag 30 maart 2010 @ 00:45:
[...]

offtopic:
Ik ben niet zo heel erg thuis in PHP maar PDO of MySQLi? Ik hoor steeds PDO maar wat is er mis met MySQLi (die ook prepared statements kent). Voor het kleine beetje php/mysql werk dat ik heb gedaan heb ik die laatste gebruikt en me dus (nog) nooit verdiept in PDO noch de voor/nadelen. Dat de procedural mysql_* eruit moet ben ik met je eensch. Ik vind hier weinig echt overtuigends tussen staan en PDO schijnt nogal buggy te zijn naar ik verneem her-en-der. Als ik de feature comparison er naast pak heeft MySQLi mijn voorkeur.

Volgens mij heb jij gelijk en is mysqli ook de betere keuze. Ook qua snelheid wint mysqli het van PDO.

Zeker als je dan nog iets gebruikt als Zend_Db uit het Zend Framework, dan is er helemaal geen enkele reden meer om te kiezen voor PDO.

[Janoz]

Fuzzillogic schreef op dinsdag 30 maart 2010 @ 00:41:
Gebruik gewoon een goede IDE. Die tekent gewoon een geel kringeltje onder dit soort dubieuze statements en meer. Netbeans bevalt me uitstekend, voor zowel PHP als Java.

Het beginnen met de constante heeft meer voordelen. Wanneer je bijvoorbeeld een equals met een constante hebt is het makkelijker om

"SomeString".equals(stringVar)

te doen. Dit gaat ook goed wanneer stringVar null is. Dat is niet iets wat een IDE zomaar voor je op gaat lossen (Tenzij je een checkstyle-achtige plugin gebruikt die een dergelijke volgorde af gaat dwingen).

[MueR]

* MueR mept de mensjes in rode jasjes.

Dan gooi ik hem wel over de heg.
Webdesign, Markup & Clientside Scripting >> Programming