3

Qual motivo minha IDE PHPStorm, aparece esta mensagem com include e require:

Dynamic include expression 'require_once $file' is not analysed.
Include expression is not resolved.

Código PHPStorm

1
  • @bigown adicionei a pergunta. Commented 29/05/2016 às 20:13

1 Resposta 1

4

O comando faz a inclusão de código baseado em uma variável, portanto o PHPStorm, que é um IDE que tenta analisar o máximo do que pode acontecer com o código, não pode ir adiante na análise disto, só durante a execução teria condições de saber o que será incluso aí. Ele dá ciência a isto.

Este código tem vários outros problemas.

Já começa com um condição de corrida verificando se um arquivo existe, e que pode deixar de existir entre a verificação e sua invocação. Pode ser raro, mas acontece.

Depois lança uma exceção sem sentido, sem informação relevante.

E ainda inclui código não só condicionalmente, que já é ruim, mas baseado em variável. Até pode ser o certo a fazer, mas é raro. Tem que saber muito bem o que está fazendo, entender todas consequências disto, precisa ter um bom motivo. Se esta variável vier de fonte não confiável, aí abre uma brecha de segurança enorme.

Não vou falar do uso de SPL, porque é só minha opinião :P

11
  • 1
    Não sei, que eu me lembre nunca precisei fazer. Se um dia eu realmente precisar e não tiver outra solução, eu teria que ver os requisitos para identificar a melhor forma de fazer. Os problemas nunca são iguais. Sem requisitos não dá para falar nada.
    – Maniero
    Commented 29/05/2016 às 20:44
  • 2
    Namespace é uma coisa que eu nunca usei no PHP, mas não vejo porque precisaria. O autoload é um recursos que pode ser usado de várias formas, é algo separado. Já vi sendo usado de diversas outras formas, não desta.
    – Maniero
    Commented 29/05/2016 às 21:00
  • 1
    Não é questão de ser patético, é só inexperiência. Ninguém nasceu sabendo.
    – Maniero
    Commented 29/05/2016 às 21:14
  • 2
    O que eu acho uma tremenda gambiarra. Mas inventaram isso e tem gente que gosta. Não digo que não tenha utilidade, mas vejo muito abuso. As coisas podem ser mais simples.
    – Maniero
    Commented 29/05/2016 às 21:37
  • 1
    Eu não faria isto :) Arrumar ou fazer códigos sempre depende de requisitos, o que eu não sei quais são.
    – Maniero
    Commented 30/05/2016 às 1:11

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .