PHP OO Pasar un parametro o variable a una clase

Slikp · 31 Octubre 2014, 20:30 PM

- Buenas estoy algo novato con esto de la poo con php y la verdad es que me a resultado dificil adaptarme todo era feliz cuando lo hacia estructurado y no usando la POO pero bueno, mi problema es el siguiente:

- Tengo el Siguiente Formulario.

Código (html4strict) [Seleccionar]


<!DOCTYPE html>
<html lang="es">
<head>
    <title>Control de Equipos</title>
    <meta charset="utf-8">
    <link rel="stylesheet" type="text/css" href="css/style.css">
</head>
<body>
 <div id="login-form">     
      <h3>Bases de Datos</h3>
        <fieldset>
            <form action="board_table.php" method="post">
                <center>
                    <select name="cod_empresa">
                        <option Value="HUYAP_N">Proyecto H</option>
                        <option Value="Zuat">Proyecto Z</option>
                        <option Value="Ricl">Proyecto C</option>
                    </select>
                </center>    
 
                <input type="hidden" name="session">
                <input id="sub" type="submit" value="Iniciar Conexión">
            </form>
        </fieldset>
</div>
</body>
</html>

- El cual posee un select con 3 option y deseo pasarle el valor del option a un .php que hara una conexion a un bd deacuerdo al indice pasado por el select pero me da el siguiente error.

Notice: Undefined index: cod_empresa in C:\xampp\htdocs\sismica\conexion.table.php on line 7

Notice: Undefined variable: ndatabase in C:\xampp\htdocs\sismica\conexion.table.php on line 7

Fatal error: Cannot access empty property in C:\xampp\htdocs\sismica\conexion.table.php on line 7

- El .php es el siguiente:

Código (php) [Seleccionar]


<?php 
    require_once 'conexion.table.php';
 
    if(isset($_POST['session']))
    {
        header('location:ready.php');
    }
?>

- Y este otro que contiene la clase conexion.

Código (php) [Seleccionar]


<?php
class Conexion
{
    public $ndatabase;
    public function conectar()
    {
        $this->$ndatabase = $_POST['cod_empresa'];
        $server = 'stmbielve';
        $database = $ndatabase;
        $username = 'root';
        $password = '';
        return $conexion = new PDO('sqlsrv:Server=' . $server . ';Database='. $database, $username, $password);
        $this->dbh->exec("SET CHARACTER SET utf8");   
    }   
}
?>

- Como podria pasarle el cod_empresa a la clase para que me haga la conexion :/ ?? de antemano muchas gracias por sus respuestas.

T. Collins · 31 Octubre 2014, 20:52 PM

Tienes la clase pero no el objeto, tienes que hacer lo siguiente:

Código (php) [Seleccionar]


$CONN = new Conexion();
$CONN->conectar($_POST['cod_empresa']);

Y cambiar la función conectar de la clase para que reciba el cod_empresa como argumento.

Slikp · 31 Octubre 2014, 21:38 PM

- Buenas T. Collins, no entendi muy bien... Que tengo que cambiar en la funcion conectar?... Esas dos lineas supongo que las tengo que poner asi ??:

Código (php) [Seleccionar]


<?php 
    require_once 'conexion.table.php';

        $CONN = new Conexion();
    	$CONN->conectar($_POST['cod_empresa']);
    	
    if(isset($_POST['session']))
    {
        header('location:ready.php');
    }
?>

- Estoy enfrascado cada vez entiendo menos :/ ...

T. Collins · 1 Noviembre 2014, 00:41 AM

Prueba así, después de $this-> la variable va sin el simbolo $

Código (php) [Seleccionar]

<?php
    class Conexion
    {
       public $ndatabase;
       public function conectar()
       {
           $this->ndatabase = $_POST['cod_empresa'];
           $server = 'stmbielve';
           $database = $this->ndatabase;
           $username = 'root';
           $password = '';
           return $conexion = new PDO('sqlsrv:Server=' . $server . ';Database='. $database, $username, $password);
           //$this->dbh->exec("SET CHARACTER SET utf8");  
       }  
    }
    ?>

La línea que he comentando

Código (php) [Seleccionar]

//$this->dbh->exec("SET CHARACTER SET utf8"); si la pones después del return nunca se va a ejecutar, de todas formas $this->dbh no está declarado en ningún sitio.

En el php principal, pon:

Código (php) [Seleccionar]

$CONN = new Conexion;
$conexion = $CONN->conectar();

