Business
Jobs
  • About Us
  • Solutions
    • Job Postings
      Post your job and receive qualified candidates in 48h.
    • Candidate Assessments
      500+ technical and psychological tests, plus anti-fraud.
    • Headhunting
      Tailor-made executive search from start to finish.
    • Payroll + EOR
      Payroll dispersal and EOR across 15+ LATAM countries.
  • Pricing
  • Jobs

0

133
Views
Array with inputs in React

Im trying to create a table with an array of products, the problem I have is that the inputs take the value of any input with the same "name". Also, when I try to remove any of the products, it always removes the last one.

https://stackblitz.com/edit/react-gto9bw?file=src/App.js

const [producto, setProducto] = useState({
codigo: '',
nombre: '',
descripcion: '',
precio: '',
cantidad: '',
estado: '',


});

  const [productos, setProductos] = useState([]);

  const addProducto = () => {
    setProductos([...productos, producto]);
  };

  const removeProducto = (e, index) => {
    e.preventDefault();
    const list = [...productos];
    list.splice(index, 1);
    setProductos(list);
  };

  const handleInputChangeProducto = (e, index) => {
    e.preventDefault();
    const { name, value } = e.target;
    const list = [...productos];
    list[index][name] = value;
    setProductos(list);
  };

The return its a table that has a button to add the product

return (
<div>
  <table className="table-size" style={{ border: '1px solid black' }}>
    <thead>
      <tr>
        <th scope="col">Nombre</th>
        <th scope="col">Descripcion</th>
      </tr>
      {productos.map((producto, index) => (
        <tr key={index}>
          <td>
            <input
              name="nombre"
              onChange={(e) => handleInputChangeProducto(e, index)}
            />
          </td>
          <td>
            <input
              name="descripcion"
              onChange={(e) => handleInputChangeProducto(e, index)}
            />
          </td>
          <td onClick={(e) => removeProducto(e, index)}>
            <Button>Borrar Producto {index}</Button>
            
          </td>
        </tr>
      ))}
    </thead>
  </table>
  <br />

  <button onClick={addProducto}>Crear Producto</button>
</div>

I've tried separating the "tr" into a separate component but that doesn't work either.

about 4 years ago · Santiago Gelvez
3 answers
Answer question

0

Although the other answers are not wrong - and they all (implicitly) fixed the issues below, I wanted to call out the root cause of the problem you were seeing. It's not the use of indexes as keys (although having unique keys is a good idea for performance reasons).

  1. When you created a new producto (at least in your original code sample), you were doing this:

    const addProducto = () => {
        setProductos([...productos, producto]);
    };
    

    Note the new producto being added to the array is always the same object. That means that your array contains a list of pointers that are all pointing to the same object. Modifying one will modify all of them. That's probably not what you want. Instead, do this:

    const addProducto = () => {
        setProductos([...productos, {...producto} ]);
    };
    

    That spreads the properties of your blank producto object onto a new object and ensures that each object in the array is really a separate thing.

  2. The form's values were not actually controlled by the state. Going from your original code sample, make these changes:

       {productos.map((producto, index) => (
         <tr key={index}>
           <td>
             <input
               name="nombre"
               value={producto.nombre} // <-- this was missing. without it there is no relation between what the user is seeing in the input and what value is stored in the productos array.
               onChange={(e) => handleInputChangeProducto(e, index)}
             />
           </td>
           <td>
             <input
               name="descripcion"
               value={producto.descripcion} // <-- Same thing here
               onChange={(e) => handleInputChangeProducto(e, index)}
             />
           </td>
    
about 4 years ago · Santiago Gelvez Report

0

This is a common issue when using map with index instead of unique key.

So you can try like this:

Please add a global variable key in your case.

let key = 0;

Then set this key in your producto and increase it. (In my full code, I used codigo as a key field.)

It should always be increased.

On backend API, you should get unique key as well.

Here is a full code.

import React, { useState } from 'react';
import { Button } from 'react-bootstrap';
import './style.css';

let key = 0;

export default function App() {
  const [producto, setProducto] = useState({
    codigo: '',
    nombre: '',
    descripcion: '',
    precio: '',
    cantidad: '',
    estado: '',
  });

  const [productos, setProductos] = useState([]);

  const addProducto = () => {
    setProductos([...productos, { ...producto, codigo: ++key }]);
  };

  const removeProducto = (e, index) => {
    e.preventDefault();
    const list = [...productos];
    list.splice(index, 1);
    setProductos(list);
  };

  const handleInputChangeProducto = (e, index) => {
    e.preventDefault();
    const { name, value } = e.target;
    const list = [...productos];
    list[index][name] = value;
    setProductos(list);
  };

  console.log({ productos });

  return (
    <div>
      <table className="table-size" style={{ border: '1px solid black' }}>
        <thead>
          <tr>
            <th scope="col">Nombre</th>
            <th scope="col">Descripcion</th>
          </tr>
          {productos.map((producto, index) => (
            <tr key={producto.codigo}>
              <td>
                <input
                  name="nombre"
                  onChange={(e) => handleInputChangeProducto(e, index)}
                />
              </td>
              <td>
                <input
                  name="descripcion"
                  onChange={(e) => handleInputChangeProducto(e, index)}
                />
              </td>
              <td onClick={(e) => removeProducto(e, index)}>
                <Button>Borrar Producto {index}</Button>
              </td>
            </tr>
          ))}
        </thead>
      </table>
      <br />

      <button onClick={addProducto}>Crear Producto</button>
    </div>
  );
}

about 4 years ago · Santiago Gelvez Report

0

You need an unique key. Try to generate this unique key on your addProducto:

const addProducto = () => {
    setProductos([...productos, { ...producto, id: Date.now() }]);
};

And then on your productos.map pass this generated id in your <tr key={producto.id}>.

Its recommended you have "static" indexes as key, React will understand better how manipulate your list and if occurs any modifications in this list, he will not recalculate everything.

Another point, if you need to manipulate something that depends of your previous state, like these add or remove operations, try to use functions for updates, like:

const addProducto = () => {
    setProductos(prevProductos => [...prevProductos, { ...producto, id: Date.now() }]);
};

const removeProducto = (e, index) => {
    e.preventDefault();
    setProductos(prevProductos => [...prevProductos.slice(0, index), ...prevProductos.slice(index + 1)]);
};
about 4 years ago · Santiago Gelvez Report
Answer question
Find remote jobs

Discover the new way to find a job!

Top jobs
Top job categories
Business
Post vacancy Pricing Sales
Legal
Terms and conditions Privacy policy
© 2026 PeakU Inc. All Rights Reserved.
Andres GPT
Show me some job opportunities
There's an error!