skrypt zapisuje do tabeli choć nie powinien/?/, else...elseif czy cos jeszcze Opcje

[-beginner-]

nie ma wogóle żadnego problemu (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Ten post edytował beginner 26.12.2005, 14:47:10

[johnson]

jednak z jakis powodów  dla plików ponizej 50 k
wykonywane jest polecenie

[PHP] pobierz, plaintext 
<?php
$query = "INSERT into pictures (id, pic_name, main) values ('$id', '$named', 'YES') ";
?>
[PHP] pobierz, plaintext 

jak to z jakichś powodów?
przecież masz warunek:

[PHP] pobierz, plaintext 
<?php
if ($filesize <= 50000)
?>
[PHP] pobierz, plaintext 

czyli to co międzi {} wykona się dla plików poniżej 50 kB więc nie rozumiem Twojego zdziwienia?

I ciekawostka ! polecenie wpisuje do tabeli  pic_name ale tylko skladową $sid  $named = "$login$sid.$app" main='YES' a nie wpisuje id.
o co tu chodzi ?

nic nie napisałeś, co to jest to $id i skąd ono się bierze, czy z formularza, czy z sesji, nic na ten temat nie wiadomo, więc trudno cokolwiek powiedzieć na ten temat.

poza tym piszesz trochę archaicznie, polecam Ci np. używanie $_FILES['userfile']['tmp_name'] zamiast $userfile

nie powinno się też pisać filesize("$userfile") tylko filesize($userfile)

funkcję SesID() też najlepiej umieścić na początku albo na końcu skryptu, a nie w środku

w tabeli pictures brak Ci również, z tego co widzę, klucza głównego, w zasadzie tą funkcję powinno tutaj pełnić id, ale u Ciebie tak nie jest, chyba, że kluczem jest pic_name, ale jak dla mnie to by było wtedy trochę przekombinowane (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

poza tym po uporządkowaniu kodu, od razu widać poszczególne warunki if ... else i zawsze jest czytelne, w którym warunku akurat się znajdujesz:

[PHP] pobierz, plaintext 
<?php
if($userfile) {    
    $filesize = filesize("$userfile");
    print("<font size="2" face="Verdana" color="#0000FF">");
    print("<b>Wielkosc pliku = $filesize</b><br>");
    print("</font>");
    if ($filesize <= 50000) {  
        $imagesize = getimagesize("$userfile");
        $query = "SELECT  login FROM members WHERE id = '$id' ";
        $result = mysql_query($query);
        $row_array =  mysql_fetch_row($result);
        $login = $row_array[0];
        if($imagesize[2] == 1) $app = "gif";
        if($imagesize[2] == 2) $app = "jpg";
        if($imagesize[2] == 3) $app = "png";
        $photo=strtolower($id);
        function SesID($length=4) {
            global $id;  
            $Pool = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";
            $Pool .= "abcdefghijklmnopqrstuvwxyz";
            $Pool .= "1234567890";
            for($index = 0; $index < $length; $index++) {
                $sid .= substr($Pool, (rand()%(strlen($Pool))), 1);
            }     
            return($sid);
        }   
        srand(time());
        $sid=SesID ();
        $named = "$login$sid.$app";
        $query = "SELECT  count(*) FROM pictures WHERE id = '$id' ";
        $result = mysql_query($query);
        if(!($result)) echo "nie mozna pobrac liczby zdjęc";
        $row_array =  mysql_fetch_row($result);
        $guest = $row_array[0];
        //print("liczba do maina= $guest");
        if ($guest == 0) {                
            $query = "INSERT into pictures (id, pic_name, main)
			values ('$id', '$named', 'YES') ";
            $result = mysql_query($query);    
            if(!($result)) die("nie da sie zapisac maina do pictures");
        } else {      
            $query = "INSERT into pictures (id, pic_name) values ('$id', '$named') ";
            $result = mysql_query($query);    
            if(!($result)) die("nie da sie zapisac do pictures");
        }         
        copy($userfile, "./images/$named"); //zapisywanie pliku w images
    } else {
        print("<font size="2" face="Verdana" color="#0000FF">");
        print("plik jest zbyt duzy Nie zapisano zdjęcia");
        print("</font>");
    }
}
?>
[PHP] pobierz, plaintext 

Ten post edytował johnson 25.12.2005, 23:41:58