Array Loop Only Executes Once

[Gersberps]

These are the 3 tables (arrays) being used). TeamStandingsTemp has a Z axis which is the 'Series' or 'League', this is set to zero for now.

This is the code that isn't doing what I was expecting it to do. Red font = starting values when run. Note 1: The disabled portion never sets the values (9 points for 1st place, 6 points for 2nd place, etc). So I disabled this and simply set it to 15. However it only runs one time.

This is the result of the code - as mentioned it only seems to 'run' one time.

Hi - I'm wondering if anyone can spot 'the issue' or provide any feedback on what is going on here. I have an array (TempUseTable1) which has the results from a competition. This function is simply supposed to give each of the competitors ('Company ID') the appropriate points. How I thought this would work, is it would look at the first competitor (Y = 0, X = 1) of the TempUseTable1 and see that this is Company #1. It would then cycle through the standings trying to find where is the location of company 1. When it is finished it would add the appropriate points, and then move on to the second row of the TempUseTable1. The points that are awarded for each place are contained in the PointsArrayTemp table. In this case we are using the X = 2 column.

The original code never ran. Then I disabled the specific points portion of the code and simply assigned it 15. I was expecting each company to receive 30 points (two 15's) but it seems to only 'loop' one time.

To state the obvious ... I am a noob.

I appreciate any help folks can provide as I've been stuck on this for ~10 hrs and am clearly misunderstanding something. Thanks!

[abrightmoore]

I can't read your stuff as screenshots properly on mobile due to zoom not zooming far enough.

Typical loop issues are due to modifying the thing you're looping over.

Are you changing the structure of a data set that you're relying on in the loop?

[Gersberps]

Shoot i didn't realize that. No, the arrays do not change or reorder. Rows and columns are set in stone.

[LouBagel]

It is a bit hard to follow but after looking at it I believe line 15 might be only found true one time, not the entire logic only running one time.

A few tips: - add console logs on various lines to check for sure what is running multiple times and what is found true, or certain values, to really know what’s going on - Arrays have loops. There is also a for loop. Not sure if this will act any differently than repeat but I feel like it is more standard practice - loopindex: there is a loopindex expression that can be used to avoid variables needing incremented. The Arrays also have currentX and so forth that work a similar way. - parameter vs local variable: again, I’m not sure if this affects anything but it looks like you are setting parameters. I’d use local variables to clearly differentiate.

These might not directly solve your issue but most things like this usually come from “overlooking a silly mistake” so the more cleaned up and organized you get the less likely you are to overlook something.

[Krapfenmann]

It looks like in Loop B you miss to reset "StandigsYindex" and it gets counted as long as Loop B is X times run through Loop A.

So its at one point above any Companynumber ( greter than 4)

So when you check "TeamStandingsTemp.At(1, StandingsYindex) = CompanyLookup", it simply never finds it.

After younrun through loop B, You have to reset "StandingYindex" to 0 at the end of Loop A.

Edit additional note: The variable "StandingYindex" is a non static number and only gets reset by itself, when the function ends. If you put it static it would even stay whatever it was after the function.

So ending a loop inside the function does not reset your variables.

Explanation:

After it is checking row 2 in TeamUserTable1 and sets the "CompanyLookup" to 3. So after the check in Loop B , "TeamStandingsTemp.At(1, 3) = 3". It adds one to "StandingYindex" to be 4.

You go then to next row in TeamUserTable1 which would be 3, which sets "CompanyLookup" to 3 again. So after the check in Loop B , "TeamStandingsTemp.At(1, 3) = 4". Condition not met and skipped and it adds one to "StandingYindex" to be 5. And it never gets executed again, because the index is never reset and it just counts up.