Ver Mensaje Individual
  #2 (permalink)  
Antiguo 11/09/2017, 09:06
alvaro_trewhela
Invitado
 
Mensajes: n/a
Puntos:
Respuesta: Repetir update mysql

El error es explicito

Unknown column 'XXX' in 'where clause'.
Eso quiere decir que la columna name es desconocida

Errores de concepto
1. No estas concatenando el usuario

2. Lo que hace IN es resumir esto:
WHERE planeta=tierra OR planeta=marte OR planeta=venus OR planeta=jupiter,...
En esto
WHERE planeta IN(tierra,marte,venus,jupiter,...)

Por lo tanto tendrias que concatenar solo el valor de IN

Código PHP:
Ver original
  1. $in = "";
  2. for($k=0;$k<sizeof($usuarios);$k++){
  3. $us = $usuarios[$k]
  4. $in = "'$us',";
  5. }
  6. $in = substr($in,0,-1); //quito la última coma

Como no entiendo realmente que quieres hacer si hacer los updates en el for, o fuera de este, si quieres fuera tendrias que usar IN y usarías el primer código que te dejo, si quieres dentro del for el segundo

De todas formas si quieres usar uno u otro tu update esta totalmente fuera de contexto, estas mezclando ambos conceptos

Código PHP:
Ver original
  1. $in = "";
  2. for($k=0;$k<sizeof($usuarios);$k++){
  3. $us = $usuarios[$k]
  4. $in .= "'$us',";
  5. }
  6. $in = substr($in,0,-1); //quito la última coma
  7.  
  8.   $sql = "UPDATE sipbuddies SET ".$sql;
  9.   $sql = rtrim($sql, ',');
  10.   $sql .= " WHERE ";
  11.   $sql .= " name='$in'";
  12.  
  13.   mysqli_query($db, $sql);
  14.   if(mysqli_errno($db)) die(mysqli_error($db));



Código PHP:
Ver original
  1. for($i=0; $i<count($usuarios); $i++)
  2. {
  3.   $sql = "UPDATE sipbuddies SET ".$sql;
  4.   $sql = rtrim($sql, ',');
  5.   $sql .= " WHERE ";
  6.   $sql .= " name='".$usuarios[$i]."'";
  7.  
  8.   mysqli_query($db, $sql);
  9.   if(mysqli_errno($db)) die(mysqli_error($db));
  10. }

Te recomiendo usar rtrim primero en $sql y no en la concatenacion:
$sql = "UPDATE sipbuddies SET ".rtrim($sql, ',');

Otra recomendacion no concatenes cosas que no son necesarias

$sql .= " WHERE ";
$sql .= " name='".$usuarios[$i]."'";

$sql .= " WHERE name='".$usuarios[$i]."'";

Otra recomendacion, usa POO para no tener que declarar la conexion en el parametro de mysqli_query, y ademas la funcion mysqli_query en POO es mas corta

$db->query($sql);

Eso.

Última edición por alvaro_trewhela; 11/09/2017 a las 09:12