insertar PHP POO MVC

[lokito1986]

bueno ante todo gracias por darse el tiempo para leer este mensaje ..me gustaria que me ayudaran con este probemita ..miren esta cuestion de (altas bajas modifcaciones ) no se me hace muy dificil ...el punto k ahora lo estoy tratando de llevar pero a la POO MVC el cual estoy aprendiendo enforma autodidacta ....me gustaria saber porque no inserta correctamente en este ejemplo ..de antemano gracias por su tiempo ...pasen buenas tardes ...

Conexion.php

Código PHP:

<?php
class Conexion{
    
    private static $servidor="localhost";
    private static $usuario="root";
    private static $clave="root";
    private static $bd="vargas";
    
    public static function Conectarse(){
    $link=@mysql_connect(self::$servidor,self::$usuario,self::$clave);
    
    if(!$link){
        echo "Error conectando a la base de datos ".mysql_error();
        exit();
        }
    
    if(!mysql_select_db(self::$bd,$link)){
        echo "Error seleccionando la base de datos.".mysql_error();
        exit();
        }
    
    return $link;
    }
    
    public function Ejecutar_consulta($cadena){
        $res=mysql_query($cadena,self::Conectarse());
        if(!$res){
            echo "Error al Ejecutar Consulta : ".mysql_error();
            exit();
            }
        return $res;
        }
    
    }
 

Persona.php

Código PHP:

<?php
class Persona{
        
    private $codper;
    private $nombre;
    private $apellidos;
   
 
    public function getCodper() {
        return $this->codper;
    }
 
    public function setCodper($codper) {
        $this->codper = $codper;
    }
 
    public function getNombre() {
        return $this->nombre;
    }
 
    public function setNombre($nombre) {
        $this->nombre = $nombre;
    }
 
    public function getApellidos() {
        return $this->apellidos;
    }
 
    public function setApellidos($apellidos) {
        $this->apellidos = $apellidos;
    }
 
 
    
    }
 

PersonaDAO.php

Código PHP:

<?php
include_once("../Clases/Conexion.php");
include_once("../Clases/Persona.php");
 
 
class PersonaDAO {
    
    public function GrabarPersona(Persona $op) {
        $objcon=new Conexion();
        $cadena= "insert into persona(nombre,apellido)values($op->getNombre(),$op->getApellidos())";
        $res=$objcon->Ejecutar_consulta($cadena);
        return $res;
    }
 
}
 
 
?>
 

y este es un archivo de prueba lo hice de esta forma para poder ver si inserta o no ...una vez k inserta lo modifico con su respectivo formulario con cajas de texto

Prueba.php

Código PHP:

<?php
include_once("../DAO/PersonaDAO.php");
include_once("../Clases/Persona.php");
$miobjetoConexion=new Conexion();
$objPersona=new Persona();
$objPersona->setNombre("juan ");
$objPersona->setApellidos("Perez");
 
$objDAO=new PersonaDAO();
 
$rpta=$objDAO->GrabarPersona($objpersona);
 
 
 
 
 
if($rpta){
    echo 'consulta Exitosa : ';
}
 

el problema es que cuando quiero insertar No guarda los datos en mi Bd de MYSql ..diganme que estoy haciendo mal ..gracias

[masterpuppet]

Si ejecutas el SQL generado en phpmyadmin o alguno similar, funciona correctamente ? y te sugiero que en vez de crear un wrapper al set de funciones mysql_* utilices PDO.

Saludos.

[lokito1986]

ante todo gracias por tu rpta amigo masterpuppet ....mira el prblema que tengo es que no puedo insertar los datos en mi Bd stoy utilizando phpmyadmin , cuando quiero grabar los datos no lohace ....si e tenido pensado aprender PDO ...pero antes de aprender PDO me e propuesto aprender 3 capas MVC php empezar desde basico e ir aumentando de dificultad ..el ejemplo mostrado es un inicio de este nuevo aprendizaje k pienso realizar ...

gracias por tu rpta amigo ..y tomare muy en cuenta tus consejos ..pero antes que nada me gustaria ue me ayudaras a resolver porque no graba en mi BD ..please

[lokito1986]

buenos dias a todos me gustaria saber porque no inserta los datos en mi BD ..soy nuevo en esto de php POO MVC ...por favor si alguien puede corregir mis errores que estoy cometiendo en este ejemplo ...gracias de antemano..

Conexion.php

