Comentarios:
- En realidad tu clase no hace más que encerrar un algoritmo que ahora necesitas .. necesitas abstraer más tu objeto.
- Tu clase debe estar bien definida, debe hacer una cosa. ¿Se supone trae información del visitante?, pues hasta ahí; el envío por correo está un tanto fuera de la función de esta clase. A eso me refiero con que ahora solo encierras un algoritmo que ahora necesitas.
- Tu destructor está eliminado variables locales a ese método, variables que no existen. Vaya, que no está haciendo nada ... ojo.
- Si la función es para obtener datos, hasta ahí. Piensa en la reutilización de código; cosas como estas no es recomendable:
- Evita silenciar errores directamente en las sentencias, osea, evita utilizar el operador @. Para quien intente utilizar tu código, esa práctica complicaría el debuggueo (...).
- Y, lo malo, estás presuponiendo cosas, como que solo existen 2 lenguajes, 2 sistemas operativos y 5 exploradores, cosa más alejada de la realidad. Piensa genérico.
- Cuidado con tu sintaxis. Procura seguir un estándar conocido, PEAR, Zend (¿recomendado?), eZ, Horde ... pero deberías adaptarte a uno.
Solo comentarios para mejorar tu código. Mis 2 centavos
