!
Tweet
Moderator
if( session_status() == PHP_SESSION_NONE && ! headers_sent() ) {
session_start();
}
<?php
/**
* Description of botstop_class
*
* @author burncycle
*/
class botstop_class {
// was soll das public hier?
// diese Member sollten privat sein
public $file = "answers_questions.php";
public $question = "";
public $answer = "";
public $lang = "de";
// der Konstruktor sollte die Fragen und Antworten als
// Parameter erhalten und selbst kein include machen
public function __construct() {
// das @ entfernen und sesion_start mit einer Bedingung umschliessen
@session_start();
// siehe Methodenhinweis
include($this->file);
$rand = rand( 0, count( $question[$this->lang] ) - 1 );
$this->question = $question[$this->lang][$rand];
$this->answer = $answer[$this->lang][$rand];
}
// keinerlei Funktion, $this->file wird außer im Konstruktor
// nicht genutzt
public function set_file( $file ) {
$this->file = $file;
}
public function get_question() {
$_SESSION["answer"] = $this->answer;
return $this->question;
}
public function is_correct( $answers ) {
// Prüfung ob der array Index existiert fehlt
if( !empty( $_SESSION["answer"] ) ) {
// warum einfache Stringvergleiche mit aufwenigen RegExp?
// diesen ganzen If Zweig kann man durch einfaches
//
// return in_array( strtolower( $answers ), explode( "|", strtolower( $_SESSION["answer"] ) ) );
//
// ersetzen
$answerss = explode( "|", $_SESSION["answer"] );
if( count( $answerss ) > 1 ) {
foreach( $answerss as $value ) {
if( $bool = preg_match( "/".preg_quote( $value, "/" )."/i", $answers ) ) {
break;
}
}
} else {
$bool = preg_match( "/".preg_quote( $_SESSION["answer"], "/" )."/i", $answers );
}
// einfach "return $bool;" statt dem if/else
if( $bool == true ) {
return true;
} else {
return false;
}
} else {
// warum soll der User Cookies aktivieren - unnötig
// außerdem sollte die Klasse keine Ausgabe machen,
// sondern dem umliegen Skript einen entsprechenden Fehler
// zurückgeben
echo "Please Activate Cookies to submit botstop!";
return false;
}
}
}
// warum einfache Stringvergleiche mit aufwenigen RegExp?
// diesen ganzen If Zweig kann man durch einfaches
//
// return in_array( strtolower( $answers ), explode( "|", strtolower( $_SESSION["answer"] ) ) );
//
// ersetzen
// einfach "return $bool;" statt dem if/else
if( $bool == true ) {
return true;
} else {
return false;
}
class botstop_class
{
public function get_question($dataFile = 'answers_questions.php', $lang = 'de')
{
@session_start();
require __DIR__ . '/' . $dataFile;
$rand = rand(0, count($questions[$lang]) - 1);
$_SESSION["answer"] = $answers[$lang][$rand];
return $questions[$lang][$rand];
}
public function is_correct($answer)
{
@session_start();
if (empty($_SESSION["answer"])) {
echo "Please Activate Cookies to submit botstop!";
return false;
}
return in_array(
strtolower($answer),
explode("|", strtolower($_SESSION["answer"]))
);
}
}
Kommentar