dateigröße fehlerhafte ausgabe

[s.roettger]

hy
ich lasse mir die dateigrößen von einzelnen dateien mit folgendem script ausgeben

$size = filesize("pdf/$file");
$sizes = Array('B', 'KB', 'MB', 'GB', 'TB', 'PB', 'EB');
$ext = $sizes[0];
for ($i=1; (($i < count($sizes)) && ($size >= 1024)); $i++) {
$size = $size / 1024;
$ext  = $sizes[$i];
$sizegerundet = round ($size);
echo "$sizegerundet"; 

im bereich bis 999 kb wird die größe korrekt ausgegeben, danach (also > 1mb) schreibt er für eine datei mit 3 MB nicht 3 MB sondern 3 KB?!

warum?
kann mir jemand helfen?

danke und guten rutsch

[andru]

benutz doch einfach folgende funktion:

function getFilesize($path) {
    if(!file_exists($path)) {
        return false;
    } else {
        $size = filesize($path);
        $return = "";
        
        //Byte
        if($size < 1024) {
              $return = number_format($size, 0, ",", ".")." Bytes";
           }
        //Kilobyte
        elseif($size < 1048576) {
              $return = number_format(($size / 1024), 0, ",", ".")." kB";
           }
        //Megabyte
        elseif($size < 1073741824) {
             $return = number_format(($size / 1048576), 0, ",", ".")." MB";
           }
        //Gigabyte
        elseif($size < 1099511627776) {
               $return = number_format(($size / 1073741824), 0, ",", ".")." GB";
           }
           
        return $return;
    }
} 

johnpatcher

[penizillin]

bei mir klappt die optimierte version ganz gut (obwohl ich nichts über 1 gb testen kann)

$size = filesize('C:\\pdf995\temp.ps');
$sizes = Array('B', 'KB', 'MB', 'GB', 'TB', 'PB', 'EB');
$anzahlKuerzel = count($sizes);
$sizegerundet = 0
$ext = $sizes[0];
for ($i=1; (($i < $anzahlKuerzel) && ($size >= 1024)); $i++) {
    $size = $size / 1024;
    $ext  = $sizes[$i];
    $sizegerundet = round($size);
}
echo $sizegerundet . $ext; 

[TBT]

wieso so umständlich?

$size = filesize('C:\pdf995\temp.ps');
$sizes = Array('B', 'KB', 'MB', 'GB', 'TB', 'PB', 'EB');
$i=1;
while( ( $size = $size>>10 ) >= 1024 )
    ++$i;
echo $size.' '.$sizes[$i]; 

[s.roettger]

@TBT

das läuft super!

@ penizillin:

ich krieg diese Fehlermeldung:

parse error, unexpected T_VARIABLE in /home/www/xxxxxxx/yyyyy/test.php on line 54

line 54: $ext = $sizes[0];



Vielen Dank für Eure Hilfe!!

[jahlives]

...$sizegerundet = 0 

ändere das mal in

... $sizegerundet = 0; 

Gruss

tobi

[Hopka]

Ihr seid alle nicht 1337 genug !!11eins

Original geschrieben von TBT
wieso so umständlich?

Diese Frage gebe ich direkt zurück

$sizes = Array('B', 'KB', 'MB', 'GB');
$size = filesize('pdf/' . $file);

$factor = floor(log($size, 1024));
$newsize = round($size / pow(1024, $factor));

echo $newsize . ' ' . $sizes[$factor]; 

kleine Erklärung:
Logarithmus usw. - naja, Mathe halt
filesize liefert einen (32 Bit) Integer zurück - d.h. größtmögliche Zahl ist 2147483648 (2^31) oder 2 GB. Also werden keine Größeren Zahlen auftauchen, und es werden nur die Größen bis GB gebraucht.
Dadurch wird auch $factor keine Werte annehmen, für die es im Array $sizes keinen Eintrag gibt - da kann also auch noch eine Prüfung mit isset weggelassen werden.

Probleme gibt es allerdings bei leeren Dateien (0 Byte) und bei Dateien größer als 2 GB, denn da liefert filesize unter Umständen negative Werte zurück. Beide Fällen würden zu einem "Division durch 0" Fehler führen, also braucht man am besten noch eine If-Abfrage:

$sizes = Array('B', 'KB', 'MB', 'GB');
$size = filesize('pdf/' . $file);

if ($size > 0) {
    $factor = floor(log($size, 1024));
    $newsize = round($size / pow(1024, $factor));
    
    echo $newsize . ' ' . $sizes[$factor];
} else {
    echo '0 B';
} 

[Payne_of_Death]

Original geschrieben von penizillin
bei mir klappt die optimierte version ganz gut (obwohl ich nichts über 1 gb testen kann)

$size = filesize('C:\\pdf995\temp.ps');
$sizes = Array('B', 'KB', 'MB', 'GB', 'TB', 'PB', 'EB');
$anzahlKuerzel = count($sizes);
$sizegerundet = 0
$ext = $sizes[0];
for ($i=1; (($i < $anzahlKuerzel) && ($size >= 1024)); $i++) {
    $size = $size / 1024;
    $ext  = $sizes[$i];
    $sizegerundet = round($size);
}
echo $sizegerundet . $ext; 

coole Mischung aus englisch/deutsch

[penizillin]

Hopka, zehn punkte!