[PHP] if statement met variable check... hmm...

[Verwijderd]

ok check this:

PHP:

function required_fields ($reqfields) {

    if($reqfields){
    // Not all fields filled in? Return and give error!
    $returnerror = "1";
    return $returnerror;
    }
}

(stripped code)

PHP:

$requiredfields = '$firstname == NULL';
required_fields($requiredfields);

dit zou dus moeten checken of $firstname LEEG is ja of nee, en zo ja, moet de error geset worden...
Maar, een

PHP:

if($reqfields){ ...

werkt niet... als ik daar (ipv $reqfields) gewoon

PHP:

if($firstname == NULL)

neerzet doet ie het prima...

Maar die $requiredfields verschilt van keer tot keer... Hoe kan ik dit oplossen???

[Verwijderd]

PHP:

$requiredfields = $firstname == null;

of met haakjes als je je daar lekkerder bij voelt, al ziijn ze niet nodig:

PHP:

$requiredfields = ($firstname == null);

overigens zou ik isset() gebruiken:

PHP:

$requiredfields = !isset($firstname);

[ Voor 0% gewijzigd door Verwijderd op 08-10-2002 00:19 . Reden: tags added ]

[WouZz]

Verwijderd schreef op 08 oktober 2002 @ 00:18:

PHP:

$requiredfields = !isset($firstname);

Dan zou ik voor empty() gaan, dan weet je tenminste zeker dat je variabele niet leeg is, ook al is die variabele wel geset..

PHP:

$requiredfields = empty($firstname);

[thomaske]

je bent op zoek naar eval Niet echt netjes, maar wel een oplossing

[Verwijderd]

je kan toch ook 2 paramaters meegeven, als $var en $value.
Ik neem aan dat je functie generiek is, dus dat firstname bijvoorbeeld ook lastname kan zijn.

Nu checkt ie op de hele string, dus incl. de ==

[ACM]

Volgens mij ga je met je huidige code-gedachte ergens gigantisch de fout in
Als je die code in je functie zou laten uitvoeren zijn ze namelijk nooit geset, tenzij je ze ook nog global gaat maken enzo.

Als ze uit je formulieren rollen kijk dan of ze bestaan in je $_GET of $_POST arrays.

Ik zou dus zoiets doen:

PHP:

// Returns true if $var exists in $array
function check_value($array, $var)
{
    return isset($array[$var]);
}

// Gebruik:
check_var($_POST, "firstname");

Of zoiets:

PHP:

// Returns true if $var exists in $array
function check_value($method, $var)
{
    if($method == "POST")
        return isset($_POST[$var]);
    elseif($method == "GET")
        return isset($_GET[$var]);
}
// Gebruik:
check_var("POST", "firstname");

Etc, waar je nog vele varianten op kan bedenken en extra checks op kan doen (is het wel een integer, is het geen lege string etc etc)

[Bosmonster]

WouZz schreef op 08 oktober 2002 @ 01:29:
[...]

Dan zou ik voor empty() gaan, dan weet je tenminste zeker dat je variabele niet leeg is, ook al is die variabele wel geset..

PHP:

$requiredfields = empty($firstname);

Alleen returnt empty() ook true als een waarde 0 is.. wat soms gewoon de bedoeling kan zijn

[Verwijderd]

Je moet er rekening mee houden dat PHP achter de schermen van alles doet met het type van de variabelen, waardoor een simpele if( $foobar ) vaak verkeerde waarden oplevert. Bv.:
while( $array ) {
}
waarbij $array een hash van integers is geeft problemen als 1 van de waarden 0 is, want dat vat PHP op als een false. Een dergelijk probleem heb jij waarschijnlijk ook.

[Verwijderd]

PHP:

$requiredfields = empty($firstname) || empty($surname) || empty($address) || empty($postalcode) || empty($city) || empty($email) || empty($year);

werkt niet terwijl ze toch zijn ingevuld... dit zou toch moeten betekenen dat als EEN van de waarden niet is ingevuld (of meerdere, maar in ieder geval 1) hij een error geeft...

als ik alleen

PHP:

$requiredfields = empty($firstname);

doe werkt ie prima...

sorry, ben n00b

[GoodspeeD]

Probeer dit eens:

PHP:

$requiredfields = '$firstname == NULL';

en dan in je functie dit:

PHP:

if( {$reqfields} )

Werkt in perl en zou ook moeten werken in php volgens mij. Doet in principe hetzelfde als eval().

[Verwijderd]

ja met EEN ENKELE werkt ie prima, maar als ik met || dus ga koppelen doet ie het niet meer echt...

[thomaske]

GoodspeeD schreef op 08 oktober 2002 @ 14:24:
Probeer dit eens:
[..]
Werkt in perl en zou ook moeten werken in php volgens mij. Doet in principe hetzelfde als eval().

zal vast werken, maar aangezien het in een functie moet werken zijn de variabelen niet global, valt allemaal op te lossen door ook de globals variabel te maken, maar dat wil je niet, want dat stinkt

[Verwijderd]

dat global maken, dus

PHP:

global $VARIABLE

moet dat nou in de hoofdfile of in de function?

[thomaske]

kan je dat zelf niet vinden?

[Verwijderd]

function dus
nee ik had een type foutje gemaakt en daardoor deej ie het niet... en toen dacht ik HUH WTF??

maar thanks...
jullie zijn wel super relaxed... hij werkt nouw.... (typefout)

[GoodspeeD]

thomaske schreef op 08 oktober 2002 @ 14:30:
[...]


zal vast werken, maar aangezien het in een functie moet werken zijn de variabelen niet global, valt allemaal op te lossen door ook de globals variabel te maken, maar dat wil je niet, want dat stinkt

Dat vraagt ie niet? Hij vraagt wat de syntax is voor wat hij wil en die geef ik em. Hoe hij die variabelen doorgeeft of wat dan ook, is weer een ander verhaal.

[thomaske]

GoodspeeD schreef op 08 oktober 2002 @ 15:39:
[...]


Dat vraagt ie niet? Hij vraagt wat de syntax is voor wat hij wil en die geef ik em. Hoe hij die variabelen doorgeeft of wat dan ook, is weer een ander verhaal.

Daar heb je helemaal gelijk in, maar ik kijk graag een beetje verder!
Want anders komt de topicstarter over een paar dagen terug met die vraag

[Verwijderd]

PHP:

function fix_string($str){
    $str = strtolower($str);
    $str = ucwords($str);
    $str = trim($str);
    return $str;
}

en in de hoofdfile:

PHP:

// Set an array with the fields to "fix"
$fields_to_fix = array($firstname, $surname, $address, $postalcode, $city, $cellphone, $email, $year);
// Fix array values
while($fixit = array_pop($fields_to_fix)){
    fix_string($fixit);
    }

dit zou dus de waarden $firstname, $lastname, etc iets mooier moeten laten uitzien...
het mooi maken opzich lukt wel... alleen hij doet de nieuwe mooie WAARDE niet binden aan de variabele naam $firstname, etc...

wat doe ik dan fout?

[GoodspeeD]

voeg dit er eens aan toe op het einde:

PHP:

$firstname = $fields_to_fix[0];
$surname = $fields_to_fix[1];
$address = $fields_to_fix[2];
$postalcode = $fields_to_fix[3];
$city = $fields_to_fix[4];