cant find array inside array in javascript

Question

this is the snippet: this code is not working, the problem it is not recognizing the array inside the array.

how can i improve it?

thanks!

<!DOCTYPE html>
<html>
<body>

<p>The new ECMASCRIPT 5 method isArray returns true when used on an array.</p>

<p id="demo"></p>

<script>
function findApple(arr, item) {
  var count = 0;
  for (var i = 0; i < arr.length; i++) {
    if (Array.isArray(arr[i])) {
       findApple(arr[i], item);
    }
    else if (arr[i] == "apple") {
      count++;
    }
  }
  return count;
}
var one = ["apple", "apple", ["orange", "apple", "banana"]];

var data = findApple(one, "apple");
alert(data);
var item = document.getElementById("demo").innerHTML = data;
</script>

</body>
</html>

The problem is here findApple(arr[i], item); - you're not using the return value. — georg
– georg, Commented Sep 28, 2016 at 13:54
it is not recognizing the array inside the array yeah, I don't think that is correct. — VLAZ
– VLAZ, Commented Sep 28, 2016 at 13:56
on a side note, shouldn't the apple in the else if be item otherwise there is no point in passing the item var in — Pete
– Pete, Commented Sep 28, 2016 at 13:58

ASDFGerte · Accepted Answer · 2016-09-28 13:55:49Z

1

Use count += findApple(arr[i], item);

answered Sep 28, 2016 at 13:55

ASDFGerte

5,2146 gold badges20 silver badges34 bronze badges

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

Comments

Jonas Wilms · Accepted Answer · 2016-09-28 13:57:27Z

0

You have the following problem:

if (Array.isArray(arr[i])) { findApple(arr[i], item); }

As you wrote the script, you know that findApple RETURNS something. So the right syntax would be:

var result=findApple();

As you want to pass it back to the callee simply do:

 count+=result;

answered Sep 28, 2016 at 13:57

Jonas Wilms

139k20 gold badges164 silver badges164 bronze badges

Comments

Vladu Ionut · Accepted Answer · 2016-09-28 13:56:44Z

function findApple(arr, item) {
  var count = 0;
  for (var i = 0; i < arr.length; i++) {
    if (Array.isArray(arr[i])) {
       count+=findApple(arr[i], item);
    }
    else if (arr[i] == "apple") {
      count++;
    }
  }
  return count;
}
var one = ["apple", "apple", ["orange", "apple", "banana"]];

var data = findApple(one, "apple");
alert(data);
var item = document.getElementById("demo").innerHTML = data;

<!DOCTYPE html>
<html>
<body>

<p>The new ECMASCRIPT 5 method isArray returns true when used on an array.</p>

<p id="demo"></p>



</body>
</html>

Collectives™ on Stack Overflow

cant find array inside array in javascript

3 Answers 3

Comments

Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Related