E_NOTICE: ¿Cuán útil es REALMENTE reparar cada una?

Question

En primer lugar Sé que esta pregunta ha dado la vuelta más de una vez aquí:

• Why should I fix E_NOTICE errors?
• Why should I fix E_NOTICE errors? Pros and cons

Pero cuanto más que puedo solucionar todos E_NOTICEs (como la gente dice que debería) más me he dado cuenta de que:

• soy micro-optimización
• De hecho, estoy haciendo más código y hacer mi código más difícil de MANTENER y más lento

Tomemos un ejemplo:

Digamos que su utilizando el controlador de MongoDB PHP y tiene un objeto MongoDate en una var clase llamada ts dentro una clase que representa una sola fila en una colección en su base de datos. Ahora accede a esta var como: $obj->ts->sec pero PHP arroja un ajuste (E_NOTICE) porque ts en este caso no está definido como un objeto en sí mismo porque esta fila en particular no tiene un campo ts. Entonces piensas que esto está bien, este es el comportamiento deseado, si no se establece devuelve nulo y yo mismo me ocuparé de ellos fuera de los propios trabajos robóticos del intérprete (ya que envuelves esto en una función date() que simplemente devuelve 1970 si la var es null o a none-object).

Pero ahora arreglar que E_NOTICE como otro desarrollador realmente quiere que yo tenga CUALQUIER E_NOTICE es terribad y hace que el código sea más lento para no hacerlo de acuerdo con los errores. Así que hago una nueva función en la clase $obj llama getTs y me doy 3 líneas, literalmente, no hacer nada, sino comprobar si el ts var es un objeto MongoDate y devolverlo si es ...

POR QUÉ? ¿No puede PHP hacer esto perfectamente bien para mí dentro de su intérprete mucho más rápido que tener que hacerlo en el tiempo de ejecución de la aplicación en sí? Quiero decir que cada vez que tengo que agregar un toque inútil a mi código, prácticamente funciones vacías para detectar variables que realmente manejo con la capacidad de PHP de devolver null o verificar su instanceof cuando realmente lo necesito (cuando es vital para el operación y comportamiento de dicha función) y no me inicie en el isset() s He agregado alrededor de 300 líneas de isset() s, se está yendo de las manos. Tengo, por supuesto, tiene que hacer que esta getTs funciones porque no se puede hacer:

class obj{ 
    public $ts = new MongoDate(); 
} 

yo bien tienen que almacenar el ts dentro del __constructor (que no estoy muy contento con cualquiera, estoy usando un montón de magia tal como es) o usar una función para detectar si está configurado (lo que hago ahora).

quiero decir que entiendo por qué debería fijar:

• Indefinido vars
• propiedades Asignación de VARs no definidas (null VARs)
• errores constantes etc

pero si usted ha probado su código y usted sabe que es seguro y solo funcionará de la manera que desee, ¿cuál es el punto en la fijación de todos los undefined index o none-object er rors? ¿No está agregando un montón de funciones de isset() sy 2 líneas a su código en realidad micro-optimización?

Me he dado cuenta después de hacer que la mitad de mi sitio cumple con E_NOTICE que en realidad usa más CPU, memoria y tiempo ahora ... ¿de qué sirve lidiar con cada error E_NOTICE y no solo con los que SON errores?

Gracias por sus pensamientos,

Answer 1

Sea o no usted debe fijarlos es ciertamente discutible, y solo dependerá de la rentabilidad en su situación; por ejemplo, es más importante si el código tendrá una vida útil más larga, más desarrolladores, etc.

En general, suponiendo que sus funciones serán utilizadas (y mal utilizadas) por otra persona es la mejor práctica, por lo que debe hacer isset /! empty/is_object checks para dar cuenta de esto. A menudo, su código encontrará su camino en usos y situaciones para las que nunca lo pensó.

En lo que respecta al rendimiento, cada vez que se produce algún tipo de error - E_NOTICE incluido - el intérprete genera el controlador de errores, crea un seguimiento de la pila y formatea el error. El punto es que, ya sea que los informes o no, los errores siempre ralentizan la ejecución; por lo tanto, 2-3 llamadas a funciones para evitar un E_NOTICE mejorarán aún más su rendimiento.

Editar: Alternativas para el ejemplo anterior

no necesariamente crear objetos adicionales para evitar los errores; puedes evitarlos con gracia sin ellos. Aquí hay un par de opciones:

1) Función que se encarga de ts que faltan:

SpecialClass class { 

    funciton getTs() { 
     return !empty($this->ts) ? $ts->sec : false; 
    } 
} 

2) Tratar con TS que faltan en la plantilla/procedimiento:

if (!empty($obj->ts->sec)) { 
    //do something 
} 

me gusta especialmente empty() porque puede usarlo para reemplazar (isset($var) && ($var or 0 != $var //etc)), guardar múltiples llamadas/comparaciones y vaciar nunca arroja avisos para la var o atributo de destino. Arrojará un error si lo está llamando en una propiedad/miembro de una variable inexistente.

Answer 2

Sin duda hace hacer obtener un mejor rendimiento mediante el uso de isset(). Hice algunos puntos de referencia, no hace mucho tiempo, y solo los errores ocultos resultaron ser 10 veces más lentos.

http://garrettbluma.com/2011/11/14/php-isset-performance/

Dicho esto, el rendimiento por lo general no es un factor crítico en PHP. Lo que hace, personalmente me vuelve loco es silent errors.

Cuando el intérprete decide no marcará algo como un error (lo que podría conducir a la inestabilidad) es un problema enorme. PHP, en particular, tiene una tendencia a

• advierten sobre las cosas que debe error (por ejemplo, fallo de conexión con la base de datos) y
• emisión avisos acerca de las cosas que debe advierten (por ejemplo, intentar para acceder a un miembro de un objeto nulo).

Quizás soy demasiado obstinado en este tipo de cosas pero he sido mordido antes por estos silenciosos errores. Recomiendo siempre incluir E_NOTICE en el informe de errores.