PHP OO [APORTE]toma los datos del visitante (ip navegador lenguaje so )

[mcun]

Bueno les dejo una clase en la que aún estoy trabajando ( para ampliara ) es parte de un proyecto en el que trabajo, la comparto pues de esta web he aprendido mucho y es mi forma de decirles gracias y porque creo en el software libre

Código PHP:

<?php
/*
 *
 *
 *      Copyright 2010 Mauricio Rodriguez <[email protected]>
 *
 *      This program is free software; you can redistribute it and/or modify
 *      it under the terms of the GNU General Public License as published by
 *      the Free Software Foundation; either version 2 of the License, or
 *      (at your option) any later version.
 *
 *      This program is distributed in the hope that it will be useful,
 *      but WITHOUT ANY WARRANTY; without even the implied warranty of
 *      MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 *      GNU General Public License for more details.
 *
 *      You should have received a copy of the GNU General Public License
 *      along with this program; if not, write to the Free Software
 *      Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
 *      MA 02110-1301, USA.
 */
 
class Get_data_vistor{
    // atributos
    private $frowarded_for;
    private $http_via;
    private $remote_addr;
    private $lang=array();
    private $so=array();
    private $browser=array();
    private $user_agent_conv_lang;
    private $user_agent_conv_so;
    private $user_agent_conv_browser;
    function  __construct() {
        // tomo la ip filtrando proxie
        $this->frowarded_for=isset($_SERVER['HTTP_X_FORWARDED_FOR']);
        $this->http_via=(isset($_SERVER['HTTP_VIA']));
        $this->remote_addr=(isset($_SERVER['REMOTE_ADDR']));
        // constructores de lenguage
        $this->lang[0]="es";
        $this->lang[1]="en";
        $this->lang[2]="fr";
        // constructores de so
        $this->so[0]="linux";
        $this->so[1]="windows";
        // constructores de browser
        $this->browser[0]="mozilla";
        $this->browser[1]="explorer";
        $this->browser[2]="chrome";
        // convierto a minúsculas 
        $this->user_agent_conv_lang=strtolower($_SERVER[HTTP_ACCEPT_LANGUAGE]);
        $this->user_agent_conv_so=strtolower($_SERVER[HTTP_USER_AGENT]);
        $this->user_agent_conv_browser=strtolower($_SERVER[HTTP_USER_AGENT]);
        }
            // metodos
    public  function get_ip_v(){
        if ($this->frowarded_for) {
            $ip = $_SERVER['HTTP_X_FORWARDED_FOR'];
        }elseif ($this->http_via)  {
            $ip = $_SERVER['HTTP_VIA'];
        }elseif ($this->remote_addr) {
            $ip = $_SERVER['REMOTE_ADDR'];
        }
        echo $ip;
        return $ip;
    }// End function get_ip()
        public function get_lang_v(){
            if (@strpos($this->user_agent_conv_lang, $this->lang[0]) !== false){
                $lang= "Espa&ntilde;ol";
            }
            if (@strpos($this->user_agent_conv_lang, $this->lang[1]) !== false){
                $lang= "Ingles";
            }
            echo $lang;
            return $lang;
        }
        public function get_so_v(){
            if (@strpos($this->user_agent_conv_so, $this->so[0]) !== false){
                $so= "linux";
            }
            if (@strpos($this->user_agent_conv_so, $this->so[1]) !== false){
                $so= "windows";
            }
            echo $so;
            return $so;
        }// end functionget_so_v
        public function get_browsrer_v(){
        // verifico que se encuentre el nombre del explorador en la
        // variable con strpos
        if(strpos($this->user_agent_conv_browser, $this->browser[0]) !==false){
            $browser="mozilla";
        }
        if(strpos($this->user_agent_conv_browser, $this->browser[1]) !==false){
            $browser="explorer";
        }
 
        if(strpos($this->user_agent_conv_browser, $this->browser[2]) !==false){
            $browser="chrome";
        }
        if(strpos($this->user_agent_conv_browser,"opera") !==false){
            $browser="opera";
        }
        if(strpos($this->user_agent_conv_browser,"mirodi") !==false){
            $browser="mirodi";
        }
        echo $browser;
        return $browser;
    } // end function get_browser_v
    public function up_mail_data_v(){
        $ip=  $this->get_ip_v();
        $so=  $this->get_so_v();
        $lang=  $this->get_lang_v();
        $browser=  $this->get_browsrer_v();
        // Varios destinatarios
        $para  = '[email protected]';
        // subject
        $titulo = 'data visitor';
        // message
        $mensaje = '
        <html>
            <head>
                <title>datos del visitante</title>
            </head>
            <body>
                <p>¡Estos son los datos tomados del visitante</p>
                <table>
                    <tr>
                       <th>SO</th><th>Navegador</th><th>IP</th><th>lenguage</th>
                    </tr>
                    <tr>
                       <td>'.$so.'</td><td>'.$browser.'</td><td>'.$ip.'</td><td>'.$lang.'</td>
                    </tr>
                    <tr>
                    </tr>
                </table>
             </body>
        </html>';
        // Almaceno el la BBDD los datos
        $date = date("Y-m-d");
        $server='xxx';
        $username='xxx';
        $password='xxx';
        $db='data';
        $con=mysql_connect($server, $username, $password);
        mysql_select_db($db, $con);
        $sql="INSERT INTO visitante ";
        $sql .="(id,ip,so,lang,browser,date)";
        $sql .="VALUES ('NULL','$ip','$so','$lang','$browser','$date ')";
        $res = mysql_query($sql,$con);
    if (!$res) {
        die("ATENCION!!! No fue posible insertar datos:<br />".mysql_error());
    }
        // Envio el mail co el reporte del visitante
        // Para enviar un correo HTML mail, la cabecera Content-type debe fijarse
        $cabeceras  = 'MIME-Version: 1.0' . "\r\n";
        $cabeceras .= 'Content-type: text/html; charset=iso-8859-1' . "\r\n";
        // Cabeceras adicionales opcionales comentadas
        //$cabeceras .= 'To: xxxx <[email protected]>, xxxx <[email protected]>' . "\r\n";
        $cabeceras .= 'From: Recordatorio <[email protected]>' . "\r\n";
        //$cabeceras .= 'Cc: [email protected]' . "\r\n";
        mail($para, $titulo, $mensaje, $cabeceras);
        }
    function  __destruct() {
        unset ($browser ,$lang, $so ,$ip);
    }
 
}// end calss get_data_visitor
echo 'class get_data_visitor <br />';
$t=new Get_data_vistor();
$t->get_lang_v();
echo '<br />';
$t->get_so_v();
echo '<br />';
$t->get_browsrer_v();
echo '<br />';
$t->get_ip_v();
$t->up_mail_data_v();
?>
 

