- Klasse Buchungen - Klasse Arbeitszeiten (bzw. Abwesenheitszeiten) - Klasse Spesen
), die könnt ihr euch ja mal anschauen und mir konstuktive Kritik geben:
class newposting
{
private $time,$user,$cun,$job,$source,$charges,$type,$errmsg=array(),$error=false;
function __construct($user,$cun,$job,$type,$source)
{
// first part: check inputs
$this->time = time();
$this->user = (int)$user;
$this->cun = (int)$cun;
$this->job = (int)$job;
$this->source = $this->checkSource($source);
$this->charges = $this->getChargesFromSource();
$this->type = $this->checkType($type);
if(!$this->error) // only jump to next method if there is no error
{
// jump to next step (check user,cun,job with database)
}
}
private function checkSource($source)
{
// only add from allowed sources
$allowed = array("pda","spesen","portal");
if(in_array($source,$allowed))
return $source;
else
{
$this->setErrMsg("This source is not allowed");
return false;
}
}
private function checkType($type)
{
// only add with allowed types
$allowed = array("work","pause","end","night");
if(in_array($type,$allowed))
return $type;
else
{
$this->setErrMsg("This type is not allowed");
return false;
}
}
private function getChargesFromSource()
{
// Posting is only able for charges if it is from one of these sources
if($this->source == 'pda' || $this->source == 'spesen')
return true;
else
return false;
}
private function setErrMsg($msg)
{
$this->error = true;
$this->errmsg[] = $msg;
}
function getErrMsg($html=true)
{
if($html)
{
return implode('<br>',$this->errmsg);
}
else
{
return implode("\r\n",$this->errmsg);
}
}
}
:
:
$Mapper = new Mapper();
$Buchungen = new Buchungen();
...
$Mapper->save($Buchungen);
vielleicht doch zu den Anfängern zurück schieben?
Einen Kommentar schreiben: