Keskustelu: Nettisivujen teko: Lisäys kantaan

Miten tämä pitäisi toteuttaa? Tossa on nyt se ongelma ettei toi koodi toimi koska if-lauseita on vähän siellä täällä. Koodin pitäisi tarkistaa ensimmäisenä että onko lisättävä leffa kannassa jo. Jos ei ole niin leffa lisätään kantaan.

<?php
if(isset($_POST['Submit']))
{
	if($kategoria<>"" and $nimi<>"")
	{
	$tulos=mysql_query("SELECT leffannimi FROM leffat WHERE leffannimi=\"$nimi\"");
	$lkm=mysql_num_rows($tulos);

		if ($lkm>=0)
		{
		print "Tietokannassa on jo tämä leffa!";
		}
		else
		}
		mysql_query("insert into leffat (id,leffannimi,kategoria,dvd,cd,pvm) values (\"$id\",\"$nimi\",\"$kategoria\",\"$dvd\",\"$cd\",\"$pvm\")");
		}
	}
}
?>

Sulla on sisennykset päin harakan liikavarvasta, sen takia se ei varmaankaan toimi

<?php
if(isset($_POST['Submit']))
{
    if($kategoria<>"" and $nimi<>"")
    {
        $tulos=mysql_query("SELECT leffannimi FROM leffat WHERE leffannimi=\"$nimi\"");
        $lkm=mysql_num_rows($tulos);

        if ($lkm>=0)
        {
            print "Tietokannassa on jo tämä leffa!";
        }
        else
    }
    mysql_query("insert into leffat (id,leffannimi,kategoria,dvd,cd,pvm) values (\"$id\",\"$nimi\",\"$kategoria\",\"$dvd\",\"$cd\",\"$pvm\")");
}
}
}
?>

<!-- Ehkä kuitenkin --->

<?php
if(isset($_POST['Submit'])) {
    if($kategoria<>"" and $nimi<>"") {
        $tulos=mysql_query("SELECT leffannimi FROM leffat WHERE leffannimi=\"$nimi\"");
        $lkm=mysql_num_rows($tulos);

        if ($lkm>=0) {
            print "Tietokannassa on jo tämä leffa!";
        }
        else {
            mysql_query("insert into leffat (id,leffannimi,kategoria,dvd,cd,pvm) values (\"$id\",\"$nimi\",\"$kategoria\",\"$dvd\",\"$cd\",\"$pvm\")");
        }
    }
}
?>

Ei sattana! Toimii... Ei kyl ennen oo ollut ongelmia sisennyksien kanssa!? :) No mut pääasia et toimii...

Tuli vaan uus ongelma... Leffan lisääminen ei toimi! hmmm...

Siis mitä? Sisennykset eivät vaikuta suoritukseen, vaan vain luettavuuteen.

<?php
       if ($lkm>=0)
        {
        print "Tietokannassa on jo tämä leffa!";
        }
?>

[edit] vertaa tarkalleen else haaraa ja siinä olevia sulkeita [/edit]
Jos lukumäärä on suurempi tai yhtäsuuri kuin nolla. Jos leffa löytyy nimellä kannasta rivejä kai olisi ainakin yksi.

if($lkm==true)
//tai
if($lkm != false)

Joo o...

Ei tuo sisennyksistä johtunut, vaan siitä, että koodissa oli elsen jälkeen } eikä {.

Joo huomasin itsekin saman :)) Ei oo helppoo :))

Deewiant kirjoitti:

Ei tuo sisennyksistä johtunut, vaan siitä, että koodissa oli elsen jälkeen } eikä {.

Jeps, hämäävä tuo kommenttini, tarkoitukseni oli kiinnittää huomiota siihen sisennykseen juuri siksi että sen avulla huomaa missä vika oli.

jarspa kirjoitti:

Tuli vaan uus ongelma... Leffan lisääminen ei toimi! hmmm...

Iske hieman debuggia väliin niin näet missä mättää, eli jokaisen if-lauseen jälkeen echotat jotain aiheeseen liittyvää kuten "submittia painettu", "leffa oli jo kannassa", "lisätään leffaa" ja saat selville mihin koodin suoritus tyssää. Jos stoppi tapahtuu itse queryssä, laitat perään or die mysql_error(); niin olet astetta lähempänä ongelmaa.