Código PHP:

<?php
class Conexion{
    
    private static $servidor="localhost";
    private static $usuario="root";
    private static $clave="root";
    private static $bd="vargas";
    
    public static function Conectarse(){
    $link=@mysql_connect(self::$servidor,self::$usuario,self::$clave);
    
    if(!$link){
        echo "Error conectando a la base de datos ".mysql_error();
        exit();
        }
    
    if(!mysql_select_db(self::$bd,$link)){
        echo "Error seleccionando la base de datos.".mysql_error();
        exit();
        }
    
    return $link;
    }
    
    public function Ejecutar_consulta($cadena){
        $res=mysql_query($cadena,self::Conectarse());
        if(!$res){
            echo "Error al Ejecutar Consulta : ".mysql_error();
            exit();
            }
        return $res;
        }
    
    }
 

Persona.php

Código PHP:

<?php
class Persona{
        
    private $codper;
    private $nombre;
    private $apellidos;
   
 
    public function getCodper() {
        return $this->codper;
    }
 
    public function setCodper($codper) {
        $this->codper = $codper;
    }
 
    public function getNombre() {
        return $this->nombre;
    }
 
    public function setNombre($nombre) {
        $this->nombre = $nombre;
    }
 
    public function getApellidos() {
        return $this->apellidos;
    }
 
    public function setApellidos($apellidos) {
        $this->apellidos = $apellidos;
    }
 
 
    
    }
 

PersonaDAO.php

Código PHP:

1.
      <?php
   2.
      include_once("../Clases/Conexion.php");
   3.
      include_once("../Clases/Persona.php");
   4.
       
   5.
       
   6.
      class PersonaDAO {
   7.
         
   8.
          public function GrabarPersona( $op) {
   9.
              $objcon=new Conexion();
  10.
              $cadena= "insert into persona(nombre,apellido)values($op->getNombre(),$op->getApellidos())";
  11.
              $res=$objcon->Ejecutar_consulta($cadena);
  12.
              return $res;
  13.
          }
  14.
       
  15.
      }
  16.
       
  17.
       
  18.
      ?>
 

Prueba.php

Código PHP:

<?php
include_once("../DAO/PersonaDAO.php");
include_once("../Clases/Persona.php");
$miobjetoConexion=new Conexion();
$objPersona=new Persona();
$objPersona->setNombre("juan ");
$objPersona->setApellidos("Perez");
 
$objDAO=new PersonaDAO();
 
$rpta=$objDAO->GrabarPersona($objpersona);
 
 
 
 
 
if($rpta){
    echo 'consulta Exitosa : ';
}
 
 
a la hora que quiero insertar NO GUARDA LOS DATOS EN LA BD ...que estoy haciendo mal please...
 
posdata : se que tal vez lo mejor sea utilizar PDO ...pero la cuestion es que quiero aprender PHPsin pdo .... luego de aprender bien MVC voy a ir aprendiendo pdo y cosas de mas nivel ..
gracias a todos
 

[DooBie]

Ayudaría ver el error que te da, postealo.

Por cierto, te has asegurado que la sentencia sql que envias en la que quieres no? es decir, con los datos correctos etc...

[lokito1986]

claro amigo esa es la consulta mi tabla persona tiene 3 campos codpersona,nombre,apellido el prblema es que cuando ejecuto no guarda en la bD

[DooBie]

No me refería a los campos de la tabla, sino el formato de la consulta, con los valores correctos, comas, comillas, paréntesis, etc...

Por cierto, que error te sale? o simplemente no inserta los datos?

[maycolalvarez]

sinceramente tienes tantos errores de implementación, entre los cuales

tienes una clase persona con la cual no veo que hagas nada, francamente una abstracción inútil si no se usa un orm como doctrine2, a no ser que phpDAO lo implemente, lo cual ignoro, al final escribes mucho código llamado getter por getter, cuando la idea es mapearlo

la clase de conexión ni siquiera es un singleton, con lo cual generas y generas conexiones a lo largo del script las cuales nunca usas, además de que la función estática ejecutar_consulta vuelve a llamar a conexión generando otro resource connection, lo que lo hace pesado, al menos si existiera un singleton la cosa funcionaria mejor

te recomiendo leer acerca de patrones de diseño, y que hagas una corrida en frío detallada para que notes la cantidad de conexiones innecesarias que haces, además si existe un clase persona, porque la clase personaDAO?????

