duplicidad de datos

aviweb2015 · #1 (**permalink**) 09/04/2016, 16:06

hola amigos tengo un problemita resulta que cuanto intento guardar los datos en mi bd me crea duplicidad de datos cuando intento registrar dos productos en adelantes mis tablas son factura y detalles factura en la primera si me gustada bien pero en la segunado que seria detalles factura me duplica los datos cuando intento registrar dos productos en adelante.

código del carrito e campos para completar el registro personal

Código PHP:

Ver original<?php
if (isset($_GET['id']))
$id = $_GET['id'];
else
$id = 1;
if (isset($_GET['action']))
$action = $_GET['action'];
else
$action = "empty";
 
switch($action){
      
case "add":
if(isset($_SESSION['carro'][$id]))
$_SESSION['carro'][$id]++;
else
$_SESSION['carro'][$id]=1;
break;
        
case "remove":
if(isset($_SESSION['carro'][$id]))
          {
            $_SESSION['carro'][$id]--;
            if($_SESSION['carro'][$id]==0)
              unset($_SESSION['carro'][$id]);
          }
          
        break;
        case "removeProd":
          if(isset($_SESSION['carro'][$id])){
            unset($_SESSION['carro'][$id]);
          }
        break;
        
        case "mostrar":
          if(isset($_SESSION['carro'][$id])){
            continue;
          }
        break;
        
        case "empty":
          unset($_SESSION['carro']);
        
        break;
            
        
      }
 
      
