PHP OO Pasar un parametro o variable a una clase

[MinusFour]

- 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] Expandir

<?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] Expandir

<?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] Expandir

<?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.

Puedes hacerlo todo en una clase si gustas.

Lo que yo haria es guardar la conexion PDO en una variable.

Código (php) [Seleccionar] Expandir



class Conexion {
  protected $dbh;

  public function __construct() {
     //Guardas la conexion en una variable del objeto.
     $this->dbh = new PDO(...);
  }

  public function usoConexionParaLoQueSea($query){
   $query = $this->dbh->query($query);
   //o:
   $query = $this->dbh->prepare($query);
   $query->excute();
}

}

[Slikp]

- Buenas, vengo por el 2do round en primer lugar gracias por la paciencia que me han tenido.

- MinusFour me parece mas practico tener la clase conexion como me indicas pero primero quiero resolver lo otro.

- MinusFour, T.Collins me indican que en mis metodos rear y read llaman a $conexion->conectar() sin ningún argumento, pero que argumento se supone que deben tener porque lo puse asi:

Código (php) [Seleccionar] Expandir

$conexion = $model->conectar($cod_empresa);

y me lanzo.

Notice: Undefined variable: cod_empresa in C:\xampp\htdocs\sismica\crud.php on line 32.

- Asi no hace nada.

Código (php) [Seleccionar] Expandir

$conexion = $model->conectar($cod_empresa="");

- Aclaro que tengo los .php Crud y Conexion por separados como lo expuse anteriormete.

[MinusFour]

- Buenas, vengo por el 2do round en primer lugar gracias por la paciencia que me han tenido.

- MinusFour me parece mas practico tener la clase conexion como me indicas pero primero quiero resolver lo otro.

- MinusFour, T.Collins me indican que en mis metodos rear y read llaman a $conexion->conectar() sin ningún argumento, pero que argumento se supone que deben tener porque lo puse asi:

Código (php) [Seleccionar] Expandir

$conexion = $model->conectar($cod_empresa);

y me lanzo.

Notice: Undefined variable: cod_empresa in C:\xampp\htdocs\sismica\crud.php on line 32.

- Asi no hace nada.

Código (php) [Seleccionar] Expandir

$conexion = $model->conectar($cod_empresa="");

- Aclaro que tengo los .php Crud y Conexion por separados como lo expuse anteriormete.

Necesitas entender visibilidad de las variable. En ese punto no existe la variable $cod_empresa porque no está definida. Lo que puedes hacer es reusar el valor por post:

Código (php) [Seleccionar] Expandir


$conexion = $model->conectar($_POST['cod_empresa']);

[Slikp]

- MinusFour, eso ya lo habia echo antes y me daba este error:

Notice: Undefined index: cod_empresa in C:\xampp\htdocs\sismica\crud.php on line 13

- Osea antes ya habia intentado de la forma que me sugeristes.

Código (php) [Seleccionar] Expandir

$conexion = $model->conectar($_POST['cod_empresa']);

- Y en mi ignorancia hice un require_once de crud aqui:

Código (php) [Seleccionar] Expandir

<?php 
    require_once 'conexion.table.php';
    require_once 'crud.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');
    }
?>

- Pensando que habia que incluirlo hay para que aceptara el post pero igual me lanza:

Notice: Undefined index: cod_empresa in C:\xampp\htdocs\sismica\crud.php on line 13.
 

[MinusFour]

Empieza por poner tu código de nuevo, con los nombres de tus archivos. Nada de código tentativo, lo que tienes. El thread está lleno de tanto código que no se que es que.

[Slikp]

- Ok MinusFour hay voy porque también he agregado unas cosas, pero no esta muy distinto a lo que ya he planteado.

1- tengo este formulario.

Código (html4strict) [Seleccionar] Expandir


<!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>


2- Ese formulario se viene aca.

Código (php) [Seleccionar] Expandir


<?php 
    require_once 'conexion.table.php';
    require_once 'crud.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] Expandir


<?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] Expandir

]<?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">
<script src="script/jquery-2.1.1.min.js"></script>
<script src="script/ajax.js"></script>
</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">
<option id="resultados"></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>

4- Donde mi clase Crud es la siguiente.

Código (php) [Seleccionar] Expandir


