Kann man das einfacher schreiben?

[RamonaS]

Hallo, mit diesem scriptteil wird die größe eine datei berechnet.
Ich finde das ist ein bischen viel code.

Kann man das einfacher schreiben oder gibts in php5 sowas schon als fertige funktion?


<pre>
# Dateigröße ermitteln
$d1="datei.txt";
$a=0;
$ext=array(" Byte"," KByte"," MByte"," GByte"," TByte");
$size=filesize($d1);
while($size>=pow(1024,$a)) {$a++;}
$size=round($size/pow(1024,$a-1)*100)/100 .@$ext[$a-1];
</pre>

[ArSeN]

Die Dateigröße kannst du nicht berechnen sondern nur auslesen. Das wird ja in deinem Beispiel schon mit filesize() gemacht. Die Funktion gibt die Größe in Bytes wieder ... wie du das dann am Ende anzeigst hängt ja ganz von dir ab. Natürlich kann man deinen Vorschlag noch zusammenfassen, aber das bekommst du sicherlich auch selber hin.

[PHP-Desaster]

Eine Optimierung wäre schon mal, auf die pow-Funktion zu verzichten.

[RamonaS]

>aber das bekommst du sicherlich auch selber hin.
Hmm , dann hätte ich doch bestimmt nicht gefragt, oda?

Ok also wenn mir jemand einen einfacheren weg weiß, bitte melden

[ArSeN]

Ich glaub du hast nicht verstanden was ich meine. Der Quelltext der da steht, den verstehst du ja wohl, oder nicht? Dann wirst du doch wohl auch wissen wie man das gleiche "kürzer" schreiben kann. Um zum gleichen Ergebnis zu kommen musst du das wohl so machen.

Andernfalls erklär mal was du unter "einfacher" verstehst.

[fireweasel]

Re: Kann man das einfacher schreiben?

Ich finde das ist ein bischen viel code.

Kann man das einfacher schreiben oder gibts in php5 sowas schon als fertige funktion?

Als fertige Funktion gibts sowas nicht, aber im PHP-Handbuch findest du sicher eine Beispielfunktion, die das leistet, wie z.B. die hier oder auch die hier.[1]


... oder eben ohne den pow()- und log()-Kram:

/// get "formatted" filesize
/// return STR
function filesize_formatted(
  $filename = '' /// STR filename
) {
  $filesize = @filesize($filename);
  if (FALSE === $filesize) {
    return 'file not found';
  }

  $mult = 1024;
  $units = array ('B', 'KiB', 'MiB', 'GiB', 'TiB');
  //$mult = 1000;
  //$units = array ('B', 'kB', 'MB', 'GB', 'TB');

  $unit = 0;
  while ($filesize > $mult) {
    $filesize = $filesize / $mult;
    ++$unit;
  }
  if (!isset($units[$unit])) {
    return 'given filesize is too big';
  }
  return sprintf('%d %s', round($filesize), $units[$unit]);
} 

Ist zwar nicht kürzer, aber IMHO etwas übersichtlicher ...

Die Tera-/Tibi-Byte-Einträge sind auch reine Augenwischerei. Ich glaube filesize() kann so große Werte gar nicht brauchbar wiedergeben -- zumindest wenn der INTEGER-Datentyp nur 32 Bit groß ist.


So richtig verstehe ich den Nutzen einer solchen Funktion zwar nicht, aber die Leute wollen anscheinend sowas haben.

--
[1] Beide hab ich gefunden, indem ich unter Beachtung der Groß-/Kleinschreibung nach "MB" auf der Seite gesucht hatte -- das hättest du bestimmt auch hinbekommen, oder?

[ModestLife]

Wieso auf den "log()-Kram" verzichten? Geht doch kurz und übersichtlich.

Anstatt

$unit = 0;
while ($filesize > $mult) {
    $filesize = $filesize / $mult;
    $unit++;
} 

einfach

$unit = floor(log($filesize, $mult)) 

[RamonaS]

Ok ich hab natürlich gegoogeld....und das schönste script das ich aufgetrieben habe war das hier:


function fSize($URL)
{$gr=filesize($URL);
if($gr<1000) {return number_format($gr,0,",",".")." Bytes";}
elseif($gr<1000000) {return number_format($gr/1024,2,",",".")." kB";}
else {return number_format($gr/1048576,2,",",".")." MB";}
}

das kann ich technisch noch nachvollziehen :-)

Danke an alle