[PHP MySQL] User Control Panel CMS + Github

  • Beitrag von c0nc1l ()

    Dieser Beitrag wurde vom Autor gelöscht ().

  • Beitrag von Fabian107 ()

    Dieser Beitrag wurde gelöscht, Informationen über den Löschvorgang sind nicht verfügbar.

    Wenn du schon mit Template Dateien arbeitest, warum greifst du dann nicht zu einer Template Engine?
    Ebenso ist es Sinnlos für jeden Wert den du brauchst eine neue Funktion zu schreiben, ich verweise mal auf das:

    PHP
    public function getUserMoney() {
			$username = $_SESSION['@username'];
			$username = $this->mysqli->real_escape_string($_SESSION['@username']);
			$result = $this->mysqli->query("SELECT * FROM $this->userDB WHERE Username='$username'");
			while ($row = $result->fetch_assoc()) {
				return $row['Money'];
			}
		}


    Erstelle dir einfach eine Funktion mit den benötigten Parametern, welche du dann zurück gibst.
    Oder "getUserCount" dabei reicht auch eine Funkion mit den Parametern für die Tabelle die dir die Anzahl der Datensätze zurück gibt.


    Alles in Allem, wenn du so mit dem Projekt weiterarbeiten solltest wirst du früher oder Später Unzählige Zeilen von Code haben wo die Hälfte die selben aufgaben hat und es einfach nur noch verwirrend wird.

    Ja das ist mir auch schon aufgefallen, bin ja bereits dran etwas "aufzuräumen" da ich ja wie erwähnt es vor 5-6 Monaten mal angefangen habe.
    Ich bin leider noch kein Profi und bin gerade noch dabei es weiter zu aneignen.


    Ich könnte es auch gleich in der user.class alles in eine variable speichern, so brauch ich keine Funktion mehr sondern kann direkt drauf zu greifen.


    Mit diesem Beitrag erhoffe ich mir eher ein paar Leute die da aktiv oder von mir aus auch inaktiv dran arbeiten


    - Danke für dein Feedback