[maturano]

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:

Código PHP:

echo $ip;
        return $ip;
 

- 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

[GatorV]

Aparte de lo que dice Maturano PHP dispone de una función get_browser la cual te puede ayudar y dar muchos más datos que los que te da tu clase.

Saludos.

[mcun]

muchas gracias por los aportes .. tomo notas de ellos...

lo de la redundancia echo return es solo para mi prueba se me paso....

lo del mail si definitivamente debería llamar a un clase para ello o hacerlo de forma inversa mmm tal vez heredar de esta clase desde una case mail.......


que doy por echo que existen x sistemas operativos y x navegadores .... te refieres a que no los liste todos o que debería tratarlos de modo mas abstracto si es lo ultimo no ve como.....pensar en ello y también en lo de la sintaxis y destructor ( no entiendo por que no destruye nada)

gracias nuevamente maturano

GatorV gracias por el link lo chequeo... supongo que esta lleno de cosas que ya existen que uno intenta hacer todo el tiempo en fin gracias a ambos muy valiosas las criticas

[GatorV]

El destructor no destruye nada porque estas borrando variables que no estan en el alcance del destructor:

Código PHP:

public function __destruct()
{
      unset($browser);
}
 

Ahí $browser nunca existio, quiza te refieras a $this->browser...

Saludos.

[mcun]

Iniciado por GatorV

Ahí $browser nunca existio, quiza te refieras a $this->browser...

Saludos.

si esa era mi intención para no repetir los destructores gracias nuevamente....

PD: ya estoy con la tarea

[maturano]

Iniciado por mcun

lo de la redundancia echo return es solo para mi prueba se me paso....

Cuidado con ello, igual busca mejores maneras de hacer pruebas ... tan simple como el hecho de imprimir el valor retornado, que es lo que te interesa saber; total, es exactamente el mismo valor.

Iniciado por mcun

