Skip to main content
6 de 6
adicionou 136 caracteres ao conteúdo
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

A primeira tentativa de deixar mais elegante:

if (file_exists($pathFize)) {

Mas se for usar esse dado muitas vezes então é melhor (ou a forma mais correta possível fazendo assim):

$response = file_exists($pathFize)

if ($response) {

Se quer fazer certo mesmo, e aí não é questão de elegância, não deve verificar se o arquivo existe, a não ser que a lógica seja realmente determinar se o arquivo existe ou não, só isto, então não pode fazer assim. Se a intenção é acessar o arquivo, o acesse e verifique se o acesso deu certo. Se não conseguiu acessar por qualquer razão trate o erro. Se fizer o que está tentando então poderá ter uma condição de corrida (não é o foco explicar aqui até porque pode estar querendo só verificar a existência mesmo).

Não analisei se a lógica está correta até porque pelo trecho e pelo postado não dá para saber.

Um dos pontos que fica mais elegante é não comparar uma variável que tem valor booleano com um literal booleano, porque o resultado sempre será igual ao da variável, portanto está usando um código redundante, pode usar a variável diretamente. O mesmo vale para uma expressão que gere um valor booleano.

Mas note que isso só funciona se o valor for garantidamente booleano. Se houver a possibilidade do valor ser outro (PHP tem tipagem dinâmica e fraca e pode acontecer isso em algumas situações. Neste caso não tem problema, é booleano garantidamente, pelo menos de acordo com a documentação.

Não faz sentido usar um operador de negação na expressão (!file_exists($pathFize)) para obter o valor invertido do que quer, por isso quando eu usei eu só eliminei o operador, o seu código está fazendo uma dupla inversão, portanto pegando o valor original.

E reforço que se você pega um valor boobleano (o retorno de file_exists() é um booleano de acordo com a documentação) não tem porque verificar o valor se o que deseja é um booleano, você já tem esse valor, use-o de forma direta. Se o seu exemplo quisesse ele invertido, ou seja, se quer false quando ele retorna true e vice versa aí faria sentido usar o operador ! para inverter, mas nem é o que quer no exemplo.

Um último ponto é que é bem esquisito usar um operador condicional (que alguns chamam erroneamente de ternário) para executar um comando complexo, em geral você só deveria gerar valores de forma direta. Vamos pensar que gostaria de gerar um número por alguma razão, o mais elegante seria fazer isto:

$response = funcaoQueRetornaBool() ? 1 : 0;

Coloquei no GitHub para referência futura.

Percebeu que a atribuição está fora do operador e não dentro como fez?

Funciona porque uma atribuição em PHP é uma expressão e não um statement puro (um statement aceita só declarações então fica confuso) então o operador aceita a atribuição, já que ele espera expressões. Se a atribuição fosse só statement não aceitaria.

Já expliquei essa questão do booleano com mais detalhes em alguns lugares:

Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil