Új hozzászólás Aktív témák

• #6539 Sk8erPeter nagyúr DiabloCorsa #6538

  Új Válasz 2013-01-29 15:11:53 #6539

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  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ó.

Új hozzászólás Aktív témák