actualizar un valor de un registro con el mismo id

[davidmrtnz413]

Hola que tal pues aqui de nuevo pidiendo ayuda tengo la siguiente cuestion necesito actualizar o modificar un valor a un registro el cual tiene el mismo id pero diferente informacion por ejemplo

id -----nombre-----departamento------sueldo
1 juan Informatica 100
1 juan Contabilidad 200

quiero que al actualizar el pago del departamento lacteos en juan no me cambien los 2 registros ya que si hago el update me los pone igual..

use la siguiente instruccion SQL para evitar eso pero la bronca es que ahora no actualiza nada

UPDATE empleados SET sueldo='$sueldo' WHERE idempleado=$idempleado AND departamento='$departamento'

no me actualiza el codigo lo saque de un ejemplo y ahora mi duda no se si sea bronca de php o ajax ya que usa ajax adjunto el codigo

Código AJAX:

//Desarrollado por Jesus Liñán
//[email protected]
//ribosomatic.com
//Puedes hacer lo que quieras con el código
//pero visita la web cuando te acuerdes
 
function objetoAjax(){
    var xmlhttp=false;
    try {
        xmlhttp = new ActiveXObject("Msxml2.XMLHTTP");
    } catch (e) {
        try {
           xmlhttp = new ActiveXObject("Microsoft.XMLHTTP");
        } catch (E) {
            xmlhttp = false;
        }
    }
 
    if (!xmlhttp && typeof XMLHttpRequest!='undefined') {
        xmlhttp = new XMLHttpRequest();
    }
    return xmlhttp;
}
 
function enviarDatosEmpleado(){
    //donde se mostrará lo resultados
    divResultado = document.getElementById('resultado');
    divFormulario = document.getElementById('formulario');
    //valores de los inputs
    id=document.frmempleado.idempleado.value;
    nom=document.frmempleado.nombres.value;
    dep=document.frmempleado.departamento.value;
    suel=document.frmempleado.sueldo.value;
    
    //instanciamos el objetoAjax
    ajax=objetoAjax();
    //usando del medoto POST
    //archivo que realizará la operacion
    //actualizacion.php
    ajax.open("POST", "actualizacion.php",true);
    ajax.onreadystatechange=function() {
        if (ajax.readyState==4) {
            //mostrar los nuevos registros en esta capa
            divResultado.innerHTML = ajax.responseText
            //mostrar un mensaje de actualizacion correcta
            divFormulario.innerHTML = "<p style=\"border:1px solid red; width:400px;\">La actualizaci&oacute;n se realiz&oacute; correctamente</p>";
        }
    }
    //muy importante este encabezado ya que hacemos uso de un formulario
    ajax.setRequestHeader("Content-Type","application/x-www-form-urlencoded");
    //enviando los valores
    ajax.send("idempleado="+id+"&nombres="+nom+"&departamento="+dep+"&sueldo="+suel)
}
 
function pedirDatos(idempleado){
    //donde se mostrará el formulario con los datos
    divFormulario = document.getElementById('formulario');
    
    //instanciamos el objetoAjax
    ajax=objetoAjax();
    //uso del medotod GET
    ajax.open("POST", "consulta_por_id.php");
    ajax.onreadystatechange=function() {
        if (ajax.readyState==4) {
            //mostrar resultados en esta capa
            divFormulario.innerHTML = ajax.responseText
            //mostrar el formulario
            divFormulario.style.display="block";
        }
    }
    //como hacemos uso del metodo GET
    //colocamos null
    ajax.setRequestHeader("Content-Type","application/x-www-form-urlencoded");
    //enviando los valores
    ajax.send("idemp="+idempleado)
}
 

CONSULTA.PHP

Código PHP:

<?php
//Desarrollado por Jesus Liñán
//ribosomatic.com
//Puedes hacer lo que quieras con el código
//pero visita la web cuando te acuerdes
 
//Configuracion de la conexion a base de datos
$bd_host = "localhost"; 
$bd_usuario = "root"; 
$bd_password = "root"; 
$bd_base = "empleados"; 
$con = mysql_connect($bd_host, $bd_usuario, $bd_password); 
mysql_select_db($bd_base, $con); 
 
//consulta todos los empleados
 
