[Lernen] Bank of Habbo

Empfehle dir:

-Nervige Konto Nummern zu entfernen. Mache das lieber mit Benutzernamen.

-Hau keine unnötigen Funktionen rein...

Interessant, viel Glück

Snippets?

Zitat von Zytrox

Kann bis jetzt nur Snippets von der Config.php zeigen:

PHP

<?php
    
class config {
        //All URL do not end with an "/"

        PUBLIC STATIC $hotel_url = 'http://habbo.de'; //Link to the Hotel    
        PUBLIC STATIC $cms_url = 'http://localhost'; //Link to the Bank    
        public STATIC $cms_name = 'Bank of Habbo'; //Bank's name
        public STATIC $cms_description = 'Die Bank deines Vertrauens'; //Bank's description
        public STATIC $cms_maintenance = false; //false = Bank is not in maintenance, true = Bank is in maintenance
        public STATIC $cms_safemode = false; //false = Bank is not in safe mode, true = Bank is in safe mode
        public STATIC $cms_max_reg = 3; //The maximum of accounts
}

?>

Alles anzeigen

Halt dich am besten von Anfang an, an die Konventionen von PHP.

Die findest du hier:

http://www.php-fig.org/psr/

Außerdem würde ich solche sachen wie max. accounts pro IP, Wartungsarbeiten,.. in die Datenbank packen und nicht in die Config Klasse.

Oder tue alles in die Datenbank und hol dir das über die Klasse.

EDIT:

Dokumentiere deinen Code

Bin auf jeden Fall gespannt, viel Glück !

Zitat von Zytrox

Kann bis jetzt nur Snippets von der Config.php zeigen:

PHP

<?php
    
class config {
        //All URL do not end with an "/"

        PUBLIC STATIC $hotel_url = 'http://habbo.de'; //Link to the Hotel    
        PUBLIC STATIC $cms_url = 'http://localhost'; //Link to the Bank    
        public STATIC $cms_name = 'Bank of Habbo'; //Bank's name
        public STATIC $cms_description = 'Die Bank deines Vertrauens'; //Bank's description
        public STATIC $cms_maintenance = false; //false = Bank is not in maintenance, true = Bank is in maintenance
        public STATIC $cms_safemode = false; //false = Bank is not in safe mode, true = Bank is in safe mode
        public STATIC $cms_max_reg = 3; //The maximum of accounts
}

?>

Alles anzeigen

Ich persönlich würde nicht so viele unnötige Kommentare schreiben. Das sieht man ja auf den 1.Blick.

Wünsche dir viel Spaß dabei. Learning by doing!

Kleiner Tipp am Rande: Mach die Konfigurationselemente zu Konstanten, da man verhindern möchte, dass diese im Verlauf der Anwendung überschrieben werden.

Ansonsten einfach mal die Standards anschauen, wie imperare schon geschrieben hat.

Ich wünsche dir viel Glück.

Bin gespannt was daraus wird. Das Wichtigste: zieh es durch!

Updates?

Weitere Snippets?

Hättest du vielleicht sonstige Bilder?

Interessiert mich.

MfG

Wie hast du das zurzeit?

Würde dir PDO empfehlen. mysql_* ist outdated.

Zitat von Tafelglotzer

Würde dir PDO empfehlen. mysql_* ist outdated.

Und nicht Objektorientiert, was der Aussage vom OP ja widersprechen würde.

Zitat von Zytrox

- PHP (Hier möchte ich in OOP programmieren. Dies habe ich noch nicht gross gemacht und möchte mich nun daran wagen) (PHP 7)

Zitat von Tafelglotzer

Würde dir PDO empfehlen. mysql_* ist outdated.

Wieso denn PDO und nicht mysqli? Er weiß dass er MySQL als DBMS nutzen wird.

Das wäre jetzt mein Vorschlag

<?php
/**
 *
 * Blablabla Dokumentation nicht vergessen
 *
 */
final class Database
{

    private static $conn = null,$init = false;

    public static function init()
    {
        if($this->init) return;
        $this->conn = new mysqli("host","name","pass","db");
        $this->init = true;
    }

    public static function getConnection() : mysqli
    {
        if(!$this->init)
        {
            self::init();
        }
        return $conn;
    }

    public function __destruct()
    {
        // Verbindung schließen, init auf false setzen
    }

}

?>

Zitat von imperare

Wieso denn PDO und nicht mysqli? Er weiß dass er MySQL als DBMS nutzen wird.

Das wäre jetzt mein Vorschlag

PHP

<?php
/**
 *
 * Blablabla Dokumentation nicht vergessen
 *
 */
final class Database
{

    private static $conn = null,$init = false;

    public static function init()
    {
        if($this->init) return;
        $this->conn = new mysqli("host","name","pass","db");
        $this->init = true;
    }

    public static function getConnection() : mysqli
    {
        if(!$this->init)
        {
            self::init();
        }
        return $conn;
    }

    public function __destruct()
    {
        // Verbindung schließen, init auf false setzen
    }

}

?>

Alles anzeigen

Singletons gehören zu den Antipattern. https://www.michaelsafyan.com/tech/design/patterns/singleton

+ dein Code ist fehlerhaft bis zum Tode.