A partir da versão 5.4, é emitido erro do tipo STRICT para o caso apresentado.
Se não está vendo mensagens de erro na tela pode ser devido as configurações do reportador de erros.
Caso esteja usando PHP versão 5.4 ou superior, defina o reportador de erros para permitir reportar erros do tipo STRICT
Uma sugestão, num ambiente de desenvolvimento, é recomendável exibir todo tipo de erro e então corrigir todo tipo de problema que aparecer, mesmo os do tipo STRICT e DEPRECATED.
ini_set('error_reporting', E_ALL);
ini_set('display_errors', true);
A decisão sobre como configurar o reportador de erros depende de cada caso. Há casos onde é necessário ocultar erros do tipo STRICT e DEPRECATED. Um exemplo, é sistemas legados onde tornar-se inviável corrigir e converter muitos códigos. (falta de tempo, falta de investimentos, falta de permissão de um superior, etc).
Voltando ao assunto, é recomendável que corrija dentro do padrão, declarando explicitamente a visibilidade e o tipo dos métodos, tal como das propriedades de uma classe.
Exemplo
Como NÃO fazer:
class Foo {
function Bar() {
}
}
Corrija definindo a visibilidade. Nesse exemplo vamos usar public
.
class Foo {
public function Bar() {
}
}
O método não é estático, portanto deve ser invocado por uma instância
$c = new Foo;
$c->Bar();
Se tentar Foo::Bar();
apresentará erros STRICT e outros erros subsequentes como o acesso ao $this
fora de contexto, por exemplo. E isso vai causando uma bola de neve de erros.
Driblando o reportador de erros
Conforme mencionado acima, pode haver situações onde é inviável corrigir todos os erros. Para esses casos, pode fazer a seguinte configuração
/*
Aqui diz que deve reportar todos os erros exceto os do tipo STRICT e DEPRECATED.
*/
ini_set('error_reporting', E_ALL & ~E_STRICT & ~E_DEPRECATED);
/*
true: para ambiente de testes (o seu localhost)
false: para ambiente de produção (o site online, no servidor)
*/
ini_set('display_errors', false);
/*
Opcionalmente pode ligar o log de erros e então definir uma local onde deseja salvar esses avisos.
*/
ini_set('log_errors', true);
/*
Local onde serão gerados os arquivos de logs de erro.
Apenas cuidado pois dependendo da quantidade de acessos e de erros, esses arquivos podem ficar enormes. O ideal é estarem sempre vazios. Portanto, qualquer erro pequeno, sempre corrija o mais rápido possível
*/
ini_set('error_log', BASE_DIR.'..'.DS.'logs'.DS.'php'.DS.'PHP_errors-'.date('Ym').'.log');
Gambiarra, invocando método não estático num método estático
class Foo {
public function Bar() {
}
public static function Bar2() {
$c = new Bar;
return $c->Bar();
}
}
Foo::Bar2();
O problema aqui é que não faz sentido. Está criando apenas uma faxada para tornar o código mais "elegante" e apenas consumindo memória e processamento desnecessário.
Uma solução é criar as definições de forma consistente com o objetivo final do que precisa ser usado dessa classe.
Evite gambiarras e ocultar erros do tipo STRICT e DEPRECATED. Normalmente, erros nesse nível, em curto ou médio prazo tornam-se de nível FATAL.
Crud
? Tem o código dela?