Witam serdecznie

Jestem tu nowy, dopiero uczę się obiektywności w php (wcześniej programowałem strukturalnie) i nie wiem czy gdzieś nie zbłądziłem. Piszę CMS'A na moją stronkę no i nie wiem czy debrzę postępuję. Mam taki kodzik:

klasa odpowiedzialna za połączenie z bazą danych:

[PHP] pobierz, plaintext 
<?php
class bd
{
    public $zapytanie;
    //przygotowanie danych do bindowania
    public function PrzygotujDoBazy($formula, $bindowanie)
    {
        global $baza;
        $this->zapytanie = $baza->prepare($formula);    
        
        if(count($bindowanie) > 0)
        {    
            foreach($bindowanie as $bind)
            {
                $this->zapytanie->BindValue($bind[0], $bind[1], $bind[2]);
            }
        }
        return $this->zapytanie->execute();
    }
    //zwraca zapytanie
    public function ZwrocZapytanie()
    {
        return $this->zapytanie;
    }
    //czysci zapytanie    
    public function WyczyscZapytanie ()
    {
        $this->zapytanie->closeCursor();
        unset($this->zapytanie);
    }
}
?>
[PHP] pobierz, plaintext 

klasa odpowiedzialna za logowanie

[PHP] pobierz, plaintext 
<?php
class uwierzytelnianie
{
    //start sesji
    public function StartSesji()
    {
        session_start();
    
        if (!isset($_SESSION['inicjuj']))
        {
            session_regenerate_id();
            $_SESSION['inicjuj'] = true;
            $_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
        }
    
        if($_SESSION['ip'] != $_SERVER['REMOTE_ADDR'])
        {
            $this->Przekierowanie("http://www.strona.pl");
        }
 
    }
    //forma do logowania
    public function FormaLogowanie($komunikat)
    {
        if(!empty($komunikat))
        {
            $string ="<p>".$komunikat."</p>";    
        }
        
        $string .= '
        <form method="post" action="index.php?akcja=logowaniesprawdz">
        Login:<input type="text" name="login">
        Haslo:<input type="password" name="haslo">
        <input type="submit" value="Zaloguj sie">
        </form>
        ';
        
        szablony::ZapisDoSzablonu("logowanie", $string);
    }
    //sprawdzenie uzytkownika i jego zalogowanie
    public function SprawdzLogowanie($login, $haslo)
    {
        if(!empty($login) && !empty($haslo))
        {
            $login = addslashes($login);
            $haslo = addslashes(sha1($haslo));
            
            $bind= array(
                         array(':login', $login, PDO::PARAM_STR),
                         array(':haslo', $haslo, PDO::PARAM_STR)
                         );
            bd::PrzygotujDoBazy("SELECT * from uzytkownicy WHERE login = :login and haslo= :haslo", $bind);
            if(($rzad = bd::ZwrocZapytanie()->fetch()) > 0)
            {
                $_SESSION['login'] = stripslashes($rzad['login']);
                $_SESSION['poziom'] = stripslashes($rzad['poziom']);
 
                bd::WyczyscZapytanie();
                $this->Przekierowanie("./index.php");
            }
            else
            {
                bd::WyczyscZapytanie();
                $this->FormaLogowanie("Zly login lub haslo");
            }            
            
        }
        else
        {
            $this->FormaLogowanie("Pola login oraz haslo musza zostac wypelnione");
        }
                
    }
    //wylogowywanie
    public function Wyloguj()
    {
        $_SESSION['login'] = $_SESSION['poziom'] = 0;
        unset($_SESSION['login']);
        unset($_SESSION['poziom']);
    }
    //import z klasy szablony
    public function WyswietlSzablon($nazwa)
    {
        szablony::ImportSzablonu($nazwa);
    }
    //przekierowanie
    private function Przekierowanie($strona)
    {
            header("HTTP/1.1 301 Moved Permanently");
            header("location: {$strona}");
            header("Connection: close");
            exit;    
    }
}
?>
[PHP] pobierz, plaintext 

klasa odpowiedzialna za import szablonu

[PHP] pobierz, plaintext 
<?php
class szablony
{
    private $szablon = array();
    
    public function ZapisDoSzablonu($nazwa, $string)
    {
        $this->szablon[$nazwa] = $string;
    }
    
    public function ImportSzablonu($nazwa)
    {
        $szab = file_get_contents("./szablony/{$nazwa}.html");
        
        foreach($this->szablon as $nazwa => $string)
        {
            $naco[] = $string;
            $co[] = "<!--{$nazwa}-->";
        }
        
        $szab = str_replace($co,$naco,$szab);
        echo $szab;
    }
    
}
?>
[PHP] pobierz, plaintext 

Nie wiem może jestem trochę rozdrażniony ale chyba podobnych czy wręcz identycznych tematów różniących się tylko nazwą klasy na tym forum można już liczyć w dziesiątkach. Więc może warto by użyć opcji która wbrew pozorą nie jest magiczna SZUKAJ. Sprawdzić z czego inni bardziej doświadczeni programiści korzystają w dostępie do bazy danych czy też obsłudze sesji.

Bo tego typu podejściem to Wy ( mówie o wszystkich autorach tego typu wątków ), daleko nie zajdziecie. Sory za OT ale nie mogę już patrzeć na te posty :/

No i wykonanie

[PHP] pobierz, plaintext 
<?php
//polaczenie z baza danych
try
{
    $baza = new pdo("mysql:host=localhost;dbname=nazwabazy;", 'root', 'haslo', array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"));
    $baza -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
}
catch(PDOException $wyjatek)
{
    echo 'Wystapil blad z podlaczeniem do bazy danych, tresc bledu: ' . $wyjatek->getMessage();
    exit;
}
 
//Tworzenie obiektów
$uwierz = new uwierzytelnianie;
 
//czesc wlasciwa
$uwierz->StartSesji();
 
if(!empty($_SESSION['login']) && $_SESSION['poziom'] > 0)
{
    //tresc widoczna po zalogowaniu
}
else
{
    switch($_GET['akcja'])
    {
    case 'logowaniesprawdz':
        $uwierz->SprawdzLogowanie($_POST['login'], $_POST['haslo']);
        break;
    default:
        $uwierz->FormaLogowanie("Witaj, zaloguj sie aby uzyskac dostep do panelu administacyjnego");
    }    
    $uwierz->WyswietlSzablon("logowanie");
}
 
?>
[PHP] pobierz, plaintext 

Jeżeli ktoś mógłby mi powiedzieć co robię dobrze, a co źle byłbym wdzięczny.

Pozdrawiam



W zasadzie skoro na wszystko jest odpowiedź to po co pytać?

Pytać, jeśli problem, z którym się zmagasz jest naprawdę ciężki do rozwiązania. A to już kolejny tego typu wątek w tym tygodniu. Poszukaj, a znajdziesz. Jeśli chodzi o np. system szablonów, który nie ma szans na wydajne działanie - też o tym już było. Jeśli chodzi o nazwy zmiennych - używaj angielskich.


Zajrzyj do angielskiej Wikipedii o MVC, wszystko znajdziesz. Przejrzyj źródła gotowych frameworków. Przykro mi, ale programista, który nie jest w stanie poradzić sobie z wyszukiwarką niech lepiej skleja modele z papieru. Wszystko jest w dokumentacji, wszystko można znaleźć. Można też przeanalizować gotowe frameworki, dokumentację też mają napisaną, wystarczy wykazać odrobinę inicjatywy, a nie wklejasz listingi z nadzieją, że ktoś zrobi Ci po prostu audyt.