lo del mail si definitivamente debería llamar a un clase para ello o hacerlo de forma inversa mmm tal vez heredar de esta clase desde una case mail.......

¿Herencia?; ERROR.

La programación orientada a objetos no es solo la utilización de clases. No pienses en tu código, en tu algoritmo, en una clase para lograr tu objetivo como ahora lo haces. Tienes que leer los conceptos, básico ...

Iniciado por mcun

que doy por echo que existen x sistemas operativos y x navegadores .... te refieres a que no los liste todos o que debería tratarlos de modo mas abstracto

Código como este:

Código PHP:

public function get_so_v(){
            if (@strpos($this->user_agent_conv_so, $this->so[0]) !== false){
                $so= "linux";
            }
            if (@strpos($this->user_agent_conv_so, $this->so[1]) !== false){
                $so= "windows";
            }
            echo $so;
            return $so;
        }// end functionget_so_v
 

Tema aparte de tu metodología (que igual habría que comentar), lo único que haces es comparar si un valor obtenido para el SO es "linux" o "windows". Viéndolo como código, en el caso que ninguna de las condiciones se cumpla tienes una variable no definida --error--, viéndolo como función de una clase, ese método no cumple su función de decirme qué SO se está utilizando, solo me dice si es Windows o Linux; ¿es esta tu intención?, entonces no crees que sería mejor crear métodos isWindows() e isLinux() que te retornen un booleano?.

Y no es solo caso de añadir "Mac" a tu lista, menos ahora con tanto "gadget" que sale a cada rato; Android --y variantes--, Moblin, Symbian, Palm OS, iPhone OS, Darwin, BSD, etc., etc., etc. ... no vas a enlistar todos los SOs, ¿o sí?

Iniciado por mcun

y destructor ( no entiendo por que no destruye nada)

De nuevo: tu destructor está eliminando variables locales a ese método, variables que no están definidas, por ende, no está haciendo nada. Sigue tu propia metodología e imprime esas variables antes de eliminarlas ...

Además, si se destruye un objeto se va con todo y sus propiedades. Si tu intención es limpiar las propiedades del objeto --que así se entiende--, no tiene ningún caso. De nuevo, conceptos ...


Mi sugerencia:
Ve la función que te dejó GatorV, extrae toda la información que puedas o necesites, todo dentro del constructor y seteando a variables privadas por cada dato, luego entonces, solo necesitas getters para los valores que vayas a necesitas o métodos propios que puedan corresponder a tu objeto, como quizá un esLinux() como mencioné, quizá un esDispositivoMovil() que quizá podría interesar a quien utilize tu clase ... etc. etc. etc.


Saludos

[maturano]

Iniciado por GatorV

El destructor no destruye nada porque estas borrando variables que no estan en el alcance del destructor:

Código PHP:

public function __destruct()
{
      unset($browser);
}
 

Ahí $browser nunca existio, quiza te refieras a $this->browser...

Saludos.

A eso me refería por una parte, la del código, pero igual no tiene sentido esa acción, pues si se destruye el objeto, se va con todo y sus propiedades, ¿qué no?.

Aunque se corrija el código, el destructor seguiría sin hacer mucho --ya no nada --.

[mcun]

maturano tus criticas son muy esclarecedoras gracias, acabo de encontrar un articulo gracias a tu critica sobre mi metodología de trabajo que me represento un buen ejemplo de como debería se la opp yo sin darme cuenta sigo programado de modo procedimental aunque usando "php5" .

bien voy a re-formular mi metodológica.

voy a programar una sola cosa pero que lo haga bien ( donde escuche eso....) y usando un MVC

saludos

edito........

no pude poner el link en bbcode

http://arleytriana.blogspot.com/2009/07/implementacion-del-patron-clasico-de.html

[maturano]

... el artículo se refiere a un patrón de diseño, es independiente del paradigma de programación utilizado. De hecho, dicho artículo no utiliza objetos, sino programación estructurada (y que me parece medio mareador ... pero será cuestión de gustos).

Solo la observación, que la lectura no te hará daño y tiene algo de lo que mencioné aquí: separación clara de funciones, sin embargo a lo que me refería era a la POO, no a patrones.

Saludos

[mcun]

si le falta POO el autor lo menciona pero el patrón me parecio simple... luego me queda pensarlo desde POO

[GatorV]

Tema movido desde PHP orientado a objetos a PHP