Tema
Boje
Pismo
Registracija

Novo

Forum kao aplikacija na telefonu

Dodajte forum na početni ekran i koristite ga kao aplikaciju, sa menijem na dnu ekrana.

Kako se dodaje

Klasa za spajanje sa bazom

Odgovori Nova tema
Štampanje RSS teme Kopiraj vezu do teme Podeli na Facebook-u Podeli na X-u (Twitter) Idi na poslednju poruku
  • 7 odgovora
  • 4.059 pregleda

Najnovija ↓

Klasa za spajanje sa bazom 17.07.2011. u 19:48 · pre 186 meseci #1
Pozdrav!

Dali mi neko može reći jel dobra ova klasa?

Code:

<?php

class database
{
    private $link;

    function __construct($host, $user, $password, $base_name, $character_set)
    {
        $this->link = @mysql_connect($host, $user, $password)
            or die($this->error());

        mysql_select_db($base_name, $this->link)
            or die($this->error());

        mysql_set_charset($character_set, $this->link);
    }

    function __destruct()
    {
        mysql_close($this->link);
    }

    private function error()
    {
        return mysql_error($this->link);
    }

    public function query($sql, $return_format = 0)
    {
        $query = mysql_query($sql, $this->link)
            or die($this->error());

        switch($return_format)
        {
            case 1:
                $query = mysql_fetch_row($query);
                return $query;
                break;

            case 2:
                $query = mysql_fetch_array($query);
                return $query;
                break;

            case 3:
                $query = mysql_fetch_row($query);
                $query = $query[0];
                return $query;
                break;

            default:
                return $query;
                break;
        }
    }
}

?>

<?php

$db = new database('localhost', 'root', '', 'test', 'utf8');

$i = $db->query("SELECT `id_user` FROM `users` WHERE `id_user` = 1 ", 2);

echo $i['id_user'];

?>


sve radi kako treba, samo me zanima dali je ovo ispravno oop?

Hvala!
Potpis
www.artwebdizajn.com
Re: Klasa za spajanje sa bazom 18.07.2011. u 00:00 · pre 186 meseci #3
e baš ne kontam nikako ovaj oop, kako god sam kreno praviti klasu za spajanje sa bazom uvijek neko nađe neku manu

dali valja vako?

Code:

<?php
    abstract class Database_Object
    {
        protected static $DB_Name;
        protected static $DB_Open;
        protected static $DB_Conn;

        protected function __construct($database, $hostname, $hostport, $username, $password)
        {
            self::$DB_Name = $database;
            self::$DB_Conn = mysql_connect($hostname . ":" . $hostport, $username, $password);
            if (!self::$DB_Conn) { die('Critical Stop Error: Database Error<br />' . mysql_error()); }
            mysql_select_db(self::$DB_Name, self::$DB_Conn);
        }

        private function __clone() {}

        public function __destruct()
        {
//            mysql_close(self::$DB_Conn);  <-- commented out due to current shared-link close 'feature'.  If left in, causes a warning that this is not a valid link resource.
        }
    }

    final class DB extends Database_Object
    {
        public static function Open($database = DB_NAME, $hostname = DB_HOST, $hostport = DB_PORT, $username = DB_USER, $password = DB_PASS)
        {
            if (!self::$DB_Open)
            {
                self::$DB_Open = new self($database, $hostname, $hostport, $username, $password);
            }
            else
            {
                self::$DB_Open = null;
                self::$DB_Open = new self($database, $hostname, $hostport, $username, $password);
            }
            return self::$DB_Open;
        }

        public function qry($sql, $return_format = 0)
        {
            $query = mysql_query($sql, self::$DB_Conn) OR die(mysql_error());
            switch ($return_format)
            {
                case 1:
                    $query = mysql_fetch_row($query);
                    return $query;
                    break;
                case 2:
                    $query = mysql_fetch_array($query);
                    return $query;
                    break;
                case 3:
                    $query = mysql_fetch_row($query);
                    $query = $query[0];
                    return $query;
                default:
                    return $query;
            }
        }
    }
?>
Potpis
www.artwebdizajn.com
Re: Klasa za spajanje sa bazom 27.09.2011. u 07:41 · pre 183 meseca #4
Nije mi jasno tvoje pitanje da li je dobra ili da li je oop?
Naravno da je oop, cim imas klasu i par metoda.
Pazi ovako: zamisli da je oop prevozno sredstvo.
Bicikl je prevozno sredstvo kao i avion. Ali bicikl je vrlo prost. Avion se krece brze, mozes da spavas u njemu, da gledas tv, rucas i itd.
Ti si napisao bicikl, a za tvoju aplikaciju ti treba nesto bolje i kvalitetnije. Skini sa neta imas dosta dobrih klasa. Pocetnik ne treba da pise klase jer nece napisati dobro sto se vidi i iz tvog slucaja. Pokusavas da glumis nekog sa par godina iskustva. Treba ti vremena. Ako hoces da naucis oop skini dobru klasu pa je analiziraj a ne da pises svoju.

A da li je dobro napisana svakako da nije.
1) To sto nema PDO znaci da nemas neki nivo znanja, tako da mislim da ne postoji savrsena klasa na svetu. Uvek nesto fali i slicno
2) Ovde u tvojoj klasi si pokazao mnogo neznanja planiranju klase i fali ti dosta oop funkcionalnosti, tako da se nadam da ti nece pasti na pamet da je negde koristis vec ti sluzi samo za vezbu.

To sto nisi shvatio oop, razlog lezi u tome sto si citao neke tutorijale i imas velike rupe i stvari su ti nepovezane prilicno. OOP je citav jedan princip, a gornji kod pokazuje kako ni pod razno ne treba programirati. Ovde je problem nacin razmisljanja a ne da li radi ili ne radi.
Re: Klasa za spajanje sa bazom 27.09.2011. u 11:58 · pre 183 meseca #5
Divan komantar, kakav svaki pocetnik voli da dobije :)
Sviđa mi se1
Re: Klasa za spajanje sa bazom 27.09.2011. u 12:04 · pre 183 meseca #6
Poceo sam i ja da pisem komentar na komentar al nisam hteo da budem grub pa sam ugasio browser na pola. Jedino konstruktivno u komentaru je da skine gotovu klasu da gleda, a kako je ovo forum za php pocetnike, zna on sam da ne zna dovoljno pa nema potrebe za tolikim potenciranjem da covek ne zna. To sto je postovao na forum i poceo sam da pravi klasu pokazuje da ima inicijative i zeli da uci, a to je cesto bitnije od iskustva, iskustvo se neminovno stice a inicijativa za ucenjem ne obavezno.

Molim ucesnike diskusije za buduce komentare primerene pocetnickom nivou znanja, ako neko nema sta konstruktivno da napise, neka se ne muci da kuca.
Potpis
http://norway.dakipro.com/
Sviđa mi se2
Re: Klasa za spajanje sa bazom 27.09.2011. u 12:25 · pre 183 meseca #8
Greska je moja zaboravio sam na kom sam delu foruma, tako da kritika na znanje korisnika ne stoji. Tacno.
Glavni savet nije da skine klasu. Glavni savet je da krene drugacije da razmislja u boljem pravcu.

Odgovori
Prijavite se da biste odgovorili u ovoj temi.PrijavaRegistracija

Premeštanje teme