Forum PHP.pl > [PHP] while - objaśnienie na przykładzie

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP] while - objaśnienie na przykładzie

shpaque

7.01.2021, 14:44:32

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

7.01.2021, 15:57:22

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

7.01.2021, 16:05:41

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

7.01.2021, 16:24:30

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

7.01.2021, 16:25:55

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

7.01.2021, 21:59:40

super, dziękuję bardzo

Cytat(nospor @ 7.01.2021, 16:25:55 )

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

8.01.2021, 10:45:56

Pokaz kod po poprawkach bo juz nie wiem jaka masz wersje

shpaque

8.01.2021, 11:07:15

[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

8.01.2021, 11:34:04

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

8.01.2021, 11:43:36

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

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.