Issue with for loop in an Array

Question

Initially the array holds the default values as 100. Once if the enemy looses its health then i want to set the default values from 100 to 0. When all the array elements gets a value of 0 then the message will trace as game win.

var enemyHealth:Array = new Array(100,100,100,100,0);

    for (var i = 0; i< enemyHealth.length; i++)
    {

        if (enemyHealth[i] == 0)
        {
            trace ("game win");
        }

    }

Actually the issue is if any one of the array element holds a value of 0 then it trace a massage as game win.

So can any one help me in this regard.

do you want an if statement for checking when ALL the elements are 0? — Joe Slater
– Joe Slater, Commented Feb 23, 2013 at 12:53

Sergey Kalinichenko · Accepted Answer · 2013-02-23 12:54:33Z

1

You need to check all elements, not just one:

var allZero:Boolean = true;
var enemyHealth:Array = new Array(100,100,100,100,0);

for (var i = 0; i< enemyHealth.length; i++)
{

    if (enemyHealth[i] != 0)
    {
        allZero = false;
        break; // We know it's not a win, so we can stop the loop early
    }

}
if (allZero) {
    trace ("game win");
}

answered Feb 23, 2013 at 12:54

Sergey Kalinichenko

729k85 gold badges1.2k silver badges1.6k bronze badges

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

1 Comment

Chandu Over a year ago

Thanks Mr dasblinkenlight its worth for me.

Alexander Cogneau · Accepted Answer · 2013-02-23 12:54:16Z

1

I would do something like this

var enemyHealth:Array = new Array(100,100,100,100,0);

var isDead:Boolean = true;
for (var i = 0; i< enemyHealth.length; i++)
{

   if (enemyHealth[i] != 0)
   {
        isDead = false;
        break;
   }
}

if(isDead)
{
    trace ("Game Win");
}

answered Feb 23, 2013 at 12:54

Alexander Cogneau

1,2744 gold badges12 silver badges25 bronze badges

Comments

Joe Slater · Accepted Answer · 2013-02-23 12:58:50Z

1

You can do what the other answerers have said or something like this which might help you more to get the exact amount of enemies dead.

var enemyHealth:Array = new Array(100,100,100,100,0);
var enemiesDead:int=0;

    for (var i = 0; i< enemyHealth.length; i++)
    {

        if (enemyHealth[i] == 0)
        {
            enemiesDead++;
        }

    }

if(enemiesDead==enemyHealth.length)
{
    trace("Game Over");
}

answered Feb 23, 2013 at 12:58

Joe Slater

2,4836 gold badges36 silver badges50 bronze badges

Comments

Benoît Guédas · Accepted Answer · 2013-02-23 13:45:11Z

1

You can use the every method to check that every element of your array (or vector) meet a criterion.

const enemyHealth:Vector.<uint> = new <uint>[100,100,100,100,0];

const zeroHealth:Function = function(item:uint, index:int, vector:Vector.<uint>):Boolean {
   return item === 0;
}

if (enemyHealth.every(zeroHealth)) {
    trace("Game win")
}

I have changed the array to a Vector because they are more efficient, and you can specify the type of the elements, but it also works fine with array.

edited Feb 23, 2013 at 13:45

answered Feb 23, 2013 at 13:34

Benoît Guédas

8017 silver badges25 bronze badges

Collectives™ on Stack Overflow

Issue with for loop in an Array

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related