[masterpuppet]

No entiendo el razonamiento que estas haciendo sobre PDO, es reemplazar tu implementacion por PDO, es la misma capa, lo que no tiene sentido hoy en día es crear un wrapper.
Ahora para ayudarte deberías indicar que error te da, se ejecuta la query ?, la query esta bien formada ?

@maycol, esta intentado implementar DAO.

[lokito1986]

ante todos gracias por darse Un tiempo para ller este mensaje ..pero lo k pasa que soy un novato que estoy programando en PHP ...
bueno al intentar grabar no se puede y sale este error ...
Categoria si inserta Normal ..el problema viene cuando intento grabar UN PRODUcTO
Estoy Haciendo Lo correcto en progrmar de Esta forma ..si no es asi me gustaria que me hiceran ver donde ESTAN MIS HORRORES xD..

Error MOSTRADO :

Fatal error: Call to a member function EjecutarConsulta() on a non-object in C:\AppServ\www\trabajoMadrugada\Clases\Producto.ph p on line 34

Categoria.php

Código PHP:

<?php
 
require_once 'Conexion.php';
class Categoria {
    private $codigoCategoria;
    private $_nombre;
    private $_descripcion;
    private $_objcon;
    
    public function getCodigoCategoria() {
        return $this->codigoCategoria;
    }
 
    public function setCodigoCategoria($codigoCategoria) {
        $this->codigoCategoria = $codigoCategoria;
    }
 
        function __construct() {
        $this->_objcon=new Conexion();
    }
 
    public function get_nombre() {
        return $this->_nombre;
    }
 
    public function set_nombre($_nombre) {
        $this->_nombre = $_nombre;
    }
 
    public function get_descripcion() {
        return $this->_descripcion;
    }
 
    public function set_descripcion($_descripcion) {
        $this->_descripcion = $_descripcion;
    }
 
public function grabarCategoria(){
    $query="insert into categoria(nombre,descripcion)values('$this->_nombre','$this->_descripcion')";
    $resultado = $this->_objcon->EjecutarConsulta($query);
    return $resultado;
}
} 
?>
 

Producto.php

Código PHP:

<?php
require_once 'Categoria.php';
 
class Producto {
    private $nombre;
    private  $objCat;
    private $objcon;
    
    
    function __construct() {
        $this->objcon=new Conexion();
        $this->objCat=new Categoria();
    }
    public function getNombre() {
        return $this->nombre;
    }
 
    public function setNombre($nombre) {
        $this->nombre = $nombre;
    }
 
    public function getObjCat() {
        return $this->objCat;
    }
 
    public function setObjCat(Categoria $objcat) {
        $this->objCat=$objcat;
    }
 
    public function grabarProducto(){
      
        $micod=  $this->objCat->getCodigoCategoria();
    $query="insert into producto (nombre,codcategoria)values('$this->nombre','$micod')";
    $resultado = $this->_objcon->EjecutarConsulta($query);
    return $resultado;
}
 
}
 
?>
 

Prueba.php

Código PHP:

<?php require_once '../Clases/Producto.php';
 
$objproducto=new Producto();
$objcat=new Categoria();
$objcat->setCodigoCategoria("2");
$objcat->set_nombre("lacteos");
$objcat->set_descripcion("tiene Calcio");
 
$objproducto->setNombre("Leche ");
$objproducto->setObjCat($objcat);
 
$resp = $objproducto->grabarProducto();
 
if ($resp) {
  echo 'Grabo exitosamente';
}
else{
    echo 'fallo algo';
}
 
?>
 

[santiagoblanco]

Bueno yo tambien ando trabajando con PDO y tambien tengo un error parecido.Pero tu error
tal vez lo podamos solucionar mandandole los datos por paramentros de la siguiente forma:
Enviandole las variables por medio de parametros:
Un ejemplo!

$consulta = "INSERT INTO $dbTabla
(nombre, apellidos)
VALUES (:nombre, :apellidos)";
$result = $db->prepare($consulta);
if ($result->execute(array(':nombre' => $nombre, ':apellidos' => $apellidos))) {
print "<p>Registro creado correctamente.</p>\n";
} else {
print "<p>Error al crear el registro.</p>\n";
}

sobre los :
estariamos realizando el insert mediante parámetros (:parametro)
Tal vez sea eso.O tal vez sea eso.Prueba y me dices.
Y ami tambien me interesa saber cual es tu error!