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

[-beginner-]

nie ma wogóle żadnego problemu

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

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

[-beginner-]

Usuwam moje posty bo drażni mnie arogancja i ignorancja administracji tego serwisu...Sorki.pa
-----------

A Panu juz podziekowalismy ...
By It`s_me

Ten post edytował itsme 26.12.2005, 23:40:00

[johnson]

W tabeli pictures kluczem jest id

po co więc liczysz rekordy o takim samym id?
skoro id jest kluczem to zapytanie:

[PHP] pobierz, plaintext 
<?php
$query = "SELECT  count(*) FROM pictures WHERE id = '$id' ";
?>
[PHP] pobierz, plaintext 

zawsze powinno dać 0, bo klucz główny powinien być zawsze unikalny

Ty jako id w tabeli pictures wstawiasz id użytkownika, co nie jest dobrym rozwiązaniem, powinieneś dodać nowe pole w tabeli np. user_id i tam przechowywać id użytkownika, natomiast id powienieneś ustanowić jako klucz główny.

Co się zaś tyczy warunku:

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

to zdaje się, że jest taka zmienna $userfile_size, a na pewno jest $_FILES['userfile']['size'], w której jest zapisany rozmiar uploadowanego pliku, nie musisz więc używać funkcji filesize().

Zobacz co wyświetla się w linijce:

[PHP] pobierz, plaintext 
<?php
print("<b>Wielkosc pliku = $filesize</b><br>");
?>
[PHP] pobierz, plaintext 

jeśli tutaj rozmiar jest większy niż 50000 to jak zobaczysz na przygotowany wyżej przeze mnie fragment kodu, nie jest możliwe aby wykonało się zapytanie insert ..., chyba, że od momentu Twojego pierwszego postu (z którego zapożyczyłem fragment kodu) dokonałeś już jakiś zmian w kodzie.