[PHP] while - objaśnienie na przykładzie Opcje

[shpaque]

Witajcie - to może paradoksalnie śmieszne, ale nigdy wcześniej nie korzystałem z pętli while a podejrzewam, że tutaj będzie jak znalazł. Chodzi o zapętlenie dopóki nazwa pliku nie będzie "wolna"

obecnie jest tak:

[PHP] pobierz, plaintext 
if (file_exists('./db/upload/'.$fileName.'.'.$fileExtension)) {$newFileName = $fileName.'[1].'.strtolower($fileExtension);}
                else {$newFileName = $fileName.'.'.strtolower($fileExtension);}
[PHP] pobierz, plaintext 

jest ok, tylko petla wykona się jeden raz, to znaczy kiedy dam plik 1.txt, to za drugim razem mi stworzy 1[1].txt, ale jesli jeszcze raz dam 1.txt to ZNÓW mi stworzy 1[1].txt i bedzie tu juz duplikat

wiec jak powinna wygladac while? cos takiego?

[PHP] pobierz, plaintext 
while (!file_exists('./db/upload/'.$fileName.'.'.$fileExtension)) {
$newFileName = $fileName.'[1].'.strtolower($fileExtension);
})
[PHP] pobierz, plaintext 

?

[nospor]

No ale zastanow sie przez chwile. W petli sprawdzasz ciagle ten sam plik, wiec jakim cudem petla ma kiedys przestac? Zdaje sie masz sprawdzac inny plik co nie?

[shpaque]

to wie,m dlatego zastanawiam sie czy takie cos przejdzie i czy ma sens:

[PHP] pobierz, plaintext 
if (file_exists('./db/upload/'.$fileName.'.'.$fileExtension)) {
                  $countWhile = 1;
                  $newFileName = $fileName.'['.$countWhile.'].'.$fileExtension;
                  while (!file_exists('./db/upload/'.$fileName.'['.$countWhile.'].'.$fileExtension)) {
                    $countWhile++;
                    $newFileName = $fileName.'['.$countWhile.'].'.$fileExtension;
                  }
                }
[PHP] pobierz, plaintext 

czy jednak trzeba pisac 3 razy wiecej i zapetlac wszystkie pliki w katalogu, robic glob'a sprawdzac nazwy, liczyc te nazwy i dopiero dodawac...

[trueblue]

Wykorzystałbym funkcję glob wyszukując po:

[PHP] pobierz, plaintext 
$nazwa . "_*." . $rozszerzenie
[PHP] pobierz, plaintext 

Mając liczbę znalezionych plików (count) dodajesz automatycznie licznik do nowej nazwy.

[nospor]

Wywal ten IF. wystarczy tylko to:

[PHP] pobierz, plaintext 
                  $newFileName = $fileName.'.'.$fileExtension;
$ = 1;
                  while (file_exists($newFileName)) {
                    $i++;
                    $newFileName = $fileName.'['.$i.']'.'.'.$fileExtension;
                  }
[PHP] pobierz, plaintext 

i juz
A w petli nie negacja tylko czy plik istnieje

[shpaque]

super, dziękuję bardzo

Wywal ten IF. wystarczy tylko to:

[PHP] pobierz, plaintext 
                  $newFileName = $fileName.'.'.$fileExtension;
$ = 1;
                  while (file_exists($newFileName)) {
                    $i++;
                    $newFileName = $fileName.'['.$i.']'.'.'.$fileExtension;
                  }
[PHP] pobierz, plaintext 

i juz
A w petli nie negacja tylko czy plik istnieje

nie jednak to nei dziala - zapisuje mi dwa razy tren sam plik...

daje w razie czego cala klamerke

[PHP] pobierz, plaintext 
if (isset($_FILES['uploads'])) {
            $filesToUpload = $_FILES['uploads'];
            $countFilesToUpload = count($filesToUpload['name']);
            $counter = 0;
            for ($i = 0; $i < $countFilesToUpload; $i++) {
              $newFileId = newStringId(8);
              $fileExtension = explode('.', $filesToUpload['name'][$i]);
              $fileExtension = strtolower(end($fileExtension));
              $fileName = str_replace('.'.$fileExtension, '', $filesToUpload['name'][$i]);
              if (in_array($fileExtension, $allowedExtensions)) {
                $newFileName = $fileName.'.'.$fileExtension;
                $countWhile = 1;
                while (file_exists($newFileName)) {
                  $countWhile++;
                  $newFileName = $fileName.'['.$i.'].'.$fileExtension;
                }
                move_uploaded_file($filesToUpload['tmp_name'][$i], './db/upload/'.$newFileName);
                // Dodajmy wpis do bazy danych
                $newFile = $loadPageDataBase -> files -> addChild('file', $newFileName);
                $newFile -> addAttribute('id', $newFileId);
                $newFile -> addAttribute('title', '');
 
                $counter++;
              }
            }
          }
