PHP expert |
|
function evaluate($post, $get, $s){ //process answers for every question on the page
$surveyMan = new SurveyManager();
$m = '';
$questionlist = $s->getQuestionList($post['curpage']);
if($questionlist != -1){
foreach($questionlist as $curQuestion){
if($_GET['prev'] != 'true'){
$subm = $curQuestion->processAnswers($post, $get, $s->getMode(), $s);
}
if($subm != ''){
$m .= $subm.'<br />';
}
}
}else{
echo $languageMan->getMessageString($s, 'sur_echo');
// yields error because languageman is not defined inside this function
}
return $m;
}
function evaluate($post, $get, $s){ //process answers for every question on the page $surveyMan = new SurveyManager(); $m = ''; $questionlist = $s->getQuestionList($post['curpage']); if($questionlist != -1){ foreach($questionlist as $curQuestion){ if($_GET['prev'] != 'true'){ $subm = $curQuestion->processAnswers($post, $get, $s->getMode(), $s); } if($subm != ''){ $m .= $subm.'<br />'; } } }else{ echo $languageMan->getMessageString($s, 'sur_echo'); // yields error because languageman is not defined inside this function } return $m; }
$post en $get zijn gelijk aan $_POST en $_GET, onaangepast:-)
wordt dus zo aangeroepen:
evaluate($_POST, $_GET, $nogeenvariabele);
evaluate($_POST, $_GET, $nogeenvariabele);
onleesbaar dus voor iemand die niet eerst alle files volledig heeft doorgespit... Allas voor mij moest ik hier dus een hoop aanpassingen in maken, en moest ik dus daadwerkelijk honderden klassen gaan overlopen voordat ik ook maar enigzins wist welke klasse welke verantwoordelijkheid had:s
Die onderste commentaar heb ik er bijgeschreven omdat dit dus onzin was en dat de eigenaar van dit script dit kon zien en hier iets aan doen:-) |