Con respecto a este tema me gustaría saber la opiniones de los demás, hace poco diría un par de horas he terminado un formulario maestro detalle y hasta ahora todo funciona bien. Resulta que un formulario maestro detalle tiene la cabecera y sus respectivos item del detalle, es aquí las acciones como grabar cabecera, agregar producto al detalle, eliminar producto del detalle, actualizar cantidad y precio del producto, etc, etc las ejecuto en el mismo formulario y no sigo las reglas que salen en los manuales de php que dice que un formulario php para cada acción y, es que a mí parecer uno se llena de archivos *.php innecesariamente, sería un archivo para cada acción.......no me atrae para nada......
Viendo el tema de los framework puede que aquí nos ayuden un poco en este tema ya que existe un controlador que ejecuta todas nuestras acciones quedando todo un poco más ordenanado.
Dejaré el código que uso para el formulario maestro detalle, podrán darse cuanta que todas las acciones las tomo en el mismo archivo php.....y es que mi forma de hacerlo todavía no me convence, compañeros quiero saber su opinión sobre este tema, llevo varios días dandole vuelta al asunto y creo sinceramente que estos temas no están bien tratado en php, o no hay mucha documentación o ejemplos sobre el tema.
Código PHP:
<?php
/*
* @Autor: Aarón García A
* @Fecha: 16 de enero de 2008
* @Descripción: Permite ingresar una nueva compra de un cliente y asociar productos
* a la compra.
*/
session_start();
include ('../../includes/funciones.php');
include ('../../includes/compras.inc.php');
$tpl = generaTpl();
// corresponde cuando ingresa por primera vez a la página
if (isset($_REQUEST['provrut'])) {
$provRut = $_REQUEST['provrut'];
$numDocto = obtenerUltimoNumCompra();
$numDoc = $numDocto['CompNumDoc'] + 1;
$fecha = fechaActual();
// corresponde cuando se selecciona un producto en la otra página
}elseif (isset($_REQUEST['prodcod']) or isset($_REQUEST['volver'])){
if (isset($_REQUEST['prodcod'])) {
$resultado = ingresarDetalleCompra($_SESSION['ses_numDoc'], $_REQUEST['prodcod'], 1,
0,0, 99, $fecha);
if ($resultado['estado'] == -1)
$mensaje = $resultado['error'];
}
obtenerDatos();
// corresponde cuando se graba la cabecera de la compra
}elseif (isset($_REQUEST['btnGrabar'])){
$mensaje = validar();
if ($mensaje == '') {
//Ingresa la cabecera de la compra dejando el estado en 1 (abierto)
$resultado = ingresarCompra($numDoc, $provRut, 'F', $compFactNum,
$compFactMonto,fechaSpaToEng($fecha), 1);
if ($resultado['estado'] == -1)
$mensaje = $resultado['error'];
else{
//activa este banderin para poder ingresar el detalle de la compra
$tieneCabecera = 1;
$_SESSION['ses_numDoc'] = $numDoc;
obtenerDatos();
}
}
// corresponde cuando elimina un producto de la compra
}elseif (isset($_REQUEST['eliminar'])){
$resultado = eliminarProductoCompra($_REQUEST['compNumDoc'], $_REQUEST['compProdCod']);
if ($resultado['estado'] == -1)
$mensaje = $resultado['error'];
obtenerDatos();
// corresponde cuando actualizan la cantida y precio del producto
}elseif (isset($_REQUEST['btnActualizar'])){
$inputCantProd = $_REQUEST['txtCompCantProd'];
$inputPrecProd = $_REQUEST['txtCompPrecVent'];
//actualiza cantidad
$j = 0;
foreach($inputCantProd as $prodCod => $keyCantProd)
foreach($keyCantProd as $cantProd){
$arrayCantProd[$j]['prodCod'] = $prodCod;
$arrayCantProd[$j]['prodCant'] = $cantProd;
$j++;
}
//actualiza precio
$i = 0;
foreach($inputPrecProd as $prodCod => $keyPrecProd)
foreach($keyPrecProd as $precProd){
$resultado = actualizarDetalleProductos($_SESSION['ses_numDoc'],
$arrayCantProd[$i]['prodCod'], $arrayCantProd[$i]['prodCant'],$precProd);
if ($resultado['estado'] == -1) {
$mensaje = $resultado['error'];
break;
}
$i++;
}
obtenerDatos();
}
$listadoDetCompra = listarDetalleCompra($numDoc);
//String con url que me permite ir a la página de articulo y regresar
// con uno especifico para insertarlo como detalle
$urlHelper = "pagAnterior=../ped_docto/ing_compras.php?";
generarHeader($tpl);
//asigno variables a template
$tpl->assign('compNumDoc', $numDoc);
$tpl->assign('compProvRut', $provRut);
$tpl->assign('compFec', $fecha);
$tpl->assign('compFactNum', $compFactNum);
$tpl->assign('compFactMonto', $compFactMonto);
$tpl->assign('urlHelper', $urlHelper);
$tpl->assign('tieneCabecera', $tieneCabecera);
$tpl->assign('mensaje',$mensaje);
$tpl->assign('listadoDetCompra', $listadoDetCompra);
$tpl->display('ped_docto/ing_compras.tpl');
generarFooter($tpl);
function validar(){
/*
* @Autor: Aarón García A
* @Fecha: 16 de enero de 2008
* @Descripción: Permite ingresar una nueva compra de un cliente y asociar productos
* a la compra.
*/
global $numDoc,$fecha,$provRut,$compFec,$compFactNum,$compFactMonto;
$mensaje = '';
// rescata valores para mantenerlos en el template
$numDoc = $_REQUEST['txtCompNumDoc'];
$provRut = $_REQUEST['txtProvRut'];
$fecha = $_REQUEST['txtCompFec'];
$compFactNum = $_REQUEST['txtCompFactNum'];
$compFactMonto = $_REQUEST['txtCompFactMonto'];
if (trim($numDoc) == '')
$mensaje = 'Ingrese número de documento';
elseif (!is_numeric($numDoc))
$mensaje = 'Número de documento debe ser númerico';
elseif (trim($fecha) == '')
$mensaje = 'Ingrese fecha de documento';
elseif (trim($compFactNum) == '')
$mensaje = 'Ingrese número de factura';
elseif (!is_numeric($compFactNum))
$mensaje = 'Número de factura debe ser númerico';
elseif (trim($compFactMonto) == '')
$mensaje = 'Ingrese monto de la factura';
elseif (!is_numeric($compFactMonto))
$mensaje = 'Monto de la factura debe ser númerico';
return $mensaje;
}
function obtenerDatos(){
global $numDoc,$fecha,$provRut,$compFec,$compFactNum,$compFactMonto,
$tieneCabecera;
$compra = obtenerCompra($_SESSION['ses_numDoc']);
$numDoc = $compra['CompNumDoc'];
$fecha = $compra['CompFec'];
$provRut = $compra['CompProvRut'];
$compFactNum = $compra['CompFactNum'];
$compFactMonto = $compra['CompFactNum'];
$tieneCabecera = 1;
}
?> Espero sus comentarios, criticas, alabanzas
, lo que sea, creo que cualquier cosa me serviría en estos momentos para empezar a disipar dudas.Hasta Luego .-
