Ругается на $this

[CeHTuJI]

Учу ООП в php, есть такой код:

PHP код:

class check {
      public $check_var;
      
      function check_var($chek_var) {
        $this->check_var = $check_var;
        $chek_var::htmlspecialchars($chek_var);
            $chek_var::strip_tags($chek_var);
            $chek_var::trim($chek_var);
        return $this->check_var;
      }
} 


[Кольша]

одинаковые названия у функции и переменной. пробуй так:

class check { 
      public $check_var; 
       
      function check_var_($chek_var) { 
        $this->check_var = $check_var; 
        $chek_var::htmlspecialchars($chek_var); 
            $chek_var::strip_tags($chek_var); 
            $chek_var::trim($chek_var); 
        return $this->check_var; 
      } 
}

[Andkorol]

Сообщение от CeHTuJI

PHP код:

        $this->check_var = $check_var;
        $chek_var::htmlspecialchars($chek_var);
            $chek_var::strip_tags($chek_var);
            $chek_var::trim($chek_var); 


Вот это всё – бред.
Особенно с точки зрения ООП.

[CeHTuJI]

аналогично
Using $this when not in object context in
на эту строку

$this->check_var = $check_var;

[CeHTuJI]

Решил забить и сделать так:

PHP код:

class check { 
 
      function check_var($chek_var) { 
        return trim(strip_tags(htmlspecialchars($chek_var))); 
      } 
} 


[Кольша]

Вот почитай:
1
2
3

[WebPr]

Учимся кодить правильно!
для этого нужно соблюдать простые правила:
1. У каждой переменной функции свое осмысленное имя, ни каких

function check_var_()
$check_var
$chek_var

2. Одна строка одно присваивание, это плохо:

return trim(strip_tags(htmlspecialchars($chek_var)));

3. и т.д.

Рекомендую прочитать книгу Steve McConnell "CODE COMPLETE", есть не русском.
PS. Все выше описанное только мое мнение, ни кому не навязываю

[Fenex]

2 пункт весьма и весьма спорный.