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.

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

jeżeli problem by dotyczył użytkowników to byś zrobił 20 tablic w stylu?

$user_nick[$i]
$user_login[$i]
$user_pass[$i]
$user_mail[$i]
$user_city[$i]
$user_sex[$i]
....

kiepskie rozwiązanie problemu


zamiast:

[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'];
?>
[PHP] pobierz, plaintext 

zrób:

[PHP] pobierz, plaintext 
<?php
for($i=1; $i <= 10; $i++)
{
  $obrazki[$i] = array(
    'name' => $_FILES['obrazek'.$i]['name'],
    'tmp' => $_FILES['obrazek'.$i]['tmp_name']
  );
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
echo 'Nazwa obrazku #3: '.$obrazki[3]['name'];
echo "<br />";
echo 'Tymczasowa nazwa obrazku #6: '.$obrazki[6]['tmp_name'];
 
echo "<br /><br /><br /><br />";
 
 
echo 'Lista obrazków: <br/>';
foreach ($obrazki as $obrazek)
{
  echo $obrazek['name'].'<br />';
}
?>
[PHP] pobierz, plaintext 

masz przykład wykorzystania for, array oraz foreach

#####################################################

zamiast

[PHP] pobierz, plaintext 
<?php
if($_POST['jezyk']=='opis_pl')
    {
        $opispl = 1;
        $opisen = 0;
        $opisde = 0;
    }    
    if($_POST['jezyk']=='opis_en')
    {
        $opispl = 0;
        $opisen = 1;
        $opisde = 0;
    }
    if($_POST['jezyk']=='opis_de')
    {
        $opispl = 0;
        $opisen = 0;
        $opisde = 1;
    }
?>
[PHP] pobierz, plaintext 

zrób

[PHP] pobierz, plaintext 
<?php
$opispl = $opisen = $opisde = 0;
 
switch($_POST['jezyk'])
{
  case 'opis_pl':
    $opispl = 1;
    break;
 
  case 'opis_en':
    $opisen = 1;
    break;
 
  case 'opis_de':
    $opisde = 1;
    break;
  
  default:
   // komunikat nie wybrałeś języka, nieznany język lub coś w tym kierunku
   break;
}
?>
[PHP] pobierz, plaintext 

jeżeli problem dotyczyłby użytkowników to zaproponowałbym inne rozwiązanie ale masz racje to twoje rozwiązanie jest lepisze

Ok dzieki, wieczorem sie pobawie.

PS Foreacha nigdy nie uzywalem ;]