Skal man bruge isset?

isset() tjekker om variablen er defineret, ikke om den indeholder noget.

if() tjekker om den indeholder en værdi.

Se på denne simple form:

<?

if(isset($_REQUEST['soegeord'])) {
echo "han trykkede send";
}

if($_REQUEST['soegeord']) {
echo "han indtastede noget";
}
?>

<form method="post" action="?id=8">
<input type="text" name="soegeord"><input type="submit" value="Søg">
</form>

if tjekker da også bare om variablen _er_ .. !?
Tror det er versionsafhængigt.. Brug hellere
if(!empty($var)) {
    // hvis der står noget i $var
} else {
    // hvis IKKE der står noget i $var
}

så er du i hvert falde på den sikre side..

nicklasb: if() tjekker kun efter, om variablens værdi er falsk eller sand - i hvilket tilfælde variablen selvfølgelig også er nødt til at eksistere ;-)

Det er pæn programmeringsstil at undersøge om variablen rent faktisk eksisterer inden du begynder at forespørge på dens værdi og afhængigt af fejlrapporteringsniveauet, vil du få advarsler fra PHP hvis du benytter variable som ikke er definerede/deklarerede (kan ikke huske, hvilken der er mest korrekt)

Hvad skal jeg så bruge?

Hvad skal jeg f.eks bruge til de her?

If($_SESSION['logget_ind'] == '1' && $_POST['ret_vote'] == '1') {

    if($row['andet']) {
    $andet = '<br><br><b>Andet:</b><br>'.nl2br($andet);
    } else {
    $andet;
    }

if($_POST['form_kommentar']) {

når jeg skal se om en variable eksistere bruger jeg:

if ($variable != "") {
  //findes
}
else {
  //findes ikke
}

er det ikke gyldig kode?

... eller rettere når jeg ser om en variable indeholder noget må det være!

alle gange hvor man skriver

if($foo) for at se om $foo eksistere, skriver en kode der generere notice's...

hvis du sætter error_reporting = E_ALL, vil du se disse notice's - ikke så pænt hvis du skulle få lyst til at flytte til en server der er sat op med dette!

---

og der er forskel på if($var) og if(isset($var)) se fx

<?php
$var = 0;
if(isset($var)) echo 'var findes';
if($var) echo ' | den findes stadig!";
?>

vil kun skrive 'var findes' - da $var = 0 == false - så if($var) vil blive falskt!

---

man bør altid skrive sin kode efter error_reporting = E_ALL og register_globals = off - på den måde ved man at det vil virke på alle nyere servere, uanset opsætning!

Morten

Det vil sige at jeg skal skrive echo error_reporting = E_ALL; øverst på alle mine sider og så se om den kommer med nogle fejler?

nej, ikke echo - det er en php.ini indstilling ;)

set_error_reporting(E_ALL); tror jeg hedder for at sætte den via et script (og kun for det script)

Morten

hov, den hedder bare error_reporting(E_ALL);

http://php.net/error_reporting

Morten

Ja men min hjmmeside ligger på wannafind.dk

du kan godt sætte det i et script - så skal du bare gøre det hver gang! så skal du bruge den funktion jeg linker til

Morten

Kan du ikke give mig noget kode til det?

error_reportin(E_ALL); - er alt du skal bruge - sæt det i toppen af alle dine scripts og du er kørende - hvor svært kan det være? (læs manualen, jeg linker endda til den!)

Morten

Ok takker skriv lige som et svar..

Mener ikke man må/kan sætte fejl-niveau, hvis server kører i safe-mode...

svaa'r

tipsen -> der kan være noget om snakken :( men kender ikke wannafind.dk - så måske ingen safe-mode)

Morten