<?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;
}
}
    public function Searchi()
{
$model = new Conexion();
$conexion = $model->conectar($_POST['cod_empresa']);

$search = $_POST['search'];

$sql = "SELECT nombres, apellidos, ci, cod_emp FROM snemple WHERE nombres LIKE '%".$search."%' or ci LIKE '%".$search."%' or cod_emp LIKE '%".$search."%'";
$consulta = $conexion->prepare($sql);
$consulta->execute();

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


5- Ahora he añadido algunas cosas como Ajax de la siguiente forma. Ya inclui las etiquetas jquery y mi otro script ajax que es donde esta mi funcion de esta forma.

Código (javascript) [Seleccionar] Expandir


$(function(){
$('#search').focus();
$('#search_form').submit(function(e){
e.preventDefault();
})

$('#search_form input[type=text]').keyup(function(){

var envio = $(this).val();

$('#resultados').html('<h2><img src="img/loadin.gif" width="20" /> Cargando...</h2>');
$.ajax({
type: 'POST',
url: 'search.php',
data: ('search='+envio),
success: function(resp){
if (resp!="") {
$('#resultados').html(resp);
}
}
})
})
})


6- Donde el Search.php es este.

Código (php) [Seleccionar] Expandir


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

sleep(1);

$search = '';
$nombres = '';
if (isset($_POST['search'])){
$search = $_POST['search'];
}

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

  if ($total>0 && $search!='')
  {
?>
<select class="selected" size="10">
<?php 
foreach ($filas as $fila) 
{
?> <option id="resultados">
<?php echo $fila['cod_emp'] . $fila['nombres'] ." ". $fila['apellidos'];
?> </option>
<?php
} 
?>
</select>
<?php 
  }
   
   else { if($total>0 && $search=='') {
   ?>
  <select size="10">
  <?php  
   foreach ($filas as $fila) 
{
?>
<option id="resultados">
<?php echo $fila['cod_emp'] . $fila['nombres'] ." ". $fila['apellidos']; ?>
</option>
<?php 
   }
   ?>
  </select>
  <?php 
  }
}
?>


- Eso ultimo me actualiza el Option de mi html y me muestra los registros en mi Select por medio de la funcion Searchi.

- Ahora aclaro todo eso me funciona de 10 si dejo mi clase conexion asi.

Código (php) [Seleccionar] Expandir


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


- Si la dejo asi funciona al pelo me muestra los registros de esta BD fino y como yo quiero, pero no es la idea porque en mi primer html tengo un select con 3 opciones que son 3 bd distintas en el mismo gestor de bd y bueno la idea principal es esa pues tener una funcion que me permita visualizar la informacion que yo quiera al selecionar cualquiera de esas 3 bd.

[MinusFour]

Lo único que se me pudiera ocurrir es que cuando haces el redirect usando header() hacia ready.php los campos posts se pierden. Tecnicamente estás haciendo una nueva peticion HTTP a ready.php y no contiene la informacion de la base de datos a consultar (cod_empresa).

[Slikp]

- Y una solucion para eso no la hay?...

- Hmmm esto lo digo en mi ignorancia, no hay forma de convertir el $_POST['cod_empresa'] en una variable global para que pueda ser leida facilmente por cualquier .php que se cree? algo asi parecido al session_start?.

- Por otra parte no hay forma de hacer lo que busco de otra manera no tan complicada?.

[T. Collins]

Slikp, puedes hacerlo utilizando una sesión, para eso tienes que poner en todos los php:

Código (php) [Seleccionar] Expandir

session_start();
y guardar $cod_empresa o $conexion en la superglobal $_SESSION, por ejemplo:

Código (php) [Seleccionar] Expandir

$_SESSION['conexion'] = $conexion;

También puedes enviar $cod_empresa por $_POST, por ejemplo:

Código (php) [Seleccionar] Expandir

header("Location: /ready.php?cod=".$cod_empresa);

y claro haciendo eso, en los métodos de la clase Crud cuando pones:

Código (php) [Seleccionar] Expandir

$conexion = $model->conectar();
tendras que hacer:

Código (php) [Seleccionar] Expandir

$conexion = $model->conectar($_SESSION['conexion']);
//o de la otra forma
$conexion = $model->conectar($_POST['cod']);

[Slikp]

- Buenas T. Collins, MinusFour realmente muchas gracias por su ayuda y perseverancia me han ayudado como no tienen idea, al parecer todo va de 10.

- Les comento que intente con la segunda Opcion, osea pasar por post el cod_empresa

Código (php) [Seleccionar] Expandir


header("Location: /ready.php?cod=".$cod_empresa);


- Y luego en mis metodos del crud poner

Código (php) [Seleccionar] Expandir


$conexion = $model->conectar($_POST['cod']);


- Tal cual como me sugirio T.Collins y esto no funciono si vi que por Url se pasaba el nombre de la bd cprrectamente y hasta intente pasar ese nombre por un input hidden desde el segundo formulario a Search.php a ver si eso era el problema y nada de nada.

- Así que con pocas esperanzas jajaja pase a intentar con session_start() sugerido por T.Collins y realmente este metodo me funciono de a 10.

Código (php) [Seleccionar] Expandir


<?php 
    require_once 'conexion.table.php';
    session_start();

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

    if(isset($_POST['cod_empresa']))
    {
        header("Location:ready.php");
    }
?>


- No use el session_star en mi clase conexion porque no le vi sentido hacerlo, pero en la clase Crud si lo use tal cual como sugirio T.Collins

Código (php) [Seleccionar] Expandir


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

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

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

public function Searchi()
{
session_start();
$model = new Conexion();
$conexion = $model->conectar($_SESSION['cod_empresa']);

$search = $_POST['search'];

$sql = "SELECT nombres, apellidos, ci, cod_emp FROM snemple WHERE nombres LIKE '%".$search."%' or ci LIKE '%".$search."%' or cod_emp LIKE '%".$search."%'";
$consulta = $conexion->prepare($sql);
$consulta->execute();

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

- Luego de esto probe denuevo y listo no lo podia creer cerre, abrir, probe, reprobe y todo fino muchas gracias de verdad, aunque me quedaron ciertas dudas sobre este tema yo creo que podriamos dejarlo como solucionado, espero contar denuevo con su ayuda por si me llegara a presentar otro problema dado que aun tengo mas idea para mi sistema...

    -Saludos.