Ver Mensaje Individual
  #1 (permalink)  
Antiguo 24/01/2005, 12:04
Avatar de verinchi
verinchi
 
Fecha de Ingreso: septiembre-2004
Ubicación: Buenos Aires
Mensajes: 647
Antigüedad: 19 años, 6 meses
Puntos: 2
Es necesaria esta función en este código???

<?php require_once('../../includes/cnn_unidb.php'); ?>
<?php
/* DEFINICION VARIABLES GENERALES */
$do = $_POST['do'];
$exito=false;

/* SI DO CUMPLE ALGUNA DE LAS CONDICIONES REALIZA OPERACION */
if ($do=="insert" || $do=="update" || $do=="delete") {

Esta es la que quiero saber si es necesaria (Formateo de textos), considerando que recibe del formulario de la pagina anterior dos campos de texto, uno con el nombre y otro con la descripción. En ocasiones puede llegar la categoría (ID de categoría) y el valor de do (Que solo puede sera update, delete e insert)
Sucede que estoy analizando codigo ajeno y hace rato que no programo en php así que no recuerdo demasiado...


/* FUNCIONES FORMATEO DE TEXTOS */
function formatString($theValue, $theType, $theDefinedValue = "", $theNotDefinedValue = "")
{
$theValue = (!get_magic_quotes_gpc()) ? addslashes($theValue) : $theValue;

switch ($theType) {
case "text":
$theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL";
break;
case "long":
case "int":
$theValue = ($theValue != "") ? intval($theValue) : "NULL";
break;
case "double":
$theValue = ($theValue != "") ? "'" . doubleval($theValue) . "'" : "NULL";
break;
case "date":
$theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL";
break;
case "defined":
$theValue = ($theValue != "") ? $theDefinedValue : $theNotDefinedValue;
break;
}
return $theValue;
}

/* FORMATEA VARIABLES */
$nombreCat = formatString($_POST['nombre'], "text");
$desc = formatString($_POST['desc'], "text");
if (isset($_POST['idCat'])) {
$idCat=formatString($_POST['idCat'], "int");
}
/* MAS VARIABLES */
$goto = "categs.php";

/* CASOS DE USO */
switch($do) {

/* CASO DE INSERTAR */
case "insert":
$SQL = "INSERT INTO cat_flota (NOMBRE, DESCRIP) VALUES ($nombreCat, $desc)";
$exito = true;
$msg = "Insercción realizada.";
break;

/* CASO DE MODIFICAR */
case "update":
$SQL = "UPDATE cat_flota SET NOMBRE=$nombreCat, DESCRIP=$desc WHERE IDCATEG=$idCat";
$exito = true;
$msg = "Modificación realizada.";
break;

/* CASO DE ELIMINAR */
case "delete":
$SQL = "DELETE FROM cat_flota WHERE cat_flota.IDCATEG=$idCat";
$exito = true;
$msg = "Eliminación realizada.";
break;

/* CASO DE ELIMINAR */
default:
$exito = false;
break;
}
}

if ($exito==false) {
$msg = "Hubo un error verifique los datos ingresados.";
header(sprintf("Location: %s", $goto."?armarMsg=si&msg=".$msg));
} else {
mysql_select_db($db, $cnn_unidb);
$Result1 = mysql_query($SQL, $cnn_unidb) or die(mysql_error());
header(sprintf("Location: %s", $goto."?armarMsg=si&msg=".$msg));
}
?>

Ya de movida me parece que la primer condición está casi de adorno, ya que el usuario solo puede elegir insertar, eliminar o modificar cada uno de los campos, otro valor no reciben. Así que decir si cumple la primera o la segunda o la tercera es lo mismo que nada porque en ningún momento sucederá que llegue una cuarta opción porque no existe.....
Se me hace que mas que analizar el codigo este voy a tener que reprogramar todo, pero prefiero arreglar y parchar que empezar de nuevo.... estoy algo vaga ultimamente, será que quiero vacaciones ya.
Bueno, Gracias de antemano a todos los que se tomen la molestia de contestarme!!!!

__________________
Why can't we not be sober?
www.partitorium.com.ar