Forum PHP.pl > skrypt zapisuje do tabeli choć nie powinien/?/

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: skrypt zapisuje do tabeli choć nie powinien/?/

Forum PHP.pl > Forum > Przedszkole

beginner

25.12.2005, 13:03:28

nie ma wogóle żadnego problemu

johnson

25.12.2005, 23:39:31

Cytat

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?

Cytat

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

beginner

26.12.2005, 00:02:26

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

-----------

A Panu juz podziekowalismy ...
By It`s_me

johnson

26.12.2005, 12:26:16

Cytat(beginner @ 2005-12-25 23:02:26)

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.

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.