[Resolvido] update em múltiplas linhas da mesma coluna

[phpnoob 0]

Boa noite galera, gostaria muito de uma ajudinha
Tenho em meu banco de dados uma tabela com três colunas (id - valor - titulo).

 

Meu formulário recebe estes dados e exibe nos inputs para uma possível atualização.

<form action="" method="post" enctype="multipart/form-data">
  
  <button>Atualizar</button>
  <input type="hidden" name="id[]" value="1">
  <label>Item 1</label>
  <input name="valor[]" value="Item 1" type="text">
  
  <input type="hidden" name="id[]" value="2">
  <label>Item 2</label>
  <input name="valor[]" value="Item 2" type="text">
  
</form>

Preciso que esse formulário atualize suas linhas correspondentes.

Executando o script a baixo direto no mysql funciona porem queria passar isso de forma dinâmica usando o php por que nem sempre o id e o valor serão o mesmo.

UPDATE tabela SET valor = CASE id WHEN 1 THEN 'Valor do item' WHEN 2 THEN 'valor do item 2' END WHERE id in(1, 2)

Alguém tem alguma ideia de como posso fazer ?

Estou usando PDO, ja tentei inúmeras formas com for, while e foreach mas sempre atualiza as linhas com o mesmo valor do ultimo campo.

 

Desde já agradeço vocês

[Matheus Tavares 167]

Olá!

 

1 - Livre-se do input hidden com o ID.

 

2 - Coloque o ID junto à informação do valor a ser editado. Assim:

<form action="" method="post" enctype="multipart/form-data">
  <button>Atualizar</button>
  <label>Item 1</label>
  <input name="valor[1]" value="Item 1" type="text">
  
  <!-- perceba que mudei o id, para mostrar que não precisa ser sequencial -->
  <label>Item 4</label>
  <input name="valor[4]" value="Item 4" type="text">
  
</form>

 

3 - Agora, no PHP, faça uma volta ao redor deles, atualizando um por um.

foreach( $_POST[ 'valor' ] as $id => $valor ) {
    if ( $valor !== '' ) { //se não estiver vazio, vamos atualizar
        $consulta = "UPDATE FROM tabela SET coluna = '{$valor}' WHERE id = {$id}";
        // e agora execute a consulta aqui...
    }
}

 

Nota que isso é bem mais simples? Para cada input valorado, você roda um UPDATE. Para situações normais, geralmente esse caminho é suficiente, além de ser mais legível e prático.

Mas algumas vezes, otimizar para reduzir as consultas é bacana, e aí a sua consulta com CASE entra.

 

Caso prefira com apenas uma consulta, você poderia fazer dessa forma:

<?php
// Apenas para testar
$array = [
    3 => 'testando',
    4 => 'hehe',
    10 => 'foo',
];

// Estou usando $array para testar, mas na prática basta substituir por $_POST[ 'valor' ]
$ids = array_keys( $array );
$ids_string = implode( ', ', $ids );

$case_string = implode( ' ', array_map(
    function( $id, $valor ) {
        return "WHEN {$id} THEN '{$valor}'";
    },
    $ids,
    $array
) );

// Agora atualizamos, mas apenas se a string não estiver vazia
if ( $case_string ) {
    $sql = "UPDATE tabela SET valor = CASE {$case_string} END WHERE id IN( {$ids_string} )";
    
    echo $sql;
}

 

Resultado do exemplo acima:

UPDATE tabela SET valor = CASE WHEN 3 THEN 'testando' WHEN 4 THEN 'hehe' WHEN 10 THEN 'foo' END WHERE id IN( 3, 4, 10 )

 

Ainda daria para otimizar mais, se fosse o caso... mas assim ao menos está rodando em uma consulta. Como efeito colateral, você tem um código maior e mais complexo. Geralmente, eu iria preferir a alternativa anterior...

 

Observe também que nenhuma das consultas está segura contra SQL injection, pois isso é outro tópico...

 

Espero ter ajudado. Até mais :)

[phpnoob 0]

25 minutos atrás, Matheus Tavares disse:

Olá!

 

1 - Livre-se do input hidden com o ID.

 

2 - Coloque o ID junto à informação do valor a ser editado. Assim:

<form action="" method="post" enctype="multipart/form-data">
  <button>Atualizar</button>
  <label>Item 1</label>
  <input name="valor[1]" value="Item 1" type="text">
  
  <!-- perceba que mudei o id, para mostrar que não precisa ser sequencial -->
  <label>Item 4</label>
  <input name="valor[4]" value="Item 4" type="text">
  
</form>

 

3 - Agora, no PHP, faça uma volta ao redor deles, atualizando um por um.

foreach( $_POST[ 'valor' ] as $id => $valor ) {
    if ( $valor !== '' ) { //se não estiver vazio, vamos atualizar
        $consulta = "UPDATE FROM tabela SET coluna = '{$valor}' WHERE id = {$id}";
        // e agora execute a consulta aqui...
    }
}

 

Nota que isso é bem mais simples? Para cada input valorado, você roda um UPDATE. Para situações normais, geralmente esse caminho é suficiente, além de ser mais legível e prático.

Mas algumas vezes, otimizar para reduzir as consultas é bacana, e aí a sua consulta com CASE entra.

 

Caso prefira com apenas uma consulta, você poderia fazer dessa forma:

<?php
// Apenas para testar
$array = [
    3 => 'testando',
    4 => 'hehe',
    10 => 'foo',
];

// Estou usando $array para testar, mas na prática basta substituir por $_POST[ 'valor' ]
$ids = array_keys( $array );
$ids_string = implode( ', ', $ids );

$case_string = implode( ' ', array_map(
    function( $id, $valor ) {
        return "WHEN {$id} THEN '{$valor}'";
    },
    $ids,
    $array
) );

// Agora atualizamos, mas apenas se a string não estiver vazia
if ( $case_string ) {
    $sql = "UPDATE tabela SET valor = CASE {$case_string} END WHERE id IN( {$ids_string} )";
    
    echo $sql;
}

 

Resultado do exemplo acima:

UPDATE tabela SET valor = CASE WHEN 3 THEN 'testando' WHEN 4 THEN 'hehe' WHEN 10 THEN 'foo' END WHERE id IN( 3, 4, 10 )

 

Ainda daria para otimizar mais, se fosse o caso... mas assim ao menos está rodando em uma consulta. Como efeito colateral, você tem um código maior e mais complexo. Geralmente, eu iria preferir a alternativa anterior...

 

Observe também que nenhuma das consultas está segura contra SQL injection, pois isso é outro tópico...

 

Espero ter ajudado. Até mais :)

Cara .. Muitíssimo obrigado, é exatamente o que eu preciso.

As duas formas resolvem meu problema.
Novamente obrigadoo