Randomise trigger

[wewso]

This is a trigger, which should give each of 12 integers in one array a unique random number from 1 to 12. The problem is that it doesnt work propperly

Varaibles I use:
Random_Numbers - Integer Array
Random_Final - Integer Array
Temp_Integer - Integer

The init:

Init
    Events
        Map initialization
    Conditions
    Actions
        For each (Integer A) from 1 to 12, do (Actions)
            Loop - Actions
                Set Random_Numbers[(Integer A)] = (Integer A)

The bugged trigger:

Randomize
    Events
        Player - Player 1 (Red) skips a cinematic sequence
    Conditions
    Actions
        For each (Integer A) from 1 to 12, do (Actions)
            Loop - Actions
                Set Temp_Integer = (Random integer number between 1 and (13 - (Integer A)))
                Set Random_Final[(Integer A)] = Random_Numbers[Temp_Integer]
                Set Random_Numbers[Temp_Integer] = (13 - (Integer A))
                Game - Display to (All players) the text: (String(Random_Final[(Integer A)]))

Thanks for hlp guys :thup:

 

[UndeadDragon]

Why are you doing: (Random integer number between 1 and (13 - (Integer A))), why don't you just do a random number between 1 and 12?

 

[wewso]

Why are you doing: (Random integer number between 1 and (13 - (Integer A))), why don't you just do a random number between 1 and 12?

That would generate random, but non-unique numbers, that might get repeated

 

[AceHart]

> Set Random_Numbers[Temp_Integer] = (13 - (Integer A))

You sure this shouldn't be:
Set Random_Numbers[Temp_Integer] = Random_Numbers[(13 - (Integer A))]


> The problem is that it doesn't work

"It doesn't work" is not a very good problem description.

 

[wewso]

oh now I see my mistake :thup: tnx ace

I wrote it in my bioligy class, but I haven t rewritten it correctly to the comp.
Actualy the lecture was about deseases like leprosy and plague and it was very good to think about stg else.