• Hallo liebe Userinnen und User,

    nach bereits längeren Planungen und Vorbereitungen sind wir nun von vBulletin auf Xenforo umgestiegen. Die Umstellung musste leider aufgrund der Serverprobleme der letzten Tage notgedrungen vorverlegt werden. Das neue Forum ist soweit voll funktionsfähig, allerdings sind noch nicht alle der gewohnten Funktionen vorhanden. Nach Möglichkeit werden wir sie in den nächsten Wochen nachrüsten. Dafür sollte es nun einige der Probleme lösen, die wir in den letzten Tagen, Wochen und Monaten hatten. Auch der Server ist nun potenter als bei unserem alten Hoster, wodurch wir nun langfristig den Tank mit Bytes vollgetankt haben.

    Anfangs mag die neue Boardsoftware etwas ungewohnt sein, aber man findet sich recht schnell ein. Wir wissen, dass ihr alle Gewohnheitstiere seid, aber gebt dem neuen Board eine Chance.
    Sollte etwas der neuen oder auch gewohnten Funktionen unklar sein, könnt ihr den "Wo issn da der Button zu"-Thread im Feedback nutzen. Bugs meldet ihr bitte im Bugtracker, es wird sicher welche geben die uns noch nicht aufgefallen sind. Ich werde das dann versuchen, halbwegs im Startbeitrag übersichtlich zu halten, was an Arbeit noch aussteht.

    Neu ist, dass die Boardsoftware deutlich besser für Mobiltelefone und diverse Endgeräte geeignet ist und nun auch im mobilen Style alle Funktionen verfügbar sind. Am Desktop findet ihr oben rechts sowohl den Umschalter zwischen hellem und dunklem Style. Am Handy ist der Hell-/Dunkelschalter am Ende der Seite. Damit sollte zukünftig jeder sein Board so konfigurieren können, wie es ihm am liebsten ist.


    Die restlichen Funktionen sollten eigentlich soweit wie gewohnt funktionieren. Einfach mal ein wenig damit spielen oder bei Unklarheiten im Thread nachfragen. Viel Spaß im ngb 2.0.

[PHP] Anzahl Datensätze ermitteln PDO select where $_POST

Cyperfriend

Der ohne Avatar

Registriert
14 Juli 2013
Beiträge
1.123
Nachdem mir von eure Seite PDO sehr ans Herz gelegt wurde, wollte ich das bei einem Projekt auch entsprechend anwenden. Leider stolpere ich gleich bei der ersten Abfrage, bei der es darum geht die Anzahl Datensätze zu ermitteln, bzw. zu Prüfen, ob ein Datensatz bereits vorhanden ist.
Hier der Code:
[src=php]
# Diese Datei enthält Funktionen, welche für das Script zwingend benötigt werden.
require_once("../../functions.inc.php");
# Diese Datei enthält die Fehlercodes.
require_once("../../codes.php");

# Datenfankverbindung herstellen
$db_connect = db_connect();

[...]

# Prüfen, ob der Benutzername bereits angelegt wurde
$db_read = $db_connect->query('select count * from db_benutzer where benutzername='.$_POST['Benutzername']);
$anzahl = $db_read->fetchColumn();
# Falls ein Eintrag gefunden wurde, Vorgang abbrechen
if($anzahl >= 1){
# Code 21 = "Diesen Benutzer gibt es bereits"
$_SESSION['code'] = 21;
header("Location: http://".$_SERVER['HTTP_HOST']."/settings/users/show.php"); exit;
}[/src]

Der Code soll laut RapidPHP in Ordnung sein, also keine Syntaxfehler. Trotzdem gibt es beim Aufruf dann folgenden Fehler:
Fatal error: Uncaught Error: Call to a member function query() on null in [...]
Der Fehler taucht in der Zeile $db_read = $db_connect auf.

[Edit] Hier noch die Verbindung zur Datenbank:
[src=php]# Datenbankinformationen
function db_connect(){
try {
$db = new PDO('mysql:host=*****;dbname=*****;charset=UTF8','*****','*****');
}
catch(PDOException $e){
echo 'Fehler bei der Datenbankverbindung' . $e->getMessage();
}
}[/src][/Edit]

Jemand einen Vorschlag, wie's richtig ist?
 
Zuletzt bearbeitet:

Rakorium-M

NGBler

Registriert
14 Juli 2013
Beiträge
413
1. Da fehlt ein [kw]return $db;[/kw] in der Funktion [kw]db_connect[/kw].
2. Du hast da eine SQL Injection Schwachstelle gebaut (in Zeile 13). Korrekt wäre (statt Zeile 13):
[src=php]$db_read = $db_connect->prepare('select benutzername from db_benutzer where benutzername=?');
$db_read->execute([$_POST['Benutzername']]);[/src]
 
Oben