Php tusciu laukeliu tikrinimas ir mysql irasymas

	else {
$klaida = "<br>";
if(empty($pavadinimas) || strlen($pavadinimas) < 5) $klaida .= "Jūs neįvedėte pavadinimo arba jis yra pernelyg trumpas<br>";
if(empty($skelbimas) || strlen($skelbimas) < 10) $klaida .= "Jūs neįvedėte aprašymo arba jis yra pernelyg trumpas<br>";
if(empty($telefonas) || strlen($telefonas) < 8) $klaida .= "Jūs neįvedėte telefono numerio arba jis neteisingas<br>";

if(strlen($klaida) > 2) 
{ echo '<br><font color="#ff0000">'.$klaida.'</font>';
} else {
if($galiojimas == 2) $galiojimas = strtotime("+2 weeks");
else $galiojimas = strtotime("+4 weeks");

$query = mysql_query(insertas)
if(!$query) echo 'Įvyko klaida, todėl jūsų skelbimas nebuvo pridėtas. Prašome pabandyti dar kartą.';
else { echo 'Jūsų skelbimas buvo sėkmingai įkeltas. <a href="'.$base.'skelbimas.php?cat='.$kategorija.'&subcat='.$ask.'&id='.mysql_insert_id().'">Peržiūrėti įkeltą skelbimą »</a>'; 

}
}

}
}

Na kodas toks, padariau, kad aptiktu jeigu laukelis tuscias arba jame truksta reiksmiu, viskas gerai - aptinka ir neikelia i duombaze, bet nors ir gerai viskas irgi neikelia, tesiog nieko neparaso, tuscias lapas. Gal kazkur kazko truksta?

$query = mysql_query(insertas) <-------------- truksta gale kabliataskio

------------------------------------------------

jei tau nemeta klaidos tai reikia isijungti php nustatymuose klaidu rodyma, arba skripto pradzioje isideti

error_reporting(E_ALL);

ini_set('display_errors', true);

-----------------------------------------------------

taip siulyciau stengtis rasyti tvarkingai koda, pvz visada rasyti if skliaustelius

if (salyga) {

kazkas;

}

arba

if (salyga)

{

kazkas

}

taip bus mazesne tikimybe suklysti

----------------------------------

taip pat kai dedi sklaustus susilygiuoti, kad matytum ar visi yra sklaustai

if (salyga) {
  if (salyga) {
     veiksmas;
  }

  if (salyga) {
     if (salyga) {
        veiksmas;
     }
     veiksmas;
  }
  veiksmas;
}

if (salyga){if (salyga) {
veiksmas;
}
if (salyga) {
if (salyga) {
 veiksmas;
    }
veiksmas;
}
veiksmas;
}

kur lengviau rasti trukstama skliausto? :)

Tavo tiesa, aciu uz patarima. :) Klaidu nemeta, balta, balta kur dairais :D

Tavo tiesa, aciu uz patarima. :) Klaidu nemeta, balta, balta kur dairais :D

kai balta balta

error_reporting(E_ALL);

ini_set('display_errors', true);

--------

arba tavo skriptas net nedaeina iki sitos vietos kuria nukopijavai, nes truksta jame pradzios

Daeina, nes jei truksta reiksmiu pavadinime pvz tai raso, kad truksta, bet jei viskas gerai neikelia.

Nesižaisk su mysql_connect, naudok http://php.net/mysqli arba http://php.net/pdo nes galu gale visi hostingai pradės naudoti PHP 5.4+ ir kodai nebeveiks tavo, o tada turėsi vargo viską pertvarkant.

Sita supratau, o arciau reikalo niekas nepades?

Sita supratau, o arciau reikalo niekas nepades?

http://stackoverflow.com/a/22253579

if(strlen($klaida) > 2) 

Is idejos ir sita salyga bloga, nes pagal

$klaida = "<br>"; 

tavo salyga jau turi 4 simbolius.

P.s. paskutini '}' nutrink nebent cia funkcija

else {
   $klaida = "<br>";
   if (empty($pavadinimas) || strlen($pavadinimas) < 5) 
	$klaida.= "Jūs neįvedėte pavadinimo arba jis yra pernelyg trumpas<br>";
   if (empty($skelbimas) || strlen($skelbimas) < 10) 
	$klaida.= "Jūs neįvedėte aprašymo arba jis yra pernelyg trumpas<br>";
   if (empty($telefonas) || strlen($telefonas) < 8) 
	$klaida.= "Jūs neįvedėte telefono numerio arba jis neteisingas<br>";
   if (strlen($klaida) > 2) {
       echo '<br><font color="#ff0000">' . $klaida . '</font>';
   } else {
       if ($galiojimas == 2) 
		$galiojimas = strtotime("+2 weeks");
       else 
		$galiojimas = strtotime("+4 weeks");

	$query = mysql_query(insertas);

	if (!$query) 
		echo 'Įvyko klaida, todėl jūsų skelbimas nebuvo pridėtas. Prašome pabandyti dar kartą.';
       else 
	{
           echo 'Jūsų skelbimas buvo sėkmingai įkeltas. <a href="' . $base . 'skelbimas.php?cat=' . $kategorija . '&subcat=' . $ask . '&id=' . mysql_insert_id() . '">Peržiūrėti įkeltą skelbimą »</a>';
       }
   }
}

if(strlen($klaida) > 2) 

 

Is idejos ir sita salyga bloga, nes pagal

 

$klaida = "<br>"; 

tavo salyga jau turi 4 simbolius.

 

P.s. paskutini '}' nutrink nebent cia funkcija

 

else {
   $klaida = "<br>";
   if (empty($pavadinimas) || strlen($pavadinimas) < 5) 
	$klaida.= "Jūs neįvedėte pavadinimo arba jis yra pernelyg trumpas<br>";
   if (empty($skelbimas) || strlen($skelbimas) < 10) 
	$klaida.= "Jūs neįvedėte aprašymo arba jis yra pernelyg trumpas<br>";
   if (empty($telefonas) || strlen($telefonas) < 8) 
	$klaida.= "Jūs neįvedėte telefono numerio arba jis neteisingas<br>";
   if (strlen($klaida) > 2) {
       echo '<br><font color="#ff0000">' . $klaida . '</font>';
   } else {
       if ($galiojimas == 2) 
		$galiojimas = strtotime("+2 weeks");
       else 
		$galiojimas = strtotime("+4 weeks");

	$query = mysql_query(insertas);

	if (!$query) 
		echo 'Įvyko klaida, todėl jūsų skelbimas nebuvo pridėtas. Prašome pabandyti dar kartą.';
       else 
	{
           echo 'Jūsų skelbimas buvo sėkmingai įkeltas. <a href="' . $base . 'skelbimas.php?cat=' . $kategorija . '&subcat=' . $ask . '&id=' . mysql_insert_id() . '">Peržiūrėti įkeltą skelbimą »</a>';
       }
   }
}

Salyga gera, del } ir klaidos nemeta, isvis man nieko neparaso, nei klaidos nieko. Jeigu kazko nedarasai, ismeta $klaida t.y. paraso, kad neuzpildei to ar ano. O jeigu viska gerai surasai, tai nieko neparaso.

zinau kad smagu yra spresti galvosukius, bet gal galetum siek tiek sugadinti mums dziaugsma ir nukopijuoti visa failo koda, o ne nuo puses :)

gal tada lengviau bus atspeti kas negerai