Push an item to array of array in javascript

Question

Hi I am having an array like below and trying to push the item on map, but i am not able to push, Please tell what wrong i am doing here.

firstClassPrices: [
  [
    "Kan ej ombokas",
    "431",
    "SEK",
    "Företagspris"
  ],
  [
    "Kan ombokas",
    "525",
    "SEK",
    "Företagspris"
  ]
]

trying to push '1' like below:

 let ticketi = firstClassPrices.map((price) => {
      price.push("1");
    });

i need o/p as

firstClassPrices: [
      [
        "Kan ej ombokas",
        "431",
        "SEK",
        "Företagspris",
         "1"
      ],
      [
        "Kan ombokas",
        "525",
        "SEK",
        "Företagspris",
         "1"
      ]
    ]

Your code already works. You're only using .map() while not returning any elements from its callback function so the return value (which you store in ticketi will only contain undefined values. But the original firstClassPrices will contain what you are looking for. — Ivar
– Ivar, Commented Jun 23, 2022 at 8:47

callback · Accepted Answer · 2022-06-23 11:16:19Z

3

You have to return the price

const firstClassPrices = [
  [
    "Kan ej ombokas",
    "431",
    "SEK",
    "Företagspris"
  ],
  [
    "Kan ombokas",
    "525",
    "SEK",
    "Företagspris"
  ]
];

 let ticketi = firstClassPrices.map((price) => {
      // skip if price already includes 1
      if (!price.includes("1")) {
        price.push("1");
      }
      return price;
    });

console.log(ticketi);

edited Jun 23, 2022 at 11:16

answered Jun 23, 2022 at 8:46

callback

4,1221 gold badge35 silver badges57 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Ivar Over a year ago

In this code, both firstClassPrices and ticketi reference the same nested arrays. There is little added value in using .map() if you are going to modify the original array also.

Sudhir Over a year ago

if "1" is already there in array how to skip the push

callback Over a year ago

@Sudhir I updated the code with this requirement.

Nina Scholz · Accepted Answer · 2022-06-23 09:10:53Z

2

Your approach mutates the old array and maps undefined.

Instead, you could map the old and new content of the array.

This approach does not mutate the old data.

let ticketi = firstClassPrices.map(price => [...price, '1']);

edited Jun 23, 2022 at 9:10

answered Jun 23, 2022 at 8:47

Nina Scholz

388k26 gold badges367 silver badges417 bronze badges

1 Comment

Yosvel Quintero Over a year ago

Notice that according to OP: 1 should be of type string

Yosvel Quintero · Accepted Answer · 2022-06-23 09:01:54Z

1

You can use Array.prototype.map() combined with Array.prototype.concat()

Code:

const firstClassPrices = [["Kan ej ombokas","431","SEK","Företagspris"],["Kan ombokas","525","SEK","Företagspris"]]

const ticketi = firstClassPrices.map(p => p.concat('1'))

console.log(ticketi)

answered Jun 23, 2022 at 9:01

Yosvel Quintero

19.1k5 gold badges39 silver badges47 bronze badges

Comments

Ele · Accepted Answer · 2022-06-23 09:10:14Z

1

Since you want to mutate the original arrays, you can use the function Array.prototype.forEach.

The problem you're facing is regarding the missing return value inside of the map's handler, so basically, the handler is returning undefined

const array = [  [    "Kan ej ombokas",    "431",    "SEK",    "Företagspris"  ],  [    "Kan ombokas",    "525",    "SEK",    "Företagspris"  ]],
      handler = (a) => a.push("1");

array.forEach(handler);

console.log(array);

.as-console-wrapper { max-height: 100% !important; top: 0; }

answered Jun 23, 2022 at 9:10

Ele

33.8k7 gold badges43 silver badges80 bronze badges

Collectives™ on Stack Overflow

Push an item to array of array in javascript

4 Answers 4

3 Comments

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

3 Comments

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related