-
IT café
Új hozzászólás Aktív témák
-
Sk8erPeter
nagyúr
válasz DiabloCorsa #6538 üzenetére
Na, fasza, hogy megosztottad.
Látod, így már legalább tudunk egyből rajta javítani, meg tudjuk beszélni.Például:
if (strpos($file, '.html',1)||strpos($file, '.html',1) )
mi értelme kétszer ugyanazt a feltételt szabni? Ez olyan, mintha azt mondanád, hogy "ha esik a hó vagy ha esik a hó" - kétszer is ugyanazt mondtad.
Még egy:$dir = opendir (dirname(__FILE__));
while (false !== ($file = readdir($dir))) {
if (strpos($file, '.html',1)||strpos($file, '.html',1) ) {
// ..........
}
}ennél van egy rövidebb, könnyebben olvasható, kevésbé macerás módszer is, mégpedig a glob() függvény használatával, épp a hivatalos oldalra felrakott példát minimálisan átalakítva:
foreach (glob('*.html') as $file) {
// ....
}Nyilván a pontok helyére megy a többi. Így ez is az aktuális könyvtárban fog keresni (a glob is bővíthető egyéb könyvtárral is, lásd a kommentben a példákat).
Persze jó, amit mutattál, csak mondom, hogy van alternatíva, ami talán könnyebben olvasható és rövidebb, plusz egyből csak a .html-fájlokra szűr.Ja, még valami. Nem ártana bele hibaellenőrzés. Erről beszéltem korábban, hogy nem az eredeti fájlokon kéne mókolni. Például ha nem biztos, hogy van egyáltalán találat a reguláris kifejezésre, akkor a $matches[1] nem lesz jó.
[ Szerkesztve ]
Sk8erPeter
Új hozzászólás Aktív témák
● olvasd el a téma összefoglalót!
- ASUS RTX 3060 Ti 8GB GDDR6 TUF GAMING OC Eladó! 95.000.-
- 16" Apple Macbook Pro Retina 2019 Touch Bar + Toch ID eladó! Intel Core i7, 16GB RAM, 512GB SSD!
- Hamilton Aviation Khaki X Patrol Auto Chrono ETA 7750 automata óra! Full set!
- IPhone 12 Pro 128GB gyári független 2026. 01.18. Telekom garancia akku 100%
- HUAWEI Band 8 aktivitásmérő (fekete), keveset használt, karcmentes kijelző