Zła nawyki? 1. Mieszanie HTML z PHP jest ble. 2.
$odpowiedz = '<script language="javascript"> location.href="index.php?app=gamehub"</script>';
echo $odpowiedz;
można zmienić na:
echo '<script language="javascript"> location.href="index.php?app=gamehub"</script>';
Chociaż powinno być:
return '<script language="javascript"> location.href="index.php?app=gamehub"</script>';
3. Po co dodajesz NIC na końcach? Przykładowo:
$this->DB->update( 'news', array( 'tytul' => $tytul ), 'id='.$this->request['id'].'' );
Na:
$this->DB->update( 'news', array( 'tytul' => $tytul ), 'id='.$this->request['id'] );
4. Pamiętaj że:
$this->request['id']
Trzyma dane wysłane przez klienta (użytkownika) a więc w myśl zasady "każde dane które może wypełnić użytkownik zewnętrzny powinny być odpowiednio filtrowane". Zrzędzenie, ale potem parę osób przychodzi z płaczem że jakiś hakier zepsuł im forum. 5. Tworzysz funkcję z dwoma argumentami, oba są wymagane (przez brak danych początkowych), mimo to sprawdzasz:
if(empty($wysylanyTekst))
Całość może wyglądać tak:
public function checkNews($wysylanyTekst = 'Pusta wiadomosc', $wysylanyTytul = 'Brak tytulu')
{
$this->sendNews($wysylanyTekst, $wysylanyTytul);
return '<script language="javascript"> location.href="index.php?app=gamehub"</script>';
}
i... tyle. Jeżeli chodzi o "moduł", czy mi się wydaje że tworzysz pętle dla jednego wyniku z sql?
$wiadomos['id'] = $row['id'];
Nazwy zmiennych, funkcji, klas... wszystkiego, powinny być w języku angielskim.
$_POST
Przejmuje:
$this->request
Zamiast:
if($this->request['section'] == NULL)
I wstawianie między tego ifa całej zawartości możesz zrobić:
if($this->request['section'] != null)
I przerwać działanie poprzez:
return false;
if($this->request['section'] != null)
{
return false;
}
// Dalsza część kodu