[PHP] Elemente aus Array löschen

[jfheins]

Ups, naja - kann schonmal passieren wenn man im Browser programmiert :angel:
(Wir brauchen eine Sourcecode-Semantik-Prüfung in der DP :mrgreen: )

Okay, da das aus irgendwelchen Gründen nicht geht :gruebel: : Mach smal andersrum und kopiere alle Werte in ein neues Array :coder2:

Code:

<?php 

$allDirs = (ScanDirectories(".")); 
$forbiddenDirs = array("Privat", "Source", "cgi-bin", "data", "luckie-online"); 

$allowed = array();
$flag = true;

for ($j = count($allDirs) - 1; $j >= 0; $j--) 
{    
    for ($i = 0; $i < count($forbiddenDirs); $i++) 
    { 
echo 'Haystack: ' . $allDirs[$j] . '  -  Needle: ' . $forbiddenDirs[$i] . '  -  ' . 'Vorhanden: ' . (strpos($allDirs[$j], $forbiddenDirs[$i]) !== false) . '
'; 

        if (strpos($allDirs[$j], $forbiddenDirs[$i]) !== false) 
        { 
            echo '"' . $forbiddenDirs[$i] . '" ist in "' . $allDirs[$j] . '" vorhanden.'; 
            $flag = false;
            break; 
        } 
    } 
    if ($flag)
      $allowed[] = $allDirs[$j];

    $flag = true;
} 

foreach ($allowed as $value) 
{ 
    echo $value."
"; 
} 

?>

[Valle]

Und als Performance-Freak schlage ich vor, das ganze komplett anders zu machen und die verbotenen Verzeichnisse erst gar nicht in das Array aufzunehmen. D.h. ScanDirectories weglassen und mit readdir arbeiten.

Code:

// Wichtig: Spezial-Einträge "." und ".." hinzufügen.
$forbiddenDirs = array(".", "..", "Privat", "Source", "cgi-bin", "data", "luckie-online"); 

if ($handle = opendir('/path/to/files')) {
    echo "Directory handle: $handle\n";
    echo "Files:\n";

    while (false !== ($file = readdir($handle))) {
        if (in_array($file, $forbiddenDirs))
        echo "$file\n";
    }
    closedir($handle);
}

Quelle

Wenn du Rekursion brauchst, dann bastle eine Klasse oder Funktion mit dem gleichen Verfahren und gebe als zweiten Parameter das Array mit. ;-)

Mit freundlichen Grüßen,

Valle

[Luckie]

@jfheins: So funktioniert es. Prima, danke.

@Valentin: ScanDirectories sieht so aus:

Code:

function ScanDirectories($dir, $allData=array() ) {
$dirContent = glob($dir.'/*', GLOB_ONLYDIR);
      if ( is_array($dirContent) ) {

               foreach($dirContent as $content) {
                      $allData[] = $content;
                      $allData = ScanDirectories($content, $allData);
                  }
      }
      return $allData;
}

Und so sieht es jetzt aus: http://www.michael-puff.de/sitemap.php

[Valle]

Code:

$forbiddenDirs = array("Privat", "Source", "cgi-bin", "data", "luckie-online"); 

function ScanDirectories($dir, array $forbiddenDirs=array(), array $allData=array())
{
    if ($handle = opendir($dir)) {
        while (false !== ($file = readdir($handle))) {
            if ($file == "." || $file == ".." || in_array($file, $forbiddenDirs)) {
                continue;
            }
            if (is_dir($dir . DIRECTORY_SEPARATOR . $file)) {
                $allData[] = $dir . DIRECTORY_SEPARATOR . $file;
                $allData = ScanDirectories(
                    $dir . DIRECTORY_SEPARATOR . $file,
                    $forbiddenDirs,
                    $allData
                );
            }
        }
        return $allData;
    } else {
        throw new Exception('Failed to open directory: ' . $dir);
    }
}

Ungetestet. :-? Jetzt getestet und verbessert. :-)

Mit freundlichen Grüßen,

Valle

[Luckie]

Code:

$forbiddenDirs = array("Privat", "Source", "cgi-bin", "data", "luckie-online"); 

function ScanDirectories($dir, array $forbiddenDirs=array(), array $allData=array())
{
    if ($handle = opendir($dir)) {
        while (false !== ($file = readdir($handle))) {
            if ($file == "." || $file == ".." || in_array($file, $forbiddenDirs)) {
                continue;
            }
            if (is_dir($dir . DIRECTORY_SEPERATOR . $file)) {
                $allData[] = $dir . DIRECTORY_SEPERATOR . $file;
                $allData = ScanDirectories(
                    $dir . DIRECTORY_SEPERATOR . $file,
                    $forbiddenDirs,
                    $allData
                );
            }
        }
        return $allData;
    } else {
        throw new Exception('Failed to open directory: ' . $dir);
    }
}

$allDirs = (ScanDirectories("."));

print_r($allDirs);

Liefert als Ausgabe:

Zitat:

Array ( )

;)

[Valle]

Zitat:

Zitat von Luckie

Liefert als Ausgabe:

Zitat:

Array ( )

;)

Ist doch super, dann gibt es wenigstens keinen Parsing-error. ;-)
Ersetze DIRECTORY_SEPERATOR durch DIRECTORY_SEPARATOR, dann geht's. ;-)

Mit freundlichen Grüßen,

Valle

[Luckie]

Jupp, funktioniert. Leider werden alle verzeichnisse ausgegeben, also auch die nicht erwünschten. Ist auch logisch, weil wenn eine Zeichenkette innerhalb des Pfades gesucht wird, funktioniert da smit in_array() natürlich nicht.