[PHP] Elemente aus Array löschen

[Luckie]

Ich glaube heute ist nicht mein Tag.

Ich versuche gerade Elemente aus einem Array zu löschen, die in einem anderen Array enthalten sind:

Code:

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

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

for ($j = 0; $j < count($allDirs); $j++)
{   
    for ($i = 0; $i < count($forbiddenDirs); $i++)
    {
        if (strpos($forbiddenDirs[$i], $allDirs[$j]) > 0)
        {
            echo "test";
            $allDirs = array_slice($allDirs, $j, 1);
        }
    }
}

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

Leider wird "test" nie ausgegeben. Also stimmt irgednwas nicht mit strpos. Oder geht da smit PHP auch einfacher?

[DeddyH]

http://www.selfphp.de/funktionsueber...funktionen.php

[jfheins]

Im speziellen könnte dir die Funktion array_diff() weiterhelfen, solange alles Kleingeschrieben ist.

Ansonsten eher array_diff_ukey() ;)

[Angel4585]

http://de.php.net/function.strpos

Wenn ich das recht interpretiere müsste die Bedingung nicht > 0 sondern >= 0 oder !==false sein

Was genau steht denn in allDirs drin? ist vllt das erste element eines strings ([0]) der erste Buchstabe? Dann liefert strpos auch 0 zurück und deine Bedingung zieht nicht

Man beachte das Beispiel auf der verlinkten Seite:

Code:

<?php
$meinString = 'abc';
$findMich   = 'a';
$pos = strpos($meinString, $findMich);

// Der !==-Operator kann ebenfalls verwendet werden. Die Verwendung von != von
// != würde in unserem Beispiel nicht wie erwartet arbeiten, da die Position
// von 'a' 0 ist. Das Statement (0 != false) evaluiert hierbei zu false.
if ($pos !== false) {
    echo "Der String '$findMich' wurde im String '$meinString' gefunden";
    echo " und befindet sich an Position $pos";
} else {
    echo "Der String '$findMich' wurde nicht im String '$meinString' gefunden";
}
?>

[Luckie]

In $forbiddenDirs stehen nur Unterverzeichnisse. Zum Beispiel habe ich den Pfad: /Ablage/Source. In $forbiddenDirs steht jetzt "Source", dann soll das Element aus dem Array gelöscht werden.

Oder ein Pfad beginnt mit /Privat und in $forbiddenDirs steht "Privat", dann sollen alle Elemente in denen Privat vorkommt gelöscht werden.

Deswegen denke ich, muss ich es doch irgendwie über strpos lösen, ichmuss ja gucken, ob der Teilstring im Pfad vorkommt.

Das soll mal eine Sitemap für meine Seite werden, damit ich die Sitemap nicht immer bearbeiten muss, wenn mal was dazu kommt oder gelöscht wird.

Der Inhalt von $allDirs sieht zum Beispiel so aus:

Code:

./Ablage
./Artikel
./Artikel/files
./Ausbildung_FIAE
./Ausbildung_FIAE/Referate
./Ausbildung_FIAE/Source
./Developer
./Developer/C_CPP
./Developer/C_Sharp
./Developer/Delphi
./Developer/Delphi/Code-Snippets
./Developer/Delphi/Demos
./Developer/Delphi/Importe
./Developer/Delphi/Importe/Assarbad
./Developer/Delphi/Importe/Hagen_Reddmann
./Developer/Delphi/Importe/Hagen_Reddmann/Archiv
./Developer/Delphi/Importe/Motzi
./Developer/Delphi/Importe/Nico
./Developer/Delphi/Importe/toms
./Developer/Delphi/Programme
./Developer/Delphi/Tutorials
./Developer/Delphi/Units
./Developer/Flex
./Developer/Java
./Developer/Kurzreferenzen
./Developer/Kurzreferenzen/Source
./Developer/PHP
./Developer/VBS
./Developer/~MyLicenses
./Latex
./Privat
./Privat/AdressDB
./Privat/Quellcodes
./Privat/Software

Das Array $forbiddenDirs entält die Elemente "Privat" und "Source". Jetzt ich alle Pfade aus $allDirs löschen in denen "Privat" oder "Source" vorkommt. Und ich dachte, dass dies mein Code tun würde. Tut er aber nicht.

[jfheins]

Probiers mal damit:

Code:

<?php

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

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

for ($j = count($allDirs) - 1; $j >= 0; $j++) // Fehler 1 - wie war das mit Schleifen, die löschen?
{    
    for ($i = 0; $i < count($forbiddenDirs); $i++) 
    { 
        if (strpos($allDirs[$j], $forbiddenDirs[$i]) !== false) // Fehler 2 - haystack, needle und mit typsicherem vergleich prüfen
        { 
            echo "test"; 
            unset($allDirs[j]); // Element löschen (und nicht Teilarray zurückgeben)
        } 
    } 
} 

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

?>

[Luckie]

Nein keine Änderung. "test" wird jetzt zwar ausgegeben, aber ich habe immer noch alle Pfade im Array.

[jfheins]

Komisch ... dann führ mal das hier aus, zum Debuggen ;)

Code:

<?php

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

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

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 "test"; 
            unset($allDirs[j]); 
            break; // Fehlte, sollte aber nicht so schlimm sein
        } 
    } 
} 

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

?>

[Luckie]

OK:

Code:

Haystack: ./luckie-online - Needle: Privat - Vorhanden:
Haystack: ./luckie-online - Needle: Source - Vorhanden:
Haystack: ./luckie-online - Needle: cgi-bin - Vorhanden:
Haystack: ./luckie-online - Needle: data - Vorhanden:
Haystack: ./luckie-online - Needle: luckie-online - Vorhanden: 1

./luckie-online ist aber immer noch im Array enthalten.

PS: Korrigier mal deine for-Schleife. Das muss $j-- heißen, sonst hast du eine Endlosschleife. Ich habe das jetzt schon zweimal mitkopiert.

[xZise]

Zitat:

Zitat von jfheins

Code:

[...]
for ($j = count($allDirs) - 1; $j >= 0; $j++) // Fehler 1 - wie war das mit Schleifen, die löschen?
{    [....]

Wie war das mit Schleifen die löschen?

Delphi-Quellcode:

for i := Max - 1 downto 0 do
begin

end;

Du machst da aber:

Delphi-Quellcode:

for i := Max - 1 to 0 do
begin

end;

Und jetzt nochmal richtig:

Code:

for ($j = count($allDirs) - 1; $j >= 0; $j--)

Das dürfte eher funktionieren!

[edit]Wo war die Redbox :D [/edit]

MfG
xZise

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