$sql=mysql_query("SELECT * FROM empleados",$con);
 
//muestra los datos consultados
//haremos uso de tabla para tabular los resultados
?>
<table style="border:1px solid #FF0000; color:#000099;width:400px;">
<tr style="background:#99CCCC;">
    <td>Codigo</td>
    <td>Nombres</td>
    <td>Departamento</td>
    <td>Sueldo</td>
</tr>
 
<?php
while($row = mysql_fetch_array($sql)){
    echo "  <tr>";
    //mediante el evento onclick llamaremos a la funcion PedirDatos(), la cual tiene como parametro
    //de entrada el ID del empleado
    echo "      <td><a style=\"text-decoration:underline;cursor:pointer;\" onclick=\"pedirDatos('".$row['idempleado']."')\">".$row['idempleado']."</a></td>";
    echo "      <td>".$row['nombres']."</td>";
    echo "      <td>".$row['departamento']."</td>";
    echo "      <td>".$row['sueldo']."</td>";
    echo "  </tr>";
}
?>
</table>
 

ACTUALIZACION.PHP

Código PHP:

<?php
//Desarrollado por Jesus Liñán
//[email protected]
//ribosomatic.com
//Puedes hacer lo que quieras con el código
//pero visita la web cuando te acuerdes
 
//Configuracion de la conexion a base de datos
$bd_host = "localhost"; 
$bd_usuario = "root"; 
$bd_password = "root"; 
$bd_base = "empleados"; 
$con = mysql_connect($bd_host, $bd_usuario, $bd_password); 
mysql_select_db($bd_base, $con); 
 
//variables POST
$idemp=$_POST['idempleado'];
$nom=$_POST['nombres'];
$dep=$_POST['departamento'];
$suel=$_POST['sueldo'];
 
//actualiza los datos del empleados
$sql="UPDATE empleados SET  sueldo='$suel' WHERE idempleado=$idemp AND departamento='Contabilidad'";
 
mysql_query($sql,$con);
 
include('consulta.php');
?>
 

CONSULTA POR ID.PHP

Código PHP:

<?php
//Desarrollado por Jesus Liñán
//ribosomatic.com
//Puedes hacer lo que quieras con el código
//pero visita la web cuando te acuerdes
 
//Configuracion de la conexion a base de datos
$bd_host = "localhost"; 
$bd_usuario = "root"; 
$bd_password = "root"; 
$bd_base = "empleados"; 
$con = mysql_connect($bd_host, $bd_usuario, $bd_password); 
mysql_select_db($bd_base, $con); 
 
//consulta los datos del empleado por su id
$idemp=$_POST['idemp'];
 
$sql=mysql_query("SELECT * FROM empleados WHERE idempleado=$idemp",$con);
 
$row = mysql_fetch_array($sql);
 
//valores de las consultas
$nom=$row['nombres'];
$dep=$row['departamento'];
$suel=$row['sueldo'];
 
//muestra los datos consultados en los campos del formulario
?>
<form name="frmempleado" action="" 
onsubmit="enviarDatosEmpleado(); return false">
    <input name="idempleado" type="hidden" value="<?php echo $idemp; ?>" />
  <p>Nombres 
    <input name="nombres" type="text" value="<?php echo $nom; ?>" />
  </p>
  <p>Departamento 
    <select name="departamento">
      <?php
      echo "<option value=\"".$dep."\">".$dep."</option>"
      ?>
      <option value="Informatica">Informatica</option>
      <option value="Contabilidad">Contabilidad</option>
      <option value="Administracion">Administracion</option>
      <option value="Logistica">Logistica</option>
    </select>
  </p>
  <p>Sueldo <strong>S/.</strong>
    <input name="sueldo" type="text" value="<?php echo $suel; ?>" />
  </p>
  <p>
    <input type="submit" name="Submit" value="Actualizar" />
  </p>
</form>
 

esos son los mas importantes ya que los otros son para que aarezca todo, espero me puedan que hacer saludos

[Raziel_Ravenheart]

En este caso,... creo que lo más conveniente sería que modificaras la base de datos, por que? muy sencillo, digamos que por ahora tienes a juan de informatica y a juan de contabilidad, pero cuando alguien esté ingresando datos puede escribir el nombre del departamento como se le venga en gana, es decir;

