Tworzenie dodatkowych klas dla aplikacji

[teez]

Hej,

 

Ostatnio przeglądałem kilka aplikacji "od środka" i zauważyłem, że większość tworzy własną klasę, którą później używa w module. Próbuję uczyć się podobnie, nie wiem czy łapie do końca poprawnie - możecie mi podwiedzieć, czy robię dobrze? Wszystko działa, ale chcę mieć pewność, czy nie wyrabiam sobie czasem jakichś złych nawyków.

 

klasa -> http://wklej.org/id/1014419/

przykładowy moduł -> http://wklej.org/id/1014420/

Edytowane 16 Kwietnia 2013 przez TeeZ

[Macsch15]

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

[teez]

 

3. Po co dodajesz NIC na końcach?

 

Rozumiem to w taki sposób: where 'id='.$cos.'' tak jak masz zamkniecie cudzysłowia w cytacie. Ale ok, zmieniłem i zapamiętam.

 

 

 

Jeżeli chodzi o "moduł", czy mi się wydaje że tworzysz pętle dla jednego wyniku z sql?

Wydaje Ci się. W bazie mam kilka rekordów.

 

Co do $this->request, co gdy jest $_POST['id'] i $_GET['id'] ? Request w końcu przejmuje i GET'y i POST'y.

I $this->request nie łapie bbcode z edytora

$editor->process( $_POST['news'] );

Daje wynik ( dla przykładu Test ):

[b] Test [/b]

A $editor->process( $this->request['news'] );

<p><strong>Test</strong></p>

Edytowane 17 Kwietnia 2013 przez TeeZ

[Macsch15]

Najlepiej jakbyś sobie poobserwował działanie $this->request jako że nie jest to tylko zwykła globalna jak $_POST, dodaj do globalTemplate (na localhost):

<php>
error_reporting(0);
var_dump($this->request);
</php>

I przeklikaj parę stron

 

A jeżeli chodzi o bbcode: https://www.invisionpower.com/support/guides/_/advanced-and-developers/api-methods/34-editor-and-parsing-methods-r229