Forum PHP.pl

Witam, mam super długi kod, w skrócie chodzi o to że mam 10 obrazków opcjonalnych do wysłania i 1 obowiązkowy + inne opisy. Oto co próbowałem zrobić

http://wklej.org/id/119745/

Wiem że ten kod wydaje się bez sensu ale jest OK, tylko gdzieś mam tego buga, bo zawsze wywala "Pozostawiles puste pola" mimo ze wymagane 3 mam wypelnione.

Ten post edytował MateuszScirka 11.07.2009, 22:07:23

Daj na wklej.org czy gdzieś ten kod a na forum tylko link.
Popraw też tytuł na bardziej sensowny.

array + for

Pr0100 a moglbys troche wyjasnic? Co mi da petla?

da Ci mniej linijek kodu. po to właśnie są pętle aby iterować po każdym elemencie. bo robisz np 100 if'ów w którym zmienia się tylko jedna cyferka - bez sensu. a tablice, ja dopiero teraz odkryłem jak poprawnie wprowadzone dane do tablic mogą ułatwić życie, i 100% się do nich przekonałem więc polecam.

Ok. Postaram się zaglebic w filozofie petel i tablic ale jeszcze ten jeden raz trzeba mi rozwiazac ten problem dlaczego wywala caly czas ten blad...

uproscilem
http://wklej.org/id/119791/

dziala tylko przy malych pliczkach. ;/

Jeżeli to nazywasz uproszczonym kodem...



W php.ini masz coś takiego jak upload_max_filesize i właśnie ta wartość Cię może ograniczać.

To nie to bo na serwerze tez mam blad ze puste pola. Poza tym w ini na 30 mb jest ustawione a obrazek ma 500 kb


Rozwiazalem tamten problem, ale mam nastepny. Pomyslalem troche nad tym co mowil Pr0100 i zastosowalem fora ale cos chyba zle

[PHP] pobierz, plaintext 
<?php
for($i=1 ; $i<=10 ; $i++)
        {
            $obrazektmp = $obrazektmp.$i;
            $obrazekname = $obrazekname.$i;
            
            if(!empty($obrazektmp))
            {
                if(is_uploaded_file($obrazektmp))
                {
                    $b = move_uploaded_file($obrazektmp, 'zdjecia/'.$_SESSION['time'].'/'.$obrazekname);
                    $obrazeczek.$i = 'zdjecia/'.$_SESSION['time'].'/'.$obrazekname;
                }
            }    
        }
?>
[PHP] pobierz, plaintext 

Bo nie uploaduje mi tych zdjec ;/

nie

[PHP] pobierz, plaintext 
<?php
$obrazeczek.$i=
?>
[PHP] pobierz, plaintext 

tylko

[PHP] pobierz, plaintext 
<?php
$obrazeczek[$i]=
?>
[PHP] pobierz, plaintext 

Niestety to nie to. Probowalem tez tak ale bez skutku. Tu wiecej kodu zeby bylo all widac

[PHP] pobierz, plaintext 
<?php
$obrazektmp1 = $_FILES['obrazek1']['tmp_name'];
    $obrazekname1 = $_FILES['obrazek1']['name'];
    $obrazektmp2 = $_FILES['obrazek2']['tmp_name'];
    $obrazekname2 = $_FILES['obrazek2']['name'];
    $obrazektmp3 = $_FILES['obrazek3']['tmp_name'];
    $obrazekname3 = $_FILES['obrazek3']['name'];
    $obrazektmp4 = $_FILES['obrazek4']['tmp_name'];
    $obrazekname4 = $_FILES['obrazek4']['name'];
    $obrazektmp5 = $_FILES['obrazek5']['tmp_name'];
    $obrazekname5 = $_FILES['obrazek5']['name'];
    $obrazektmp6 = $_FILES['obrazek6']['tmp_name'];
    $obrazekname6 = $_FILES['obrazek6']['name'];
    $obrazektmp7 = $_FILES['obrazek7']['tmp_name'];
    $obrazekname7 = $_FILES['obrazek7']['name'];
    $obrazektmp8 = $_FILES['obrazek8']['tmp_name'];
    $obrazekname8 = $_FILES['obrazek8']['name'];
    $obrazektmp9 = $_FILES['obrazek9']['tmp_name'];
    $obrazekname9 = $_FILES['obrazek9']['name'];
    $obrazektmp10 = $_FILES['obrazek10']['tmp_name'];
    $obrazekname10 = $_FILES['obrazek10']['name'];
 
        for($a = 1 ; $a <=10 ; $a++)
        {
            $obrazektmp[$a] = 'obrazektmp'.$a;
            $obrazekname[$a] = 'obrazekname'.$a;
        
                if(!empty($obrazekname[$a]))
                {
                    if(is_uploaded_file($obrazektmp[$a]))
                    {
                        $b = move_uploaded_file($obrazektmp[$a],   'zdjecia/'.$_SESSION['time'].'/'.$obrazekname[$a]);
                        $obrazeczek.$a = 'zdjecia/'.$_SESSION['time'].'/'.$obrazekname[$a];
                    }
                }    
            
        }
?>
[PHP] pobierz, plaintext 

nie tworz tyle zmiennych tylko jedną tablicę
$obrazektmp[1]=$_FILES...
$obrazekname[1]=$_FILES...

przeczytaj jeszcze raz dokładnie o tablicach.

W akcie desperacji zrobilem tak:

[PHP] pobierz, plaintext 
<?php
$obrazektmp = array($_FILES['obrazek1']['tmp_name'], $_FILES['obrazek2']['tmp_name'], $_FILES['obrazek3']['tmp_name'],
                    $_FILES['obrazek4']['tmp_name'], $_FILES['obrazek5']['tmp_name'], $_FILES['obrazek6']['tmp_name'],
                    $_FILES['obrazek7']['tmp_name'], $_FILES['obrazek8']['tmp_name'], $_FILES['obrazek9']['tmp_name'],
                    $_FILES['obrazek10']['tmp_name']);
    $obrazekname = array($_FILES['obrazek1']['name'], $_FILES['obrazek2']['name'], $_FILES['obrazek3']['name'], 
                    $_FILES['obrazek4']['name'], $_FILES['obrazek5']['name'], $_FILES['obrazek6']['name'], 
                    $_FILES['obrazek7']['name'], $_FILES['obrazek8']['name'], $_FILES['obrazek9']['name'], 
                    $_FILES['obrazek10']['name']);
 
 
for($a = 0 ; $a <=9 ; $a++)
        {        
                if(!empty($obrazekname[$a]))
                {
                    if(is_uploaded_file($obrazektmp[$a]))
                    {
                        $b = move_uploaded_file($obrazektmp[$a], 'zdjecia/'.$_SESSION['time'].'/'.$obrazekname[$a]);
                        $obrazeczek.$a = 'zdjecia/'.$_SESSION['time'].'/'.$obrazekname[$a];
                    }
                }    
            
        }
?>
[PHP] pobierz, plaintext 

Szkoda ze nie dziala ;/ Caly czas sie uploaduje az pisze ze przekroczono limit czasu a mam to na localhoscie powinno szbko

Ten post edytował MateuszScirka 12.07.2009, 10:45:20

jeżeli problem dotyczyłby użytkowników to zaproponowałbym inne rozwiązanie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ale masz racje to twoje rozwiązanie jest lepisze

Ok dzieki, wieczorem sie pobawie.

PS Foreacha nigdy nie uzywalem ;]