[PHP] Przejście parametrów przez __set Opcje

[Yhy]

Mam następny problem, fragment kodu -

[PHP] pobierz, plaintext 
    class Avatar{
 
        public $file_size;
        public $file_type;
        public $file_tmp;
        public $file_name;
 
        function __set($name, $value){
            if( ($name = "file_size") && ($value < 20000)){
                return $this->file_size = $value;
            }
            elseif( ($name = "file_type") && ($value == "image/jpeg" or "image/png")){
                return $this->file_type = $value;
            }
            else{
                echo 'Plik nie spełnia wymagań.';
            }
        }
 
        public function upload(){
 
            if( move_uploaded_file($this->file_tmp, "avatars/$this->file_name")){
                echo 'Plik został zauploadowany.';
            }
 
 
        }
 
    }
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
        <?php
 
            echo '<form action="#" method="POST" enctype="multipart/form-data">
                       <input type="file" name="avatar">
                       <input type="submit" name="submit" value="Wgraj">
                  </form>';
 
            if( isset($_FILES['avatar'])){
 
                include('class.Avatar.php');
 
                $avatar = new Avatar;
 
                $avatar->$file_size = $_FILES['avatar']['size'];
                $avatar->file_tmp = $_FILES['avatar']['tmp_name'];
                $avatar->file_name = $_FILES['avatar']['name'];
 
                $avatar->upload();
            }
 
 
        ?>
[PHP] pobierz, plaintext 

Błąd:

Notice: Undefined variable: file_size in index.php on line 32
Plik został zauploadowany.

Problem - nie widzę błędu, ponieważ z tego co wiem znak dolara powodu przefiltrowanie parametru przez funkcję __set. Ponadto plik którzy wrzucam jest zbyt duży, więc powinien zostać odrzucony.

Ten post edytował Yhy 10.10.2011, 15:24:48

[CuteOne]

$avatar->$file_size = $_FILES['avatar']['size'];

podmień na

$avatar->file_size = $_FILES['avatar']['size'];

[Yhy]

Okej, do tego już doszedłem, z tym że funkcja __set nadal nie filtruje mi danych i przepuszcza pliki większe niż 20kb.

Ktoś może mi pomóc?

Ten post edytował Yhy 10.10.2011, 17:12:07

[nospor]

yhym chlopie, zapoznaj się z podstawami języka, w którym piszesz :/

if( ($name = "file_size")
= - przypisanie
== - porównanie

elseif( ($name = "file_type") && ($value == "image/jpeg" or "image/png")){
$value == "image/jpeg" or "image/png" - co to jest za jakis skrótowiec? W if mabyć warunek a nie jakiś skrót myślowy...

tak ma być
$value == "image/jpeg" or $value == "image/png"

Poza tym poczytaj o metodach magicznych __set oraz __get, a konkretnie kiedy one działają - tzn. kiedy są odpalane, bo u Ciebie nie będę odpalane nigdy.

[Yhy]

Z tego co wiem, to przed nazwą pola muszę dodać znak dolara, np.

[PHP] pobierz, plaintext 
$obiekt->$pole = 20;
[PHP] pobierz, plaintext 

Z Tym że wtedy sypie mi błędem. Co do porównywania, przeoczenie.

Ten post edytował Yhy 10.10.2011, 19:22:24

[nospor]

A skąd ty to wiesz? Bo to jakaś bzdura.

Tu masz opisane kiedy działają __set
http://pl.php.net/manual/pl/language.oop5....loading.members
Zapoznaj się z podaną tam teorią, zapoznaj się z podanym przykładem, który teorię wyjaśnia. Odpal go sobie na kompie, zobacz co się dzieje.
Choć i tak wszystko wyjaśnia to jedno zdanie:

__set() is run when writing data to inaccessible properties.

Powód edycji: [nospor]:

[Yhy]

http://www.youtube.com/watch?v=ZXrNeRcI9Xc

stąd, no ładnie, człowiek nie ma się z czego uczyć, a jak coś znajduje to jeszcze same głupoty.

Dzięki nospor, niestety nieudolnie się dokształciłem, poczytam.

[nospor]

Ałć.... zwracam ci honor. Faktycznie, nie Twoja wina, że na taki "zajebisty" kurs trafiłeś :/ Jak ludzie mogą coś takiego wypuszczać w świat?

Nawet tam jest porównanie tak zapisane:
if ($zm = 'costam')
:/

Choć wywołanie:
$obiekt->$pole = 20;
Też jest poprawne, ale tylko wówczas gdy istnieje zmienna $pole. Tutaj poprostu dobieramy się w dynamiczny sposób do pól.
Ale w tym kurscie co dałeś linka koleś to skopał i totalnie źle wyjaśnił, stąd nie miało to żadnego sensu w kodzie co ty pokazałeś w tym temacie.