Remove elements from an array according to empty elements of another array

Question

I want to remove elements from an array only if exist empty elements in another array. I have this:

var fruits = ["Banana", "Naranja", "Manzana", "Mango", "Fresa", "Limón", "Lima", "Sandia"];
var vegetales = ["", "Ajo", "", "Cebolla", "Tomate", "", "", "Zanahoria"];
    
// The output should be this:
// ["Naranja", "Mango", "Fresa", "Sandia"]
function myFunction() {
  for (var i in vegetales) {
    if (vegetales[i] == '') {
      frutaAeliminar = fruits[i];
      indexFruta = fruits.indexOf(frutaAeliminar);
      if (indexFruta != -1) {
        fruits.splice(indexFruta, 1)
      }
    }
  }
  console.log(fruits);
}

myFunction();

StudioTime · Accepted Answer · 2017-05-07 19:01:48Z

2

Example using ES6

Use Array.filter and using && in a ternary allows us to not bother with else if it's not needed

const fruits = ["Banana", "Naranja", "Manzana", "Mango", "Fresa", "Limón", "Lima", "Sandia"];
const vegetales = ["", "Ajo", "", "Cebolla", "Tomate", "", "", "Zanahoria"];

/* only pick if corresponding array item does not equal '' */ 
const newArray = fruits.filter((fruit, idx) => (vegetales[idx] !== '' && fruit))

console.log(newArray)

edited May 7, 2017 at 19:01

answered May 7, 2017 at 18:50

StudioTime

24.4k40 gold badges129 silver badges217 bronze badges

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

Comments

Nina Scholz · Accepted Answer · 2017-05-07 20:44:05Z

2

You could filter with the truthy value of the corresponding vegetales item.

var fruits = ["Banana", "Naranja", "Manzana", "Mango", "Fresa", "Limón", "Lima", "Sandia"];
    vegetales = ["", "Ajo", "", "Cebolla", "Tomate", "", "", "Zanahoria"],
    newArray = fruits.filter((_, i) => vegetales[i]);

console.log(newArray)

edited May 7, 2017 at 20:44

answered May 7, 2017 at 18:58

Nina Scholz

388k26 gold badges367 silver badges417 bronze badges

1 Comment

StudioTime Over a year ago

Slick - reduced me even more!

Alexander Nied · Accepted Answer · 2017-05-07 18:47:53Z

I think you're getting mixed up because you're changing the length of the array while looping through it. This solution uses a third array to push results to w/o changing the original arrays.

Also, a few quick reminders:

Use for...in only for iterating over object key/value pairs-- otherwise use a standard for loop.
Don't forget to declare your variables w/ a var statement --otherwise you're going to pollute the global namespace.

var fruits = ["Banana", "Naranja", "Manzana", "Mango", "Fresa", "Limón", "Lima", "Sandia"];
var vegetales = ["", "Ajo", "", "Cebolla", "Tomate", "", "", "Zanahoria"];

var i;

var outputArr = [];

// The output should be this:
// ["Naranja", "Mango", "Fresa", "Sandia"]

function myFunction() {
  for (i=0; i<vegetales.length; i++) {
    if (vegetales[i] !== '') {
      outputArr.push(fruits[i]);
    }
  }
  console.log(outputArr);
}

myFunction();

arshovon · Accepted Answer · 2017-05-07 18:47:01Z

Simple solution:

var fruits = ["Banana", "Naranja", "Manzana", "Mango", "Fresa", "Limón", "Lima", "Sandia"];
var vegetales = ["", "Ajo", "", "Cebolla", "Tomate", "", "", "Zanahoria"];
var result = [];
for(var i = 0;i<fruits.length;i++){
    if(vegetales[i]!=""){
        result.push(fruits[i]);
    }
}
console.log(result);

Output:

["Naranja", "Mango", "Fresa", "Sandia"]

Run here:

var fruits = ["Banana", "Naranja", "Manzana", "Mango", "Fresa", "Limón", "Lima", "Sandia"];
var vegetales = ["", "Ajo", "", "Cebolla", "Tomate", "", "", "Zanahoria"];
var result = [];
for(var i = 0;i<fruits.length;i++){
	if(vegetales[i]!=""){
		result.push(fruits[i]);
	}
}
console.log(result);

N.B.: Modification can be made.

Collectives™ on Stack Overflow

Remove elements from an array according to empty elements of another array

4 Answers 4

Comments

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related