PHP OO ¿Dudas con la POO?

[Vaneek]

Hola a todos he estado aprendiendo php hace poco tiempo pero ahora que estoy entrando en la POO hay algunas cosas que tengo duda, si alguien me puede explicar que estoy haciendo mal en este código, se lo agradecería mucho.

Código PHP:

  <?php
class DataBase{

public $connect_db;

function Connect($sername, $dbuser, $dbpass, $dbname){
$this->connect_db = mysqli_connect($sername, $dbuser, $dbpass, $dbname);
return $this->connect_db;
}

function QueryInsert($sql, $table_name){
$query_sql = 'INSERT INTO '.$table_name.'('.implode(', ', array_keys($sql)).')'.' VALUES('.implode(', ',$sql).')';
$query = mysqli_query($this->connect_db, $query_sql);
return $query;
}
}
?>   

Solo es conectar a la base de datos y insertar datos usando un formulario nada más y no se que hago mal por que no me inserta el dato

Tengo un archivo llamado global.php y en este llamo al objeto

Código PHP:

  <?php
require_once("source/config.php");
require_once("source/oop/class_database.php");

$db = new DataBase;
$db->Connect(Sername, Dbuser, Dbpass, Dbname);
?>   

en config.php tengo constantes para conectar a la base de datos y en mi archivo index.php

Código PHP:

  <?php
require_once "global.php"; 

if(!isset($_GET['values'])){
?>
<form action="index.php" method="GET">
<input type="text" name="values"/>
<input type="submit" value="Enviar"/>
</form>
<?php
}else{
$sql = array('name' => $_GET['values']);
$db->QueryInsert($sql, 'config');
}
?>   

Y aclaro que no me da ningún erro en la sintasys
Espero su ayuda

[SetheR]

Hola. ¿Que error obtienes?

Mientras estén en un entorno de desarrollo activa el error_reporting (ALL) y el display_errors en tu php.ini para poder observar todos los errores y warnings que te arroja la aplicación.

Así simple vista no sabría decirte sin ver el resto del código. Revisa bien las constantes definidas que utilizas en tu método conexión

PD: ya que utilizas POO, te recomiendo utilizar mysqli en su orientación a objetos para acelerar tu aprendizaje.

Saludos

[GatorV]

Yo lo que veo es que el segundo parámetro de tu método QueryInsert debe de ser un array, y tú le estas pasando un string, por eso no funciona correctamente, pero sigue los consejos de SetheR, así puedes ver los errores que te marque.

Saludos.

[Vaneek]

Iniciado por SetheR

Hola. ¿Que error obtienes?

Mientras estén en un entorno de desarrollo activa el error_reporting (ALL) y el display_errors en tu php.ini para poder observar todos los errores y warnings que te arroja la aplicación.

Así simple vista no sabría decirte sin ver el resto del código. Revisa bien las constantes definidas que utilizas en tu método conexión

PD: ya que utilizas POO, te recomiendo utilizar mysqli en su orientación a objetos para acelerar tu aprendizaje.

Saludos

Ya los tengo activados y en realidad no me aparece ningún error y las constantes estas definadas así.

Código PHP:

  <?php
const Sername = "localhost";
const Dbuser  = "root";
const Dbpass  = "";
const Dbname  = "proyecto";
?>   

Y mi servidor local trabaja con php 5.4 no creo que sean las constantes.

[abimex]

las constantes se definen con la funcion de php define

Código PHP:

define('Sername',"localhost");
// etc
 

si tienes los errores activados deberian aparecer esos errores, por favor revisa tu configuración

[Vaneek]

Iniciado por abimex

las constantes se definen con la funcion de php define

Código PHP:

define('Sername',"localhost");
// etc
 

si tienes los errores activados deberian aparecer esos errores, por favor revisa tu configuración

Que no es lo mismo?? según la documentación de php esta forma se puede aplicar desde php 5.3.

Aunque aun cambiando a define no me cambia el resultado, es decir que no me hace nada, ningún error ni nada.

[carlos_belisario]

pues concuerdo con setter, deberias sacar ventaja a mysqli orientado a objetos, de todas maneras siempre es bueno ver si hay algun error y lanzar una excepcion, puedes ver como usar mysqli::error, con eso pudieras lanzar una excepcion, saludos

[abimex]

si estas en una clase si se puede hacer esto

Código PHP:

class MiClase{
   const CONST = 'Valor';
}
 

pero fuera del contexto de una clase, las constantes se definen con define,
y por favor revisa que esté activado la salida de errores, si no te dice nada pega esto al inicio de tu archivo para que veas los errores.

Código PHP:

ini_set('display_errors', 1);
error_reporting(E_ALL);
 

[masterpuppet]

En realidad a partir de PHP 5.3+ las constantes se pueden definir de las formas, ahora en relación a tu problema, has debugeado para ver donde "muere" el script ?, si es que muere, mysqli_error no te indica nada ?

[GatorV]

Yo solo veo que no haz hecho lo que te dije , estas mandando llamar tu método de forma incorrecta.

[masterpuppet]

@GatorV, talves estoy dormido(puede que sea la hora :P), pero el segundo parámetro no es el nombre de la tabla ?

[GatorV]

Cierto jaja creo que el dormido soy yo xD

[masterpuppet]

Jeje, suele suceder, yo miraba y miraba el código :P.
@Vaneek, hay alguna razón para utilizar GET en el formulario.

[Vaneek]

Iniciado por carlos_belisario

pues concuerdo con setter, deberias sacar ventaja a mysqli orientado a objetos, de todas maneras siempre es bueno ver si hay algun error y lanzar una excepcion, puedes ver como usar [URL="http://www.php.net/manual/es/mysqli.error.php"]mysqli::error[/URL], con eso pudieras lanzar una excepcion, saludos

El problema no es si quiero usar de una u otra forma para la base de datos si no que es aprender a usar la POO y la mejor forma es hacer una estructura de base de datos funcional, aunque eso pienso yo y no es necesario que los demás piensen igual.

Iniciado por GatorV

Yo solo veo que no haz hecho lo que te dije , estas mandando llamar tu método de forma incorrecta.

Sí, como te dicen arriba es solo es el nombre de la tabla donde inserto el dato.

Iniciado por masterpuppet

Jeje, suele suceder, yo miraba y miraba el código :P.
@Vaneek, hay alguna razón para utilizar GET en el formulario.

Ninguna, es una pura prueba y la verdad erra por que no quería poner más script y como es prueba no sirve para nada xD.

Pero ya vi que erra el erro y no es mi código, o bien que sí xD pero me fije que en la consulta SQL no había puesto un simple carácter y es la comillas simple, por eso no me daba ningún problema en el script y al poner mysqli_error me daba que le faltaba un parámetro, así que lo puse bien y funciona perfectamente.

Gracias por la ayuda, aprenderé a hacer script con un sistema de error desde ahora que es algo común en mi caso ya que la mayoría de script que hago no me dan errores xD(claro que son scripts simples)