Guten Tag,
ich möchte eine eigene Klasse schreiben die, die $wpdb Variable enthält und zwei Methoden davon Kapselt. Die Klasse sieht wie folgt aus:
PHPclass Helper { private $wpdb = false; public function __construct() { global $wpdb; if (is_object($wpdb)) { $this->wpdb = $wpdb; } } public function get_results($data) { return $this->wpdb->get_results($data); } public function get_prefix() { return $this->wpdb->prefix; } }
Der Code ist sinnvoll, aber nicht optimal. Hier sind einige Verbesserungsvorschläge:
- Verwende keine globalen Variablen, sondern injiziere die Abhängigkeiten in den Konstruktor. Das macht deinen Code testbarer und modularer.
- Verwende Typdeklarationen für die Parameter und Rückgabewerte deiner Funktionen. Das macht deinen Code sicherer und lesbarer.
- Verwende aussagekräftige Namen für deine Variablen und Funktionen. Das macht deinen Code verständlicher und wartbarer.
PHP
<?php
class DatabaseHelper {
// Deklariere die Eigenschaft $wpdb als private und vom Typ wpdb
private wpdb $wpdb;
// Injiziere die Instanz von wpdb in den Konstruktor
public function __construct(wpdb $wpdb) {
$this->wpdb = $wpdb;
}
// Gib den Rückgabetyp der Funktion an
public function getResults(string $data): array {
return $this->wpdb->get_results($data);
}
// Gib den Rückgabetyp der Funktion an
public function getPrefix(): string {
return $this->wpdb->prefix;
}
}
