Problema con Exception disparado dentro de __toString()

[ArrauKano]

Bueno, tengo clarísimo que dentro de un método __toString no pueden dispararse excepciones, pero...

Tengo un stack de objetos que pretendo escapar como string, haciendo simplemente un implode(), de tal modo que el casting llame automáticamente a la salida en forma de texto de cada objeto y los junte en una sola cadena.

Todos los objetos del stack son hijos de "Widget", y mi clase Widget tiene un método __toString() que llama a otro método getContent().

Código php:

abstract class Rox_Wigdet
{
 
    public function getContent()
    {
        return 'Contenido';
    }
 
    final public function __toString()
    {
        try {
            return (string) $this->getContent();
        } catch ( Rox_Widget_Exception $e ) {
            trigger_error($e->getMessage(), E_USER_WARNING);
            }
        return '';
    }
}
 

La idea es que cada widget pueda definir su contenido y que siempre se dibujen de igual manera.

Para prevenir que el contenido del objeto dispare cualquier excepcion, pensé en rodear todo el __toString con un bloque try...catch.

Pensé que estaba todo bien, pero hoy al modificar un widget, tuve un problema que disparó una excepcion, y descubrí que por algún motivo, la excepcion simplemente se saltó el bloque .

Acá hice un ejemplo reproducible:

Código php:

<?php
 
class A {
 
    public function output()
    {
        return ' algo A ';
    }
 
    public function __toString()
    {
        try {
            $out = $this->output();
        } catch ( Exception $e ) {
            trigger_error($e->getMessage(), E_USER_WARNING);
            $out = $e->getMessage ();
        }
       return $out;
    }
}
 
class B extends A {
    public function output()
    {
        return ' algo B ';
    }
}
 
class C extends B {
    public function output()
    {
        throw new Exception('boooooooo!');
        return ' algo C ';
    }
}
 
 
/** Prueba: */
 
$stack = array(
    new A(),
    new B(),
    new C()
);
 
echo implode( $stack );
 

[GatorV]

Mmm esque según lo que tengo entendido es que __toString() devuelva el contexto actual en un string de la clase, no que sea __toString() el encargado de devolver la representación de texto del objeto.

Es un poco complicado entenderlo pero la idea es por ejemplo, construyes tu objeto, llamas a algunos métodos para que haga algo, y al final llamas a __toString() este te tiene que devolver el estado actual, no es __toString() el encargado de realizar ultimos procesos de salida o algo.

Tu idea de sobreescribir un solo método (por ejemplo output()) no es la más correcta ya que __toString() no es una función terminadora, es mas bien un informante del estado actual.

Saludos.

[ArrauKano]

Entonces, la idea sería llamar a método "ejecutor" de cada objeto "Widget" en el stack, guardar el output en una propiedad y solo retornar el string, para poder concatenar todo con el implode() ?

Igual sería algo latoso.

Pero independiente de la técnica a seguir, en ningún lado del manual vi ese comportamiento de __toString(), porque simplemente no le hace caso al bloque try...catch().

Además, juraría que en algún minuto me funcionó (una versión más antigua de php tal vez).

[GatorV]

Mmm puede que sea algo que hayan desactivado en alguna versión de PHP, por ende el error, esque antes si lees el Manual __toString() solo era llamado en por ejemplo echo, o print, ahora es llamado en cualquier lugar donde se haga el casting a string, entonces por ejemplo str_replace() si le pasas el objeto y te lanza una excepcion deja al interprete en un estado inconsistente, por ende desactivaron el uso de lanzar excepciones y mejor tirar un warning.

Saludos.

[enriqueplace]

El problema que estoy viendo es que intentas crear un "cañón destructor de planetas" dentro de un "excusado", y este se creó para otra cosa.

Crea métodos, si quieres que todos hagan lo mismo, create una interfaz para definir los métodos de los Widget's y la clase que los use solo acepte de tipo la interfaz, deja el toString para lo que es, cuando lo quieres convertirlo a texto (si no te sirve que haga algo en ese contexto, no lo hagas).