if(isset($_SESSION['carro'])){
echo "<form action='carro.php' method='POST'>";
echo "<table class=''>";
$totalcoste = 0;//Inicializamos el contador de productos seleccionados.
$xTotal = 0;
        
echo "<tr>";
echo "<td>Producto</td>";
echo "<td>Cantidad</td>";
echo "<td>Acciones</td>";
echo "<td colspan=2 align=right >&nbsp;&nbsp;&nbsp;&nbsp;SubTotal</td>";
echo "</tr>";
echo "<tr><td colspan=5><hr></td></tr>";
  
foreach($_SESSION['carro'] as $id => $x){
$resultado = mysql_query("SELECT id,descripcion,precio FROM servicios WHERE id=$id");
$mifila = mysql_fetch_array($resultado);
$id = $mifila['id'];
$descripcion = $mifila['descripcion'];
//acortamos el nombre del producto a 40 caracteres
$descripcion = substr($descripcion,0,40);
$precio = $mifila['precio'];
//Coste por artículo según la cantidad elegida
$coste = $precio * $x;
//Coste total del carro
$totalcoste = $totalcoste + $coste;
//Contador del total de productos añadidos al carro
$xTotal = $xTotal + $x;
          
echo "<input type='hidden' name=descripcion value='$descripcion'>";
 
 
echo "<tr>";
echo "<td align='left'>$descripcion</td>";
echo "<td align='center'>$x<input type='hidden' name=chk_group3[] value='$x'></td>";
          
echo "<td align='left'>";
echo "<a href='carro.php?id=". $id ."&action=add'><img src='img/aumentar.png' style='padding:0 0px 0 5px;' alt='Aumentar cantidad' /></a>";
          //Controlamos el display para cuando se vaya a eliminar el producto del carro o bien
          //se vaya a reducir la cantidad.
          //if ($x > 1)
echo "<a href='carro.php?id=". $id ."&action=remove'><img src='img/restar.png' alt='Reducir cantidad' /></a>";
          //else
echo "<a href='carro.php?id=". $id ."&action=removeProd'><img src='img/eliminar.png' alt='Reducir cantidad' /></a></td>";
          
echo "<td align='right'>  </td>";
echo "<td align='right' style='margin-left:10px'>$coste Bsf
<input type='hidden' name=chk_group2[] value='$coste'>";
echo "</tr>";
 
}
        echo "<tr><td colspan='5'><hr></td></tr>";
        echo "<tr>";
        echo "</table>";
 
echo "<table>";
echo "<td align='right' 
colspan='4'><b>
<h4 class='blue-text text-darken-2 
right CONDENSED LIGHT5'>Total&nbsp;=&nbsp;$totalcoste&nbsp;Bsf</h4></b></td>
<input type='hidden' name=costetotal value='$totalcoste'>";
echo "</tr>";
echo "</table>";?>
<!-- final de la tabla-->
</div><!-- fin colunna izquiereda -->
 
 
 
<div class="col s12 m8 l5"> <!-- colunna derecha -->
<!-- fecha y numero de factura-->
<div class='input-field col s12 m4'><p class='blue-text text-darken-2'>Fecha:</p>
<input type="text" name="fecha" value="<?php echo date("d/m/Y"); ?>" 
readonly="true"/>
</div>
<div class='input-field col s12 m4'><p class='blue-text text-darken-2'>N° de Fact.</p>
<input type="text" name='idfactura' readonly='readonly' value="
<?php 
include("conexion.php"); 
$res=mysql_query("select * from factura");
if($campo=mysql_fetch_array($res))
{ 
$codigo=mysql_num_rows($res)+1;
echo $codigo=$codigo;
}
if(mysql_num_rows($res)==0)
echo $codigo='1' ;
?>" >
</div>
<div class='input-field col s12 m4'><p class='blue-text text-darken-2'>Personas</p>
<input id="icon_prefix" type="text" name="personas" autocomplete="off"
title="Se Necesita el numero de Personas Asistir" >
</div>
<div class='input-field col s12 m12'><p class='blue-text text-darken-2'>Tipo de Organización.</p>
<select select name="evento" title="Disculpa, seleccione el Tipo de Organizacion!" required/>
<option value=""  disabled selected>Seleccione</option>
<?php
include('conexion.php');
$result = mysql_query("SELECT * FROM evento");
while ($row=mysql_fetch_array($result)) {
?>
<option value="<?php echo $row['descripcion'];?>"> 
<?php echo $row['descripcion'];?> </option>
<?php
}
mysql_close($link);
?>
</select>
</div>
 
 
<div class='input-field col s12 m12'>
<p class='blue-text text-darken-2'>Cliente</p>
<input type=text placeholder='Busqueda de clientes' 
autocomplete="off"  id="inputString" name='idcedula'>
 
 
 
</div>
 
<div class='center-align'>
<div class='center-btn'><!-- botones -->
 
<div class='row'>
<button class='btn waves-effect red accent-4 
btn-medium tooltipped' data-position='right'
data-tooltip='Generar Contrato' type='submit' name='guardar'>
<i class="material-icons">description</i></i>
</button></form>
</div>
 
 
 
 
<div class='row'>
<a href="./contrato.php">
<button class='btn waves-effect cyan deep-purple accent-4 
btn-medium tooltipped' data-position='right'
data-tooltip='Volver a la lista de productos'
id="submit" type="submit" >
<i class='material-icons'>reply_all</i>
</button></a>
</div>
<div class='row'>
<a href='carro.php?id=$_SESSION[id]'>
<button class='btn waves-effect cyan darken-4  
btn-medium tooltipped' data-position='right'
data-tooltip='Vaciar Carrito' type='reset' >
<i class='tiny mdi-image-adjust'></i>
</button></a>
 
 
 
 
</div>
</div><!-- fin botones -->
 
 
 
</div><!-- fin div colunna izquierda y colunna derecha -->
 
<br>
 
<br>
 
<br>
 
 
<?php    
    }
      else
 
echo "<br>
<br>
<br>
<br>
<br>
<div class='center-align'>
<h5 class='red-text text-darken-4 center-align'>Ops, El carro 
está vacío, en breve sera redireccionado a la lista de productos</h4>
</div>";
// Redireccionar a contrato.php después de 5 segundos
//header( "refresh:10; url=contrato.php" );
 
 
 
    ?>

código para el proceso de registro en la bd:

Código PHP:

