tjek mail / include function

har du oprettet en instans af den klasse du includer?

instans?

class validering{
    private $error=false;

    public function __construct(){}
/**
* Denne funktion tjekker om en email adresse er gyldig
*
* @param string $mail
* @param string $fejl
* @return bool
*/
    public function  mail($mail, $fejl='Den anførte email adresse er ugyldig!'){
        if (preg_match("/[[:alnum:]][a-z0-9_\.\-]*@[a-z0-9\.\-]+\.[a-z]{2,4}$/", $mail)) {
            return true;
        } else {
            $this->error[]=$fejl;
            return false;
        }
    }

instance...

$validering = new ClassNavn();

ClassNavn ændres til klassens navn!

$mail_tjek = new validering();
if($mail_tjek->mail('din@email.dk')){
  //dit sql-gøjl
}

<?php
$Email = $_POST['Email'];
include('class.validering.php');

//CLASS FUNKTIONER
$validering = new validering();

if($validering->mail($Email)){
$sql = "INSERT INTO ansatte (Email) VALUES ('$Email')";
$result = mysql_query($sql);
header('location: index.php');
}else{
echo "Den ansatte kunne ikke oprettes pga. emailen ikke er godkendt...";
}
?>

Jeg ville nok lave funktionen static hvis du kun kalder denne...

-> #6 - var osse min første tanke da jeg så klassen, det er måske lige voldsomt nok til så simpel en ting :)

puha, tusind tak :D
går stadig i skole, så det var en meget stor hjælp! takker!

Alt virker!

Lidt tricky med point her synes jeg :/
20 til splazz
40 til zynzz

det er så i orden :)

svar

-> #7 ja måske nok, men at starte klassen bare for at se en enkelt ting er også voldsomt.. :)

Men hvis $this->error[] skal være der kan det jo ikke lade sig gøre...


Hvad ang. valideringen så er der jo kun grund til retunere true eller false for alle funktionerne ?

Undskyld jeg først har sendt point's nu.

det går nok - tak for dem :)