Slikp · 3 Noviembre 2014, 13:55 PM

- Buenos dias T. Collins disculpa por escribir tan tarde, pero este fin de semana no tuve tiempo... Intente lo que me sugeristes cambiando algunas cosas y al parecer funciono muchas gracias sin embargo creo que a raiz de esa solucion se me esta presentando otro problema y es que yo tengo otro .php que me genera una busqueda y una insercion pero ahora no funcionan y no muestran tampoco mensajes de error simplemente no hacen nada.

- Si yo a la clase conexion le paso directamente la bd a la que me quiero conectar la funcion de insercion y mostrar si funcionan pero cuando lo hago como me sugeristes dejan de funcionar por ejemplo si lo hago asi sirve.

Código (php) [Seleccionar]

<?php
class Conexion
{

    public function conectar()
       {
           
            $server = 'sbvbieloven';
            $database = 'huyapari_n';
            $username = 'profit';
            $password = 'profit';
            return $conexion = new PDO('sqlsrv:Server=' . $server . ';Database='. $database, $username, $password);
    	} 
    
}
?>

- Así si funciona, obviamente no es la idea hacerlo de esa forma. En mi ignorancia no se si es porque la conexion no se mantiene despues de pasarle el cod_empresa desde el form te muestro las desmas funciones.

Código (php) [Seleccionar]

<?php 
	class Crud
{
		public $insertInto;
		public $insertColumns;
		public $insertValues;
		public $mensaje;
		public $rows;

	public function Create() 
	{
		$model = new Conexion;
		$conexion = $model->conectar();
		$insertColumns = $this->insertColumns;
		$insertValues = $this->$insertValues;
		$sql = "INSERT INTO $insertInto ($insertColumns) VALUES ($insertValues)";
		$consulta = $conexion->prepare($sql);
		if (!$consulta) 
		{
			$this->mensaje = "Error...";
		}
		else
		{
			$consulta->execute();
			$this->mensaje = "Registro Creado";
		}
	}

	public function Read()
	{
		$model = new Conexion();
		$conexion = $model->conectar();
		$sql = "SELECT * FROM snemple";
		$consulta = $conexion->prepare($sql);
		$consulta->execute();

		while ($filas = $consulta->fetch())
		{
			$this->rows[] = $filas;
		}
	}
}
?>

- Intente incluirlas en un solo archivo para ver si asi se solucionaba el problema y nada, intente de esta forma.

Código (php) [Seleccionar]

<?php
class Conexion
{
    public $ndatabase ="";
    public $insertInto;
	public $insertColumns;
	public $insertValues;
	public $mensaje;
	public $rows;
    
    public function conectar($cod_empresa="")
       {
            $this->ndatabase = $cod_empresa;
            $server = 'sbvbieloven';
            $database = $this->ndatabase;
            $username = 'profit';
            $password = 'profit';
            return $conexion = new PDO('sqlsrv:Server=' . $server . ';Database='. $database, $username, $password);
    	} 
    
    public function Create() 
	{
		$model = new Conexion;
		$conexion = $model->conectar();
		$insertColumns = $this->insertColumns;
		$insertValues = $this->$insertValues;
		$sql = "INSERT INTO $insertInto ($insertColumns) VALUES ($insertValues)";
		$consulta = $conexion->prepare($sql);
		if (!$consulta) 
		{
			$this->mensaje = "Error...";
		}
		else
		{
			$consulta->execute();
			$this->mensaje = "Registro Creado";
		}
	}

	public function Read()
	{
		$model = new Conexion();
		$conexion = $model->conectar();
		$sql = "SELECT * FROM snemple";
		$consulta = $conexion->prepare($sql);
		$consulta->execute();

		while ($filas = $consulta->fetch())
		{
			$this->rows[] = $filas;
		}
	}
}
?>

- Tambien Intente.

1- Agregarle como argumento el $cod_empresa="" a las funciones Read y Create y nada

2- Les quite el $model = new Conexion; porque pense que ya no tenia caso dejarselos y nada
hay me mostro.
- Fatal error: Call to undefined function prepare() in C:\xampp\htdocs\sismica\conexion.table.php on line 42.

- Espero puedas ayudarme con esto de antemano muchas gracias.

MinusFour · 3 Noviembre 2014, 14:47 PM

Hay algo raro con que no te detecte el $_POST['cod_empresa'], realmente tu implementación en POO no tiene la culpa. ¿Que te regresa var_dump($_POST); ? Si tu indice aparece bien en el var_dump debe ser algun tipo de problema con el encoding...

