O erro está na sua condição.
Ao utilizar o empty
, que pode gerar resultados inesperados, pois empty(false)
retorna verdadeiro, entrando no if
, mas empty(true)
retorna falso;
Ao utilizar strlen != 1
, pois também está suscetível a resultados inesperados, visto que strlen(false)
retorna 0, que é diferente de 1, entrando no if
, mas strlen(true)
retorna 1;
Na parte ($valor !=1 || $valor !=0)
, pois se $valor
for igual a 0, a condição $valor != 1
será verdadeira, fazendo com que o programa entre no if
e retorne inválido; o mesmo acontece quando $valor
for igual a 1, pois a condição $valor != 0
passa a ser verdadeira, também entrando no if
. A correção do problema resume-se em mudar o operador ||
para &&
;
Para mostrar quão estranho poderiam ser os resultados, basta fazer o teste:
function valida_variavel_zero_um($valor){
if((empty($valor)) OR (strlen($valor) != 1) OR ($valor != 1 && $valor != 0)){
return "INVALIDO";
}else{
return $valor;
}
}
$tests = [0, 1, false, true, "0", "1", [], "", 0.0, -0.0, 1.0];
foreach ($tests as $test)
{
var_dump( valida_variavel_zero_um($test) );
}
Veja funcionando no Ideone | Repl.it
Obtendo o resultado:
string(8) "INVALIDO" // $valor = 0
int(1) // $valor = 1
string(8) "INVALIDO" // $valor = false
bool(true) // $valor = true
string(8) "INVALIDO" // $valor = "0"
string(1) "1" // $valor = "1"
string(8) "INVALIDO" // $valor = []
string(8) "INVALIDO" // $valor = ""
string(8) "INVALIDO" // $valor = 0.0
string(8) "INVALIDO" // $valor = -0.0
float(1) // $valor = 1.0
Percebe quão estranhos são esses resultados? Alguns pares de valores semelhantes produzem resultados diferentes: 0 é inválido, mas 1 é válido; false
é inválido, mas true
é válido; 0.0 é inválido, mas 1.0 é válido.
Não faz muito sentido, então proponho outra solução:
Solução 1
A solução que eu proponho tem uma condição bem simples:
function valida_variavel_zero_um($valor)
{
if ($valor !== 0 && $valor !== 1)
{
return "inválido";
}
return $valor;
}
Ao utilizar o operador !==
, você verificará tanto o valor quanto o tipo. Ao fazer alguns testes breves:
$tests = [0, 1, false, true, "0", "1", [], "", 0.0, -0.0, 1.0];
foreach ($tests as $test)
{
var_dump( valida_variavel_zero_um($test) );
}
Teremos a saída:
int(0) // $valor = 0
int(1) // $valor = 1
string(9) "inválido" // $valor = false
string(9) "inválido" // $valor = true
string(9) "inválido" // $valor = "0"
string(9) "inválido" // $valor = "1"
string(9) "inválido" // $valor = []
string(9) "inválido" // $valor = ""
string(9) "inválido" // $valor = 0.0
string(9) "inválido" // $valor = -0.0
string(9) "inválido" // $valor = 1.0
Veja funcionando no Ideone | Repl.it
Solução 2
Outra forma de implementar a função seria explicitar o tipo definindo-o na lista de argumentos da função:
function valida_variavel_zero_um(int $valor)
{
if ($valor != 0 && $valor != 1)
{
return "inválido";
}
return $valor;
}
Veja funcionando no Ideone | Repl.it
Porém, ao chamar a função com valores não inteiros, o PHP tentará convertê-los para o tipo int
antes de passá-los como parâmetro. Isto é, chamar a função valida_variavel_zero_um(false)
retornaria 0 e valida_variavel_zero_um(true)
retornaria 1, pois false
e true
, quando analisados como inteiros, são 0 e 1 respectivamente. Isso pode gerar problemas na aplicação caso esse retorno não seja esperado para esses parâmetros.
A saída dos testes para esta solução é:
int(0) // $valor = 0
int(1) // $valor = 1
int(0) // $valor = false
int(1) // $valor = true
int(0) // $valor = "0"
int(1) // $valor = "1"
Argument 1 passed to valida_variavel_zero_um() must be of the type integer, array given, called in /home/tzgjpG/prog.php on line 18
Argument 1 passed to valida_variavel_zero_um() must be of the type integer, string given, called in /home/tzgjpG/prog.php on line 18
int(0) // $valor = 0.0
int(0) // $valor = -0.0
int(1) // $valor = 1.0
Solução 3
Outra forma, citada inclusive pelo Victor nos comentários, seria tentar analisar o valor do parâmetro com a função intval
, porém, o problema da solução 2 persiste nesta solução, pois a mesma conversão de tipos que o PHP faz implicitamente quando é definido o tipo na lista de parâmetros ele fará com a função intval
:
function valida_variavel_zero_um($valor)
{
$valor = intval($valor);
if ($valor != 0 && $valor != 1)
{
return "inválido";
}
return $valor;
}
Veja funcionando no Ideone | Repl.it
O resultado dos testes para esta solução é:
int(0) // $valor = 0
int(1) // $valor = 1
int(0) // $valor = false
int(1) // $valor = true
int(0) // $valor = "0"
int(1) // $valor = "1"
int(0) // $valor = []
int(0) // $valor = ""
int(0) // $valor = 0.0
int(0) // $valor = -0.0
int(1) // $valor = 1.0
Ou seja, mesma passando booleanos, strings, arrays e números com ponto flutuante como parâmetro, a função sempre retornará um valor válido conforme o valor passado. Novamente, esta solução pode trazer problemas para aplicação por trazer resultados possivelmente inesperados.
As três soluções são válidas e dependerá somente da sua necessidade. Por questões de semântica, eu prefiro a solução 1, que só retorna valores válidos quando a entrada é 0 ou 1, inteiros, que aparentemente é o que se pede na pergunta.
Leitura obrigatória
O que é uma comparação frouxa?
$valor != 1
é verdadeira e entra noif
. Se for 1, a condição$valor != 0
é verdadeira e entra noif
. Consegue entender o problema?