Wiele return w funkcji a zwraca tylko jeden, czyli jak to poprawić aby działało. Opcje

[nitro18]

Mam kod który oblicza objętość folderów z podfolderami z plikami

[PHP] pobierz, plaintext 
<?php
function policz($sciezka) {
  if (!is_dir($sciezka)) return 0;
  $katalog = opendir($sciezka);
  while (false !== ($plik = readdir($katalog))) {
	if ($plik<>"." and $plik<>"..") {
	  $f = "$sciezka/$plik";
	  if (is_dir($f)){
	   $wielkosc+=policz($f);
	   $katalogi_licz=$katalogi_licz+1;
	}  else{
	 $wielkosc+=filesize($f);
	  $pliki_licz=$pliki_licz+1;
	  }
	}
  }
  closedir($katalog);
  return $wielkosc;
  return $katalogi_licz;
  return $pliki_licz;
}
?>
[PHP] pobierz, plaintext 

objętość danego folderu jest zwracana przez return $wielkosc.

Dodałem jeszcze do tej funkcji obliczanie ile jest plików i folderów w tym folderze.
Liczba plików ma być zwracana przez return $pliki_licz a liczba folderów - return $katalogi_licz

Tylko jest jedno ale. Funkcja zwraca mi tylko wielkość folderu i jak przeczytałem w dokumentacji php.net kończy działanie funkcji przez co nie zwraca liczby plików i folderów.

W jaki sposób zwrócić dwie ostatnie zmienne?

Wie ktoś może jak to naprawić?

[Cienki1980]

[PHP] pobierz, plaintext 
<?php
$tab['wielkosc']=$wielkosc;
$tab['katalogi_licz']=$katalogi_licz;
$tab['pliki_licz']=$pliki_licz;
 
return $tab;
?>
[PHP] pobierz, plaintext 

Zwracasz tablicę i po wywołaniu funkcji odczytujesz dane prosto z niej.

[nitro18]

Pisze mi:
Fatal error: Unsupported operand types in C:\serwer\users\browser\index.php on line 11

choć dodałem tak jak napisałeś.


później jak chce wywołać:

$tab=policz($sciezka."/".$katalogi[$i]);

echo $tab['wielkosc'];

ale też nie działa.

[Cienki1980]

Dziwne .. skoro wcześniej funkcja działała a nie zmieniłeś nic ważnego w samej funkcji.

A spróbuj zmienić tak aby funkcja zwracała pojedyncze wartości i zobacz czy będzie dla każdej działała. Jak dla mnie powinno działać zwracając tablicę tak samo dobrze jak zwracając pojedynczą wartość.

[nitro18]

ogólnie to jest mi potrzebne bo buduje menagera plikow na potrzeby mojego panela.

Podaje Wam pełny kod nieskończony więc tylko patrzcie tam gdzie te zmienne chce wrócić a reszta jest w fazie rozwoju (oj, wiem że namieszane, ale trudno).

[HTML] pobierz, plaintext 
<?
$setup=$_GET['setup'];
$dir=$_GET['dir'];
function policz($sciezka) {
  if (!is_dir($sciezka)) return 0;
  $katalog = opendir($sciezka);
  while (false !== ($plik = readdir($katalog))) {
    if ($plik<>"." and $plik<>"..") {
      $f = "$sciezka/$plik";
      if (is_dir($f)){
	   $wielkosc+=policz($f);
	   $katalogi_licz=$katalogi_licz+1;
    }  else{
	 $wielkosc+=filesize($f);
	  $pliki_licz=$pliki_licz+1;
	  }
    }
  }
  closedir($katalog);
  return $wielkosc;
  return $katalogi_licz;
  return $pliki_licz;
}
 
if($setup==""){
$sciezka="../a/";
$sciezka=$sciezka.$dir;
 
$f = opendir($sciezka);
while (false !== ($plik = readdir($f))) {
  if ($plik<>"." and $plik<>"..") {
    if (is_dir($sciezka."/".$plik)) $katalogi[] = $plik;
    else $pliki[] = $plik;
  }
}
closedir($f);
 
@sort($katalogi);
@sort($pliki);
?>
 
<? 
$foldery[]=explode("/", $dir);
$ile= count($foldery[0]);
 
echo "<a HREF=\"?dir=\">Katalog Główny</A>";
for($i=0;$i<$ile;$i++){
$folderys.=$foldery[0][$i]."/";
$folderyss=substr($folderys,0,-1);
echo "<a HREF=\"?dir=".$folderyss."\">".$foldery[0][$i]."/</A>  ";
}
?>
<BR>
<BR>
<BR>
 
 
<table border=1 width="100%">
<tr>
<td> </TD><th>Nazwa</TH><th>Typ</TH><th>Rozmiar</TH><td> </TD>
</TR>
<? 
for($i=0;$i<count($katalogi);$i++){
?>
<tr>
<td style="width:25px;"><img SRC="type/folder.gif"></TD>
<td><a HREF="?dir=<?echo $dir."/".$katalogi[$i];?>"><?echo $katalogi[$i]; ?></A></TD>
<td>folder plików</TD>
<td>
<? 
$size=policz($sciezka."/".$katalogi[$i]);
 
if($size<1024){
echo $size." bajtów (".$pliki_licz;
}elseif(($size>=1024)&&($size<2048)){
echo $size/1024;
echo "-".$pliki_licz;
}
 
 
 
?>
</TD>
<td></TD>
 
</tr>
 
<?
}
 
?>
 
</TABLE>
<?
/*
for($i=0;$i<count($pliki);$i++){
echo $pliki[$i]."-----".filesize($sciezka."/".$pliki[$i])."<br>";
}
 
 
 
*/
}
 