Tu último script no funciona porque en ningun momento estableces el nombre de la base de datos (llamas al metodo conectar sin ningun parametro y se asume el valor por default que esta vacio) y la inicialización del objeto PDO falla.

Slikp · 3 Noviembre 2014, 21:37 PM

- Hola MinusFour, en el ejemplo que expuse no establezco el nombre de la bd como parametro pero tambien expuse que.

1- Agregue como argumento el $cod_empresa="" a las funciones Read y Create y nada, ahora una pregunta si yo hago algo como esto no deberia de funcionar ??

2- Tambien intente ponerlo asi $cod_empresa...

- La verdad es que no entiendo porque no funciona... Intentare ser lo mas claro posible en

1- tengo este formulario.

Código (html4strict) [Seleccionar]


<!DOCTYPE html>
<html lang="es">
<head>
    <title>Control de Equipos</title>
    <meta charset="utf-8">
    <link rel="stylesheet" type="text/css" href="css/style.css">
</head>
<body>
 <div id="login-form">     
      <h3>Bases de Datos</h3>
        <fieldset>
            <form action="board_table.php" method="post">
                <center>
                    <select name="cod_empresa">
                        <option Value="HUYAP_N">Proyecto H</option>
                        <option Value="Zuat">Proyecto Z</option>
                        <option Value="Ricl">Proyecto C</option>
                    </select>
                </center>    

                <input type="hidden" name="session">
                <input id="sub" type="submit" value="Iniciar Conexión">
            </form>
        </fieldset>
</div>
</body>
</html>

- Eso me genera lo siguiente.

2- Con la ayuda de T.Collins realize lo siguiente ese formulario se viene aca.

Código (php) [Seleccionar]


<?php 
    require_once 'conexion.table.php';

        $cod_empresa = (isset($_POST['cod_empresa'])) ? $_POST['cod_empresa'] : "";
		$Conexion = new Conexion();
		$Conexion->conectar($cod_empresa);

    if(isset($_POST['session']))
    {
        header('location:ready.php');
    }
?>

2.1 - Donde la clase conexion es la siguiente.

Código (php) [Seleccionar]


<?php
class Conexion
{
    public $ndatabase ="";
    
    public function conectar($cod_empresa="")
       {
            $this->ndatabase = $cod_empresa;
            $server = 'sbvbieloven';
            $database = $this->ndatabase;
            $username = 'profit';
            $password = 'profit';
            return $conexion = new PDO('sqlsrv:Server=' . $server . ';Database='. $database, $username, $password);
    	} 
}
?>

3- Seguidamente por la condicion del Punto 2 me redirecciona a esta pag.

Código (html4strict) [Seleccionar]


<?php 
	require_once "conexion.table.php";
	require_once "crud.php";

	$model = new Crud;
	$model->Read();
	$filas = $model->rows;
	$total = count($filas);

?>
<html>
<head>
	<title></title>
	<meta charset="UTF-8">
	<link rel="stylesheet" type="text/css" href="css/style.css">
</head>
<body>	
	<section class="criterio">
		<div id="crit-form">
			<h3>Criterio de Busquedas</h3>
			<fieldset>
			<form class="crit-form" name="search_form" id="search_form">
				<small> El Total de Filas en Base de Datos: <?php echo $total ?> </small>
					<select size="10">
						<?php