[PHP] pobierz, plaintext 

ło matko jakie przeoczenie az wstyd

[PHP] pobierz, plaintext 
$newFileName = $fileName.'['.$countWhile.'].'.$fileExtension;
[PHP] pobierz, plaintext 

...jednak mimo tej poprawki dalje tworzy caly czas ten sam plik - czyli po prostu while nie dziala

[nospor]

Pokaz kod po poprawkach bo juz nie wiem jaka masz wersje

[shpaque]

[PHP] pobierz, plaintext 
if (isset($_FILES['uploads'])) {
 $filesToUpload = $_FILES['uploads'];
     $countFilesToUpload = count($filesToUpload['name']);
            $counter = 0;
            for ($i = 0; $i < $countFilesToUpload; $i++) {
              $newFileId = newStringId(8);
              $fileExtension = explode('.', $filesToUpload['name'][$i]);
              $fileExtension = strtolower(end($fileExtension));
              $fileName = str_replace('.'.$fileExtension, '', $filesToUpload['name'][$i]);
              if (in_array($fileExtension, $allowedExtensions)) {
                $newFileName = $fileName.'.'.$fileExtension;
                $countWhile = 1;
                while (file_exists($newFileName)) {
                  $countWhile++;
                  $newFileName = $fileName.'['.$countWhile.'].'.$fileExtension;
                }
                move_uploaded_file($filesToUpload['tmp_name'][$i], './db/upload/'.$newFileName);
                // Dodajmy wpis do bazy danych
                $newFile = $loadPageDataBase -> files -> addChild('file', $newFileName);
                $newFile -> addAttribute('id', $newFileId);
                $newFile -> addAttribute('title', '');
 
                $counter++;
              }
            }
          }
[PHP] pobierz, plaintext 

[nospor]

No ale panie kolego, jaj sobie ze mnie nie rob

W while sprawdzasz teraz tylko $newFileName a do upload robisz sciezke './db/upload/'.$newFileName... no przeciez to sa dwie rozne lokalizacje. W while tez masz sprawdzac './db/upload/'.$newFileName

[shpaque]

Faktycznie niezłe przeoczenie Teraz jest bosko i wszystko hula jak powinno - dziękuję bardzo; Zmieniłem jeszcze początkowy count dla while na "0" bo i tak liczy przed sprawdzeniem więc wtedy zaczyna od 1...

Cała klamerka wyglądać powinna właśnie tak:

[PHP] pobierz, plaintext 
// W tym kroku sprawdźmy, czy nie należy dodać jakichś plików
          if (isset($_FILES['uploads'])) {
            $filesToUpload = $_FILES['uploads'];
            $countFilesToUpload = count($filesToUpload['name']);
            $counter = 0;
            for ($i = 0; $i < $countFilesToUpload; $i++) {
              $newFileId = newStringId(8);
              $fileExtension = explode('.', $filesToUpload['name'][$i]);
              $fileExtension = strtolower(end($fileExtension));
              $fileName = str_replace('.'.$fileExtension, '', $filesToUpload['name'][$i]);
              if (in_array($fileExtension, $allowedExtensions)) {
                $newFileName = $fileName.'.'.$fileExtension;
                $countWhile = 0;
                while (file_exists('./db/upload/'.$newFileName)) {
                  $countWhile++;
                  $newFileName = $fileName.'['.$countWhile.'].'.$fileExtension;
                }
                move_uploaded_file($filesToUpload['tmp_name'][$i], './db/upload/'.$newFileName);
                // Dodajmy wpis do bazy danych
                $newFile = $loadPageDataBase -> files -> addChild('file', $newFileName);
                $newFile -> addAttribute('id', $newFileId);
                $newFile -> addAttribute('title', '');
 
                $counter++;
              }
            }
          }
[PHP] pobierz, plaintext