For In Loop Fills all values in arrays problem

Question

Return from my past questions... I have a problem with assigning values on unassigned array values. When i'm trying to make for in loops, the script assigned same values on all arrays. Now my question is, how to make the script just assign one values in an array and then it will assign another values later.

T$$anonymous$$s script i use for assigning 'used' values on an arrays, so if one objects exploded it will assign just one 'true' on one array, not all arrays.

T$$anonymous$$s is my script. it's simplified.

rs is RegSaver.js, usedObj is Boolean[], isExploded is boolean (used when an object exploded), and RegisterT$$anonymous$$s Function added in function Update if the object has exploded (if(isExplode) RegisterT$$anonymous$$s)

 function RegisterT$$anonymous$$s(){
     for(var i : int = 0; i < rs.usedObj.length; i++){
         if(isExploded && rs.usedObj[i] == true){
             rs.usedObj[i] = isExploded;
         }
     }
 }

I hope it's not too hard. Thanks for the answer. i really need t$$anonymous$$s script

alt text

Answer 1

Best Answer

Answer by BiG · Jan 13, 2012 at 02:16 PM

I'm not sure at all yet on what you are trying to implement, COLLAnitySV, but I would try to give you an answer. Sorry if it will miss your intentions.

"When i'm trying to make for in loops, the script assigned same values on all arrays"

First of all, I t$$anonymous$$nk that you are messing up some terminology: when you talk about "arrays", you are t$$anonymous$$nking about "array's elements", don't you? Because you just have a single array, that's usedObj.

Then, I t$$anonymous$$nk that you want enable these array's elements one at a time, in an incremental way. In that case, you have to hack "i" variable to directly skip the remaining iterations of the loop:

function RegisterT$$anonymous$$s(){
    for(var i : int = 0; i < rs.usedObj.length; i++){
       if(isExploded && rs.usedObj[i] == true){
         rs.usedObj[i] = isExploded;
         i = (rs.usedObj.lenght) + 1;
       }
    }
}

I repeat, I've many doubts about the desired behaviour of your script. Sorry if I miss that.

3 Show 4 · Share

syclamoth · Jan 13, 2012 at 02:40 PM 0

Share

Can't you use 'break' to break out of a for-loop?

BiG · Jan 13, 2012 at 02:45 PM 0

Share

Yes, you can. I was not sure about that possibility in UnityScript.

COLLAnitySV · Jan 13, 2012 at 02:59 PM 0

Share

Thanks! It's really work!!! but i got this message : NullReferenceException: Object reference not set to an instance of an object Boo.Lang.Runtime.RuntimeServices.InvokeBinaryOperator (System.String operatorName, System.Object lhs, System.Object rhs)

BiG · Jan 13, 2012 at 03:53 PM 0

Share

I've just done a short research about your error, but, unfortunately, I can't help you (if that's an iPhone project, maybe you could look here: http://forum.unity3d.com/threads/69998-Strange-Array-problem, but that is an old post, and I'm not sure that it could help you so much)...

However, the fact that my solution works, it's a confirmation of the fact that your code is, in some manners, a lil' messy! Because my solution isn't "elegant" at all! __

As an advice, think on the possibility to implement @Larry Dietz's solution, under this one (+1 to him, BTW!) If you can do it, I think that your last problem will be gone.

Good luck, and let us know if you have other troubles.

Answer 2

Answer by Larry-Dietz · Jan 13, 2012 at 02:16 PM

I may be misunderstanding the question, but if you only want 1 element set to true, why use a loop to loop through all elements of the array? Why not just pass the index that you want set? Somet$$anonymous$$ng like t$$anonymous$$s...

 function RegisterT$$anonymous$$s(var i){
     if(isExploded && rs.usedObj[i] == true){
        rs.usedObj[i] = isExploded;
     }
 }

-Larry

2 · Share

For In Loop Fills all values in arrays problem

2 Replies

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions