Forum PHP.pl

Słuchajcie, skrypt powinien działać jak należy, a wywala mi:

[PHP] pobierz, plaintext 
Warning: fopen() [function.fopen]: Filename cannot be empty in C:\xampp\htdocs\funkcje_uwierzytel.php on line 32
 
Warning: fread() expects parameter 1 to be resource, boolean given in C:\xampp\htdocs\funkcje_uwierzytel.php on line 32
[PHP] pobierz, plaintext 

Co jest nie tak? Używam Xampp:Apache.

[PHP] pobierz, plaintext 
$tmp = $_FILES['zdjecie']['tmp_name'];
$nazwa = $_FILES['zdjecie']['name'];
$rozmiar = $_FILES['zdjecie']['size'];
$typ = $_FILES['zdjecie']['type'];
[PHP] pobierz, plaintext 

32 linijka:

[PHP] pobierz, plaintext 
$obraz = addslashes(fread(fopen($tmp, "r"), $rozmiar));
[PHP] pobierz, plaintext 

Ładuję plik przez formularz przeglądaj do tabeli obraz.

Ten post edytował waqmaz 18.09.2011, 12:28:50

Wyświetl sobie

[PHP] pobierz, plaintext 
var_dump($_FILES)
[PHP] pobierz, plaintext 

Teraz pokazało:

[PHP] pobierz, plaintext 
array(1) { ["zdjecie"]=> array(5) { ["name"]=> string(6) "b1.PNG" ["type"]=> string(9) "image/png" ["tmp_name"]=> string(23) "C:\xampp\tmp\php20A.tmp" ["error"]=> int(0) ["size"]=> int(458) } }
 
 
 
Warning: fopen() [function.fopen]: Filename cannot be empty in C:\xampp\htdocs\funkcje_uwierzytel.php on line 32
 
Warning: fread() expects parameter 1 to be resource, boolean given in C:\xampp\htdocs\funkcje_uwierzytel.php on line 32
[PHP] pobierz, plaintext 

A nie możesz najpierw przenieść pliku do katalogu strony i dopiero wtedy go odczytać ?

Zamień

[PHP] pobierz, plaintext 
$obraz = addslashes(fread(fopen($tmp, "r"), $rozmiar));
[PHP] pobierz, plaintext 

na

[PHP] pobierz, plaintext 
$obraz = addslashes(fread(fopen("tmp/".$tmp, "r"), $rozmiar));
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
array(1) { ["zdjecie"]=> array(5) { ["name"]=> string(12) "pytajnik.PNG" ["type"]=> string(9) "image/png" ["tmp_name"]=> string(23) "C:\xampp\tmp\php4F8.tmp" ["error"]=> int(0) ["size"]=> int(309) } }
Warning: fopen(tmp/) [function.fopen]: failed to open stream: No such file or directory in C:\xampp\htdocs\funkcje_uwierzytel.php on line 33
 
Warning: fread() expects parameter 1 to be resource, boolean given in C:\xampp\htdocs\funkcje_uwierzytel.php on line 33
Nie udało się załadować avatara, lecz twoje konto zostało utworzone bez niego. Gratulacje! Konto zostalo utworzone! 
[PHP] pobierz, plaintext 

Chyba nigdy nie będą działać te avatary =/ Dlaczego tak się dzieje, wie ktoś z góry? Wskazałem katalog, jak kazaliście i jak jeden z was podał w przykładzie, no i błąd.

Dam wam cały kod. Bez zabezpieczeń i pisany na szybko, więc nie wytykajcie mnie na razie.

<B>FUNKCJE_UWIERZYTEL.PHP</B>

[PHP] pobierz, plaintext 
function wyswietl_formularz_rej() {
?>
    <div class="ramka_rejestracji">
        <form enctype="multipart/form-data" action="index.php?action=nowa_rejestracja" method="post">
<center>
            <p>Adres e-mail:</p>
				<input type="text" name="email" size="30" maxlength="100" />
             <p>Hasło:</p>
                <input type="password" name="haslo" size="16" maxlength="16" />
            <p>Wprowadź hasło ponownie:</p>
                <input type="password" name="haslo2" size="16" maxlength="16" />
			<p>Nazwa użytkownika:</p>
                <input type="text" name="nazwa_uz" size="16" maxlength="16" />
			<p>Awatar:</p>
				<input type="hidden" name="max_file_size" value="5000" />
				<input name="zdjecie" type="file" /><br /><br />
 
                <input type="submit" value="Rejestruj">
</center>
        </form>
    </div>
<?php
}
 
function rejestrowanie_uzytkownika($nazwa_uz, $email, $haslo, $obraz) {
 
$lacz = lacz_bd();
$wynik = $lacz->query("select * from uzytkownik where nazwa_uz='".$nazwa_uz."'");
 
//$obraz = addslashes(fread(fopen($tmp, "r"), $rozmiar));
$obraz = addslashes(fread(fopen("tmp/".$tmp, "r"), $rozmiar));
 
    if (!$wynik) {
       throw new Exception('Wykonanie zapytania nie powiodlo sie.');
    }
 
    if ($wynik->num_rows>0) {
        throw new Exception('Niestety wybrana przez Ciebie nazwa uzytkownika juz istnieje w bazie danych. Prosimy wybrac inna nazwe.');
    }
if($obraz)
//$rsz = explode(".",$nazwa_obrazu);
 
//if(($rsz[1] == "jpg" or $rsz[1] == "gif" or $rsz[1] == "bmp" or $rsz[1] == "png") and (is_uploaded_file($tmp))){
 
$wynik = $lacz->query("insert into uzytkownik values('".$nazwa_uz."', sha1('".$haslo."'), '".$email."','".$obraz."')");
//$wynik = $lacz->query("INSERT INTO zdjecie SET zdjecie='".$obraz."'");
 
if($obraz){
    echo 'Plik:', $nazwa_obrazu, 'został przesłany na serwer! ';
}
else{
    echo 'Nie wybrales avatara, lecz twoje konto zostalo utworzone bez niego. ';
}
 
if (!$wynik) {
    throw new Exception('Rejestracja uzytkownika w bazie danych jest nie mozliwa');
}
 
  return true;
}
[PHP] pobierz, plaintext 

<B>NOWA_REJESTRACJA.PHP</B>

[PHP] pobierz, plaintext 
<?php
 
$email=$_POST['email'];
$nazwa_uz=$_POST['nazwa_uz'];
$haslo=$_POST['haslo'];
$haslo2=$_POST['haslo2'];
 
$tmp = $_FILES['zdjecie']['tmp_name'];          //MOZLIWE ZE TU COS SKNOCILEM!!!
$nazwa_obrazu = $_FILES['zdjecie']['name'];     //MOZLIWE ZE TU COS SKNOCILEM!!!
$rozmiar = $_FILES['zdjecie']['size'];          //MOZLIWE ZE TU COS SKNOCILEM!!!
$typ = $_FILES['zdjecie']['type'];              //MOZLIWE ZE TU COS SKNOCILEM!!!
var_dump($_FILES);
 
try {
if (!wypelniony($_POST)) {
    echo '<div class="ramka_error"><p>';
          throw new Exception('Pola oznaczone gwiazdka * musza zostac wypelnione.');
    echo '</p></div>';
}
 
if (!prawidlowy_email($email)) { 
    echo '<div class="ramka_error"><p>';
        throw new Exception('Podano nieprawidlowy adres email!');
    echo '</p></div>';
}
 
if ($haslo != $haslo2) { 
    echo '<div class="ramka_error"><p>';
         throw new Exception('Podano dwa rozniace sie od siebie hasla. Pole haslo i powtorz haslo musza byc takie same.');
    echo '</p></div>'; 
}
 
if (strlen($nazwa_uz) > 16) { 
    echo '<div class="ramka_error"><p>';
         throw new Exception('Nazwa uzytkownika, nie moze miec wiecej niz 16 znakow!');
    echo '</p></div>'; 
}
 
if ((strlen($haslo) < 6) || (strlen($haslo) > 16)) {
    echo '<div class="ramka_error"><p>';
          throw new Exception('Haslo musi skladac sie od 6 do 16 znakow!.');
    echo '</p></div>';
}
 
rejestrowanie_uzytkownika($nazwa_uz, $email, $haslo, $obraz);
 
echo 'Gratulacje! Konto zostalo utworzone!';
}
 
   catch (Exception $e) {
     echo $e->getMessage();
    exit;
   }
?>
 
[PHP] pobierz, plaintext 

Ten post edytował waqmaz 20.09.2011, 02:47:52

W złej kolejności wywołujesz funkcje. Powinno być:

[PHP] pobierz, plaintext 
$obraz = fread(fopen(addslashes($tmp), "r"), $rozmiar);
[PHP] pobierz, plaintext 

edit:
Nadużywasz wyjątków w swojej aplikacji. W przedstawionym wyżej kodzie powinieneś raczej logować/zapisywać błędy, aby je wszystkie jednocześnie wyświetlić. Tymczasem Ty przy każdym błędzie zatrzymujesz aplikację poprzez wyrzucenie wyjątku. W najgorszym razie pięciokrotnie zatrzymasz działanie aplikacji za każdym razem wyrzucając inny wyjątek. Potencjalnego użytkownika może to zniechęcić. Wyjątki nie nadają się do walidacji danych wprowadzonych do formularza.

Ten post edytował mortus 20.09.2011, 08:03:24

Wiem, mam juz przygotowana funkcje:

[PHP] pobierz, plaintext 
function blad($blad){
echo '<div class="ramka_error"><p>';
	echo 'BLAD:<br />'.$blad;
echo '</p></div>';
}
[PHP] pobierz, plaintext 

,przez co bede sobie wywolywal bez bloku try i z return false:

[PHP] pobierz, plaintext 
if (!wypelniony($_POST)) {
    blad('Pola oznaczone gwiazdka * musza zostac wypelnione.');
    return false;
}
[PHP] pobierz, plaintext 

zamiast tak, co ucinalo mi stopke w stronie:

[PHP] pobierz, plaintext 
if (!wypelniony($_POST)) {
    echo '<div class="ramka_error"><p>';
          throw new Exception('Pola oznaczone gwiazdka * musza zostac wypelnione.');
    echo '</p></div>';
}
[PHP] pobierz, plaintext 

Ok, juz sprawdzam ten kod, co mi podales.

Cholera, nic nie pomogło.

[PHP] pobierz, plaintext 
array(1) { ["zdjecie"]=> array(5) { ["name"]=> string(10) "waqmaz.PNG" ["type"]=> string(0) "" ["tmp_name"]=> string(0) "" ["error"]=> int(2) ["size"]=> int(0) } }
Warning: fopen(tmp/) [function.fopen]: failed to open stream: No such file or directory in C:\xampp\htdocs\funkcje_uwierzytel.php on line 34
 
Warning: fread() expects parameter 1 to be resource, boolean given in C:\xampp\htdocs\funkcje_uwierzytel.php on line 34
Nie wybrales avatara, lecz twoje konto zostalo utworzone bez niego. Gratulacje! Konto zostalo utworzone! 
[PHP] pobierz, plaintext 

Czuję się jak totalny noob =/

Ten post edytował waqmaz 20.09.2011, 13:44:46