Использование переключателей

Question

Я делаю проверку безопасности с помощью своих переключателей, и она возвращает ошибку, хотя я не думаю, что она должна быть. Любая идея, что я делаю неправильно?

Это класс для обновления

public function set ($iStatus) 
{ 
    $this->iStatus = $iStatus; 
} 
public function create() 
{ 
    if ($this->iStatus != 0 || $this->iStatus != 1 || $this->iStatus != 2) 
    { 
     echo "Your idea must have a valid status"; 
    } 
    else 
    { 
     //update the database 
    } 
} 

, а затем HTML форма

if (isset($_POST["submit"])) 
{ 
    $class->set($_POST["status"]); 
    $class->create(); 
} 
else 
{ 
    <input type="radio" name="status" value="0" checked/> 
    <input type="radio" name="status" value="1" /> 
    <input type="radio" name="status" value="2" /> 
} 

И это возвращает ошибку действительного статуса. Я полагал, что, может быть, входы сохранить все, как строки или символы, так что я переделал проверки ошибки сказать

if ($this->iStatus != '0') { /*blah*/ } 

, но это не сработало. Поэтому я смущен

Answer 1

ваша логика назад:

if ($this->iStatus != 0 || $this->iStatus != 1 || $this->iStatus != 2) 

должно быть:

if ($this->iStatus != 0 && $this->iStatus != 1 && $this->iStatus != 2) 

полный тест:

<?php 
class foo{ 
    private $iStatus; 

    public function set($iStatus){ 
     $this->iStatus = $iStatus; 
    } 
    public function create(){ 
     if ($this->iStatus != 0 && $this->iStatus != 1 && $this->iStatus != 2){ 
      echo "Your idea must have a valid status"; 
     }else{ 
      echo "All good"; 
     } 
    } 
} 

if (isset($_POST["submit"])){ 
    $class = new foo; 
    $class->set($_POST["status"]); 
    $class->create(); 
}else{ 
    echo ' 
    <form method="post" action="./"> 
    <input type="radio" name="status" value="0" checked> 
    <input type="radio" name="status" value="1"> 
    <input type="radio" name="status" value="2"> 
    <input type="radio" name="status" value="3"> 
    <input name="submit" type="submit"> 
    </form>'; 
}?> 

Answer 2

ошибка в выражении if. Вы должны подставить || с & &

в противном случае вы всегда получите сообщение об ошибке

Потому что даже если это 0 ваш, если оператор возвращает истину, потому что = 1 верно