foreach ($filas as $fila) 
		{
	?> <option id="resultados">
	<?php echo $fila['cod_emp'] . $fila['nombres'] ." ". $fila['apellidos'];
	?> </option>
					</select>
					<small>Por Ficha</small>
					<input type="text" name="search" id="search_fi" placeholder="introduzca Codigo Ficha Ej: Ch001">
					<small>Por Cedula</small>
					<input type="text" name="search" id="search_ci" placeholder="Cedula Use Puntos Ej: 49.781.190">
					<small>Por Nombre</small>
					<input type="text" name="search" id="search" placeholder="introduzca un Nombre Ej: Antonio">
					<small>Por Estatus</small>
					<select >
						<option>Seleccione Estatus</option>
						<option selected>Activo</option>
						<option>Inactivo</option>
						<option>Por Liquidar</option>
						<option>Liquidado</option>
					</select>
					<input type="submit" Value="Procesar">
				</form>
			</fieldset>
		</div>
	</section>	
</body>
</html>

3.1- Y eso me da esta pantalla donde evidentemente se que hay un problema porque de bajo del total esta en blanco y "" Como Dije Antes Pasandole Directamente O Manualmente La BD a La Clase Conexion Todo Funciona Fino "".

4- Donde mi clase Crud es la siguiente.

Código (php) [Seleccionar]


<?php 
	class Crud
{
		public $insertInto;
		public $insertColumns;
		public $insertValues;
		public $mensaje;
		public $rows;

	public function Create() 
	{
		$model = new Conexion;
		$conexion = $model->conectar();
		$insertColumns = $this->insertColumns;
		$insertValues = $this->$insertValues;
		$sql = "INSERT INTO $insertInto ($insertColumns) VALUES ($insertValues)";
		$consulta = $conexion->prepare($sql);
		if (!$consulta) 
		{
			$this->mensaje = "Error...";
		}
		else
		{
			$consulta->execute();
			$this->mensaje = "Registro Creado";
		}
	}

	public function Read()
	{
		$model = new Conexion();
		$conexion = $model->conectar();
		$sql = "SELECT * FROM snemple";
		$consulta = $conexion->prepare($sql);
		$consulta->execute();

		while ($filas = $consulta->fetch())
		{
			$this->rows[] = $filas;
		}
	}
}
?>

5 - Ahora cosas Sugeridas que hice MinusFour no se bien si aplique correctamente el Var_dump disculpa esa es que todo esto es nuevo para mi pero lo hice de esta forma.

- Elimine la condicion y aplique el var_dump aqui.

Código (php) [Seleccionar]


<?php 
    require_once 'conexion.table.php';

        $cod_empresa = (isset($_POST['cod_empresa'])) ? $_POST['cod_empresa'] : "";
		$Conexion = new Conexion();
		$Conexion->conectar($cod_empresa);

    // if(isset($_POST['session']))
    //{
    //    header('location:ready.php');
    //}

   var_dump($_POST['cod_empresa']);
?>

5.1- Resultado: string(10) "HUYAP_N".

6- Aplique Var_dump aqui.

Código (html4strict) [Seleccionar]


<?php 
	require_once "conexion.table.php";
	require_once "crud.php";
        echo var_dump($_POST['cod_empresa']);

	$model = new Crud;
	$model->Read();
	$filas = $model->rows;
	$total = count($filas);

?>
<html>
<head>
	<title></title>
	<meta charset="UTF-8">
	<link rel="stylesheet" type="text/css" href="css/style.css">
</head>
<body>	
	<section class="criterio">
		<div id="crit-form">
			<h3>Criterio de Busquedas</h3>
			<fieldset>
			<form class="crit-form" name="search_form" id="search_form">
				<small> El Total de Filas en Base de Datos: <?php echo $total ?> </small>
					<select size="10">
						<?php
foreach ($filas as $fila) 
		{
	?> <option id="resultados">
	<?php echo $fila['cod_emp'] . $fila['nombres'] ." ". $fila['apellidos'];
	?> </option>
					</select>
					<small>Por Ficha</small>
					<input type="text" name="search" id="search_fi" placeholder="introduzca Codigo Ficha Ej: Ch001">
					<small>Por Cedula</small>
					<input type="text" name="search" id="search_ci" placeholder="Cedula Use Puntos Ej: 49.781.190">
					<small>Por Nombre</small>
					<input type="text" name="search" id="search" placeholder="introduzca un Nombre Ej: Antonio">
					<small>Por Estatus</small>
					<select >
						<option>Seleccione Estatus</option>
						<option selected>Activo</option>
						<option>Inactivo</option>
						<option>Por Liquidar</option>
						<option>Liquidado</option>
					</select>
					<input type="submit" Value="Procesar">
				</form>
			</fieldset>
		</div>
	</section>	
</body>
</html>

6.1- Resultado: Notice: Undefined index: cod_empresa in C:\xampp\htdocs\sismica\ready.php on line 10
NULL.

7- y luego lo aplique aqui.

Código (php) [Seleccionar]

<?php
class Conexion
{
    public $ndatabase ="";    
    public function conectar($cod_empresa="")
       {

            $this->ndatabase = $cod_empresa;
            $server = 'sbvbieloven';
            $database = $this->ndatabase;
            $username = 'profit';
            $password = 'profit';
            return $conexion = new PDO('sqlsrv:Server=' . $server . ';Database='. $database, $username, $password);
    	} 
}
echo var_dump($_POST['cod_empresa']);
?>

7.1- Resultado: Notice: Undefined index: cod_empresa in C:\xampp\htdocs\sismica\conexion.table.php on line 16
NULL

MinusFour · 3 Noviembre 2014, 21:51 PM

Observa tus metodos Read y Create.

Ambos llaman a $modelo->conectar();

Esto ejecuta la funcion conectar de tu clase Conexion que a su vez crea un objeto PDO. Este objeto PDO se crea con un DSN invalido por lo que falla. Fijate que como llamas a la funcion sin pasar $cod_empresa, se asume '' por lo que el DSN acaba:

Código (php) [Seleccionar]


$database = '';
'sqlsrv:Server=' . $server . ';Database='. $database;

Código [Seleccionar]


'sqlsrv:Server=sbvbieloven;Database='

En cuanto a lo del var_dump, era una prueba porque en tu post original estabas enviando el campo cod_empresa pero el indice no lo reconoce PHP. Si PHP te lo reconoce ahora entonces no hay problema.

Slikp · 3 Noviembre 2014, 22:18 PM

- Cada vez me siento mas frustrado dado que intento comprender y no puedo jajaja, no entiendo porque me haces referencia a esto.

Código (php) [Seleccionar]


$database = '';
'sqlsrv:Server=' . $server . ';Database='. $database;

- Y me estas aclarando que me hace falta algo en mis metodos Read y Create y que es el $cod_empresa? debo tenerlos asi?

Código (php) [Seleccionar]


<?php 
	class Crud
{
		public $insertInto;
		public $insertColumns;
		public $insertValues;
		public $mensaje;
		public $rows;
 
	public function Create($cod_empresa) 
	{
		$model = new Conexion;
		$conexion = $model->conectar();
		$insertColumns = $this->insertColumns;
		$insertValues = $this->$insertValues;
		$sql = "INSERT INTO $insertInto ($insertColumns) VALUES ($insertValues)";
		$consulta = $conexion->prepare($sql);
		if (!$consulta) 
		{
			$this->mensaje = "Error...";
		}
		else
		{
			$consulta->execute();
			$this->mensaje = "Registro Creado";
		}
	}
 
	public function Read($cod_empresa)
	{
		$model = new Conexion();
		$conexion = $model->conectar();
		$sql = "SELECT * FROM snemple";
		$consulta = $conexion->prepare($sql);
		$consulta->execute();
 
		while ($filas = $consulta->fetch())
		{
			$this->rows[] = $filas;
		}
	}
}
?>

- Si lo pongo asi me sale este error:

Warning: Missing argument 1 for Crud::Read(), called in C:\xampp\htdocs\sismica\ready.php on line 6 and defined in C:\xampp\htdocs\sismica\crud.php on line 29

-Otra cosa no es mejor tener esa clase junto con la conexion asi??

Código (php) [Seleccionar]


<?php
class Conexion
{
    public $ndatabase =""; 
    public $insertInto;
	public $insertColumns;
	public $insertValues;
	public $mensaje;
	public $rows; 

    public function conectar($cod_empresa="")
       {

            $this->ndatabase = $cod_empresa;
            $server = 'sbvbieloven';
            $database = $this->ndatabase;
            $username = 'profit';
            $password = 'profit';
            return $conexion = new PDO('sqlsrv:Server=' . $server . ';Database='. $database, $username, $password);
    	} 
    public function Create() 
	{
		$model = new Conexion;
		$conexion = $model->conectar();
		$insertColumns = $this->insertColumns;
		$insertValues = $this->$insertValues;
		$sql = "INSERT INTO $insertInto ($insertColumns) VALUES ($insertValues)";
		$consulta = $conexion->prepare($sql);
		if (!$consulta) 
		{
			$this->mensaje = "Error...";
		}
		else
		{
			$consulta->execute();
			$this->mensaje = "Registro Creado";
		}
	}

	public function Read()
	{
		$model = new Conexion();
		$conexion = $model->conectar();
		$sql = "SELECT * FROM snemple";
		$consulta = $conexion->prepare($sql);
		$consulta->execute();

		while ($filas = $consulta->fetch())
		{
			$this->rows[] = $filas;
		}
	}

}
?>

- Ya estoy que tiro la toalla.

T. Collins · 3 Noviembre 2014, 22:45 PM

Slikp, en los métodos crear y read llamas a $conexion->conectar() sin ningún argumento, por lo que $database queda = ""