- Házimozi belépő szinten
- Milyen TV-t vegyek?
- AMD Ryzen 9 / 7 / 5 / 3 5***(X) "Zen 3" (AM4)
- Milyen billentyűzetet vegyek?
- Fejhallgató erősítő és DAC topik
- NVIDIA GeForce RTX 3080 / 3090 / Ti (GA102)
- AMD K6-III, és minden ami RETRO - Oldschool tuning
- Apple asztali gépek
- TCL LCD és LED TV-k
- Milyen alaplapot vegyek?
Hirdetés
-
Olcsóbb lett a OnePlus Watch 2
ma Csak arra kell figyelni, Watch X néven vásároljuk meg az Oppo kínálatából.
-
Ellopták a Tesla akkumulátor-titkait
it Beperelte egy korábbi beszállítóját a Tesla, és azzal vádolja, hogy üzleti titkokat lopott a Tesla akkumulátorgyártási technológiájával kapcsolatban.
-
Switch-re is befut a Darkest Dungeon 2
gp A PlayStation kiadások mellett a Nintendo konzoljára is elérhető lesz jövő hónapban az új kiadás.
Új hozzászólás Aktív témák
-
mallee
tag
válasz
DNReNTi #15412 üzenetére
Gondolatébresztők:
Database_Connection
private static $DB_Host = ........: Ennek a helye egy config fájlban lenne és valamilyen okosság mentén kéne beadni az osztályodnak.
error_reporting(0);: Miért kezel az adatbázis osztályod error reportolást? Mi köze van a kettőnek egymáshoz? (azon túl, hogy az adatbázis-kapcsolat felépítése esetén is jöhet hiba). Ennek inkább egy inicializáló, környezetet beállító, stb php-ben kellene lennie
echo 'Database connection failed. Code : ' . $DB_Connect->....; Ha csak kiírsz egy üzenetet a képernyőre, attól az alkalmazásod még fut tovább, noha ő arra számított, hogy lesz adatbázis-hozzáférése. Ehelyett használj exception-t
Mi volt ezzel az osztállyal a célod? Hány helyen és hol hívod meg?
Database
Hát ez így nagyon nem jó. Több kisebb osztályba kéne szétvágni az executeSQL-t az SQL parancs típusok alapján, pl Database_Select_SQL valósítaná meg a selectes logikákat, Database_Insert_SQL az inserteset, stb. Ezzel elkerülhetnéd azt a csúnya és nehezen értelmezhető switch-case szerkezetet. Egyébként bár látom, hogy mi akar az osztály célja lenni, mégis nagyon rosszul olvasható a kód. A sok egymásba ágyazott if-else throw exception megoldás helyett inkább azt kéne vizsgálnod a feltételben, hogy sikertelen volt-e a végrehajtás: ha így van, akkor exception, egyébként fusson tovább a kód, pl:
$stmt = $this->_DB_Connect->prepare($SQL_command);
if ($stmt === false) { throw new Exception("blablabla"); }
$parameter_type_list = '';
foreach($SQL_parameters as $parameter) {
és nincs utána else!Ez sem tökéletes megoldás, de átláthatóbb a kevesebb egymásba ágyazott szerkezet miatt.
[ Szerkesztve ]
Új hozzászólás Aktív témák
- AMD Gamer PC / AMD Radeon RX 7900 XTX / AMD Ryzen 7 7800 X3D
- iphone 14 Pro Max 128gb, független, makulátlan, 100% akku
- iphone 15 Pro Max 256gb Natur Titanium,független,bontatlan
- ÚJ Bontatlan APPLE MACBOOK AIR 13 - M2 (MLY33MG/A) ÉJFEKETE - 8GB/256GB SSD- Magyar - 3 év Garancia
- Corsair RM850x Gold Moduláris Tápegység 850W