?>
<BR>
<BR>
<BR>
<BR>
<BR>
<BR>
<? 
$file="jarek.malpa.pl.cos.pl.a.s.f.g.s.w..t.sd.d.f.e.s.=.d.d.d.-.jpg";
$ex=explode(".", $file);
 
$ile_ex=count($ex);
$roz=$ex[$ile_ex-1];
 
unset($roz);
$rozp="union";
 
 
$roz[jpg]="JPG";
$roz[gif]="GIF";
$roz[exe]="EXE";
 
//if(isset($roz[$rozp]))echo 'ok';
?>
[HTML] pobierz, plaintext 

[Cienki1980]

Jest OK
Zwróć uwagę, że wewnątrz funkcji policz() wywołujesz funkcję policz();

Jeżeli zmieniłeś to co zwraca funkcja, musisz przystosować jej wywołanie wewnątrz funkcji, np tak:

[PHP] pobierz, plaintext 
<?php
function policz($sciezka)
{
 if (!is_dir($sciezka))
  return 0;
 
 $katalog = opendir($sciezka);
 while (false !== ($plik = readdir($katalog)))
 {
  if ($plik<>"." and $plik<>"..")
  {
   $f = "$sciezka/$plik";
   if (is_dir($f))
   {
	$x=policz($f);
	$wielkosc+=$x['wielkosc'];
	$katalogi_licz=$katalogi_licz+1;
   }
   else
   {
	$wielkosc+=filesize($f);
	$pliki_licz=$pliki_licz+1;
   }
  }
 }
 closedir($katalog);
 $tab['wielkosc']= $wielkosc;
 $tab['kat']= $katalogi_licz;
 $tab['plik']= $pliki_licz;
 return $tab;
}
?>
[PHP] pobierz, plaintext 

Mi ta wersja zabanglała.

[nitro18]

tak jest bo jest rekurencyjnie czy jak to się nazywa.
ten przykład wziąłem z algorytmy.pl

[revyag]

A może skorzystać z dobrodziejstw php5 i spl ?

[PHP] pobierz, plaintext 
<?php
function GetSize($sDir) {
	$oIter = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($sDir),true);
	$iSize = 0;
	foreach($oIter as $sFile) {
		if($sFile->isFile()) {
			$iSize += $sFile->getSize();
		}
	}
	echo  $iSize;
}
?>
[PHP] pobierz, plaintext 

Wyświetla wielkość wszystkich plików w danym katalogu (wliczają jego podkatalogi)

[NuLL]

[PHP] pobierz, plaintext 
<?php
$size=0;
foreach(glob('katalog/*.*) as $f)) $size+=filesize($f);
echo $size;
?>
[PHP] pobierz, plaintext 

[nitro18]

Dziękuję, dziękuję.

Dzięki za skrypt revyag. Przeczytałem dodatkowo w manualu o SPL i stworzyłęm krócej liczenie plików i katalogów.

A oto efekt mojej pracy, manager plików

+ wyświetla ikonki dla 169 rodzajów plików
+ podgląd pliku dla 17 rozszerzeń
+ łatwe tworzenie folderu w aktualnym folderze którym jesteś
+ usuwasz folder -> usuwasz automatycznie od razu pliki i foldery które są w nim
+ upload do 5 plików, rozmiar pliku dyktuje serwer

Zobaczcie, spróbujcie, oceńcie.

[Turgon]

http://www.eip.ovh.org/browser/?dir=/jarek/../../../
Zabezpiecz to

[nitro18]

poprawiony. sprawdź. jeszcze jakieś błędy?

[Spirit86]

no jasne, wgrywanie plików, wgrałem sobie plik php panie Jarku M....

Ten post edytował Spirit86 31.01.2007, 23:21:32

[nitro18]

dodano funkcję sprawdzającą czy rozszerzenie pliku któy ma być wysłany na serwer jest na liście zakazanej