[PHP] Fehler in Script (Dateien zählen mit aktuellem Datum)

[samuelshp]

Bei folgendem Script liegt irgendwo ein Feheler. Ich weiß aber nicht wo. Normalerweise sollten alle Dateien gezählt werden, deren Erstellungsadatum mit dem aktuellem Datum übereinstimmt. Es wird immer 1 zurückgegeben. Vielleicht könnt ihr mir helfen.

PHP-Code:

function countJpgRecursiv( $pfad = null )
{
  if( is_null( $pfad ) ) $pfad = dirname( __FILE__ );

  $result = null;

  if( !file_exists( $pfad ) )
    die( $pfad . " existiert nicht" );

  $opendir = opendir( $pfad );

  while( $file = readdir( $opendir ) )
    if( $file != '.' && $file != '..' ):

      $file = $pfad. "/" . $file;

      if( is_Dir( $file ) )
      {
          $tmp = countJpgRecursiv( $file );
          if( is_null( $result ) || date("d.m.Y",filemtime($file))==date("d.m.Y", $datum) )
            $result += $tmp;

      }else{
          if( substr(strtolower( $file ), -4) == ".jpg" )
            if( is_null( $result ) || date("d.m.Y",filemtime($file))==date("d.m.Y", $datum) )
              $result++;
      }
    endif;
  closedir( $opendir );
  return $result;
} 


[steffenk]

Der Haken liegt an $result.
Was nutzt z.B. die Rekursion, wenn $result dann wieder NULL gesetzt wird.

so funktioniert es:

PHP-Code:

$result=0;
echo countJpgRecursiv("/images");
function countJpgRecursiv( $pfad = null ) 
{ 
  global $result;
  if( is_null( $pfad ) ) $pfad = dirname( __FILE__ ); 

  if( !file_exists( $pfad ) ) 
    die( $pfad . " existiert nicht" ); 

  $opendir = opendir( $pfad ); 

  while( $file = readdir( $opendir ) ) 
    if( $file != '.' && $file != '..' ): 

      $file = $pfad. "/" . $file; 

      if( is_Dir( $file ) ) 
      { 
          $tmp = countJpgRecursiv( $file ); 
          if( date("d.m.Y",filemtime($file))==date("d.m.Y", $datum) ) 
            $result += $tmp; 

      }else{ 
          if( substr(strtolower( $file ), -4) == ".jpg" ) 
              echo "$file - ".date("d.m.Y",filemtime($file))."<br>";
            if( date("d.m.Y",filemtime($file))==date("d.m.Y", $datum) ) 
              $result++; 
      } 
    endif; 
  closedir( $opendir ); 
  return $result; 
} 


[samuelshp]

nee,das geht noch nicht.

erstmal was soll das da am Anfang, vor der Funktion?

2. Es werden jetzt alle Dateien ausgegeben mit dem Datum aufgrund des Echos das du eingefügt hast, aber zählen tut das Script nicht.

[steffenk]

bei mir geht das schon.
Der Anfang - da wird die Variabel $result initialisiert.
Das echo ist nur aus Debuggründen drin, das kannst Du auch löschen.
Da $result auch in der Rekursion bekannt ist, wird immer weiter hochgezählt - ich bekomme das korrekte Ergebnis geliefert.

[samuelshp]

Es tut mir wirklich Leid, dass sagen zu müssen, aber dieser Script funktioniert wirklich nicht. Ich hab den Script jetzt um sicher zu gehen aus dem Gesamtscript herausgenommen. Doch es wird eine unsinnige ausgegeben. Ich hab jetzt eigentlich alles am Code verstanden. Und hab das mit Debug gelassen um per Hand zu zählen wieviel Dateien es sind. Es sind 7, die auch in der Liste richtig angezeigt (mit datum) werden, aber unter der Liste steht 28. Hilfe!

[steffenk]

Wenn da 0 steht, dann liegt das daran, dass filemtime einen anderen Wert als "heute" liefert.
$datum ist bei Dir auch nicht angegeben.

Also:
1. nimm das $datum raus oder definiere es
2. benutze z.B. fileatime

[samuelshp]

Ich habs mal etwas anders gemacht, so gehts bei mir.

PHP-Code:

function countJpgRecursiv($path)
{
$result = 0;
$handle = opendir($path);
if ($handle)
{
while (false !== ($file = readdir($handle)))
{
if ($file != "." && $file != "..")
{
$name = $path . "/" . $file;
if (is_dir($name))
{
$result += countJpgRecursiv($name);
}
else
{
if (substr(strtolower($name), -4) == ".jpg" && date("d.m.Y",filemtime($name))==date("d.m.Y", time()))
$result++;
}
}
}
}
closedir($handle);
return $result;
}

echo countJPGRecursiv("../wallpaper"); 


[steffenk]

aha, also das undefinierte $datum

übrigens, date("d.m.Y", time()) geht auch einfacher mit date("d.m.Y")

[samuelshp]

Danke, für die Hilfe die ihr mir gegeben habt, aber das mit der Variable $datum war nur ein Fehler. Es ging trotzdem nicht.