Ver original<?php
if (isset ($_POST['guardar'])){
$idfactura=$_REQUEST['idfactura'];
$idcedula=$_REQUEST['idcedula'];
$fecha=$_REQUEST['fecha'];
$iddetalle_factura=$_REQUEST['iddetalle_factura'];
$descripcion=$_REQUEST['descripcion'];
$cantprod=$_REQUEST['chk_group3'];
$coste=$_REQUEST['chk_group2'];
$costetotal=$_REQUEST['costetotal'];
$personas=$_REQUEST['personas'];
$evento=$_REQUEST['evento'];
 
include("conexion.php");
$sql="SELECT * FROM factura WHERE id='$id'";
   
  $res=mysql_query($sql,$link);
  $nrows=mysql_num_rows($res);
  if($nrows==0){
 
 
        for ($j = 0; $j < count($cantprod); $j++) {
                for ($k = 0; $k < count($coste); $k++) {
 
 
if (mysql_query("INSERT INTO factura (idfactura,idcedula,fecha,costetotal) VALUES 
('$idfactura','$idcedula','$fecha','$costetotal')",$link));
 
if (mysql_query("INSERT INTO detalles_factura (iddetalle_factura,idfactura,descripcion,cantprod,coste,personas,evento) VALUES 
('$iddetalle_factura','$idfactura','$descripcion','$cantprod[$j]','$coste[$k]','$personas','$evento')",$link));
 
}
        }
 
echo "<script>alert ('Se ha generado el contrato!');</script>
<META HTTP-EQUIV='REFRESH' CONTENT=0;URL=http:presupuesto.php>";
          }
 
else echo "<script>alert ('Disculpe este presupuesto ya existe!');</script>
<META HTTP-EQUIV='REFRESH' CONTENT=0;URL=http:presupuesto.php>";
          }
 
 
?>

gracias de ante mano

gnzsoloyo · #2 (**permalink**) 09/04/2016, 17:10

A nivel de BBDD, sólo podrían duplicarse registros en detalle_factura si la PK de la tabla no se ha definido sobre el par de campos (iddetalle_factura, idfactura).
Si la PK es otro campo diferente, entonces tienes un error en la definición de la tabla, y eso no ayuda.

aviweb2015 · #3 (**permalink**) 09/04/2016, 19:15

muestro mi base de datos

Código SQL:

Ver originalCREATE TABLE `factura` (
  `idfactura` INT(11) NOT NULL, **** primaria
  `idcedula` INT(11) NOT NULL,  *****INDEX
  `fecha` VARCHAR(25) NOT NULL,
  `costetotal` VARCHAR(45) NOT NULL
) ENGINE=InnoDB DEFAULT CHARSET=latin1;

Código SQL:

Ver originalCREATE TABLE `detalles_factura` (
  `iddetalle_factura` INT(5) NOT NULL, **** primaria
  `idfactura` VARCHAR(45) NOT NULL,
  `descripcion` VARCHAR(45) NOT NULL,
  `cantprod` VARCHAR(45) NOT NULL,
  `coste` VARCHAR(45) NOT NULL,
  `personas` INT(11) NOT NULL,
  `evento` VARCHAR(60) NOT NULL
) ENGINE=InnoDB DEFAULT CHARSET=latin1;

`iddetalle_factura` cuenta los registros de la tabla y `idfactura` las facturas

que tendria que hacer o como defino esos dos campos, gracias

gnzsoloyo · #4 (**permalink**) 09/04/2016, 19:48

Pero en esa estructura no hay ninguna PK definida en tabla.
¿Lo has notado?

Que le pongas a una columna un nombre "idfactura", no implica que sea PK. Debe tener expresamente puesta esa condición.
Veo que hay otros errores: Si el IDFACTURA es un INT, el campo que apunte a ese dato como FK debe ser del mismo tipo y rango de representación, y tu estás usando un VARCHAR(45)... cosa que está mal.

Código MySQL:

Ver originalCREATE TABLE `factura` (
  `idfactura` INT(11) NOT NULL PRIMARY KEY, 
  `idcedula` INT(11) NOT NULL, 
  `fecha` VARCHAR(25) NOT NULL,
  `costetotal` VARCHAR(45) NOT NULL
) ENGINE=InnoDB DEFAULT CHARSET=latin1;
 
CREATE TABLE `detalles_factura` (
  `iddetalle_factura` INT(11) NOT NULL PRIMARY KEY, 
  `idfactura` INT (11) NOT NULL,
  `descripcion` VARCHAR(45) NOT NULL,
  `cantprod` VARCHAR(45) NOT NULL,
  `coste` VARCHAR(45) NOT NULL,
  `personas` INT(11) NOT NULL,
  `evento` VARCHAR(60) NOT NULL
) ENGINE=InnoDB DEFAULT CHARSET=latin1;

Además tampoco tiene definidas las FK que debería. El IDCEDULA debería ser FK del cliente o de una tabla "persona", y no se ve en la tabla esa condición (INDEX solo expresa que es un campo indexado, no que sea FK

Me parece que debes repasar todos los conceptos básicos de SQL y definición de estructuras de datos, porque hay demasiados errores.

El tema se excede de este foro. Todo esto es OFF TOPIC por cuanto es tema de BBDD, y no de PHP.