Dep.inf
Info.
Informatica
informática
infor
depto.Sistemas
contabilidad
cont
depto.cont

y al hacer esto. la integridad de la base de datos se vería comprometida, en el momento en que hayan varios funcionarios de un mismo departamento y todos tengan diferente el nombre del departamento, como en el ejemplo anterior. Para evitar esto, es mejor categorizar los departamentos, creando una tabla aparte para estos, por muy pocos que sean es lo más aconsejable. en donde tendrías una tabla con dos campos idDepartamento y nomDepartamento, luego, en el campo de la base de datos donde tienes departamento, lo cambias por idDepartamento y haces una relacion con llave foránea.

Cual es el truco aquí? muy sencillo, una vez creada la llave foránea, puedes crear una llave principal compuesta, donde suponiendo que los idDepartamento sean:

idDepartamento nomDepartamento
1 Informatica
2 Contabilidad
3 Cartera

la tabla de funcionarios te quedaría algo como

id -----nombre-----idDepartamento------sueldo
1 - juan - 1 - 100
1 - juan - 2 - 200

y con base en esto puedes armar una llave principal compuesta para esta tabla, donde sus partes serían id e idDepartamento, de este modo pueden existir dos registros con id=1, pero como la llave compuesta también necesita del idDepartamento, no habrá conflicto, y te ahorras cualquier error que provenga de escribir mal los nombres de los departamentos, además que ahora tendrá un nivel más de normalización una parte de tu base de datos, lo cual la puede hacer más fácil de mantener en una futura mejora.

Por otro lado revisando tu código no he visto nada mal... pero déjame le hecho otra ojeada a ver que encuentro, pero prefiero que vayas considerando hacer los cambios que te acabo de escribir.

Ten cuidado que en el archivo "ACTUALIZACION.PHP", estoy observando que en la línea 23, tienes estático el nombre de "Contabilidad", cuando en este caso debería ir la variable $dep. O es posible que lo tengas ahí intencionalmente.

Un consejo que te doy a la hora de hacer las consultas es el siguiente, siempre que puedas utiliza comillas sencillas, por que las comillas dobles le indican al motor de php que debe buscar variables que procesar, y esto no es aconsejable cuando son muchas. En este caso no creo que haya mucho problema, pero en una futura aplicación donde se hagan cientos de consultas de esta manera, puede representar una baja en el rendimiento. Un ejemplo con comillas sencilas sería

Código PHP:

$sql = 'UPDATE empleados SET  sueldo='.$suel.' WHERE idEmpleado='.$idemp.' AND departamento='.$dep;
 

[davidmrtnz413]

efectivamente gracias por el consejo y separe empleados ydepartamentos con una tabla que las relaciona y la llame emple_dep en donde envio los puros id´s de cada cosa para poder trabajar la actualizacion mas fiable la tabla se llama

emple_dep tiene campos

id_empleado | id_departamento | sueldo
0001 (juan) 0001 (lacteos) 100
0001 (juan) 0002 (abarrotes) 200

y ya me modifica por separado usando la instruccion que pusiste pero ahora tengo una bronquilla, cada vez que quiero cambiar ese valor me toma el primer registro del mismo id y no puedo elegir el 2 registro

puedo cambiar 100 sin alterar el 200 pero no me deja seleccionar el de 200

que sera??

[oXImLinkXo]

¿podrías postear la consulta sql que usas? ¿Cómo la editaste?

[davidmrtnz413]

Iniciado por oXImLinkXo

¿podrías postear la consulta sql que usas? ¿Cómo la editaste?

Código PHP:

$sql="UPDATE emple_dep SET sueldo='$sueldo' WHERE id_empleado=$id_empleado AND id_departamento='$departamento'";
 

aqui esta la consulta en donde para poder hacerlo en el formulario donde actualizo hice que en un label me sacara el id_departamento y me lo tomara con la variable departamento para que comprebe que el id_empleado y id_departamento fueran iguales ya que lometi primero de esta manera

Código PHP:

$sql="UPDATE emple_dep SET sueldo='$sueldo' WHERE id_empleado=$id_empleado AND id_departamento='$id_departamento'";
 

y no me funciono no actualizaba ni nada y no se a que por que
saludos