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

A primeira tentativa:

if (file_exists($pathFize)) {

Mas se for usar esse dado muitas vezes então é melhor:

$response = file_exists($pathFize)

if ($response) {

Mas 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. Se a intenção é acessar o arquivo, o acesse e verifique se o acesso deu certo. Se não conseguiu fazer trate o erro. Se fizer o que está tentando então poderá ter uma condição de corrida.

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 de uma vez. 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 ele de forma direta. Se o seu exemplo quisesse ele invertido, ou seja, se quer false quando eele retorna true e vice versa aí faria sentido usar o operador ! para inverter, mas nem é o que quer no exemplo.

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