Posts
2,789
Respect
230Add +1
Forum Rank
King of the Zombies
Primary Group
Community Daedra
UGX-Mods Login
Only three or four array values work?
0 Members and 1 Guest are viewing this topic.
3,061 views
- Topic Options
Reply- Topic Options
daedra descent - Community Daedra
- 2,789 Posts
- 230 Respect +1
×
☭ Soviet Commander ☭
Location: 
Date Registered: 13 August 2012
Last active: 10 years ago
My Groups
More
My Contact & Social Links
More
Signature
Let's keep this thread on topic from here on in. -DBZ
+1 to off-topic reply -DBZ
lmao. Too funny.
Goliath Script Placer: http://ugx-mods.com/forum/index.php/topic,11234.msg125257/topicseen.html#new
"...Christ, people. Learn C, instead of just stringing random characters
together until it compiles (with warnings)..."
-Linus Torvalds
+1 to off-topic reply -DBZ
lmao. Too funny.
Goliath Script Placer: http://ugx-mods.com/forum/index.php/topic,11234.msg125257/topicseen.html#new
"...Christ, people. Learn C, instead of just stringing random characters
together until it compiles (with warnings)..."
-Linus Torvalds
Only three or four of my array values work, and i don't know why. If i switch the order the ones that are in the front three or four work, while the rest don't.
code:
code:
Code Snippet
Plaintextlevel.random = randomint(level.rm.size);
if(level.random == 1 && iscurrent != level.rm[0])
{
level.powerup = level.rm[0];
iscurrent = level.powerup;
}
if(level.random == 2 && iscurrent != level.rm[1])
{
level.powerup = level.rm[1];
iscurrent = level.powerup;
}
if(level.random == 3 && iscurrent != level.rm[2])
{
level.powerup = level.rm[2];
iscurrent = level.powerup;
}
if(level.random == 4 && iscurrent != level.rm[3])
{
level.powerup = level.rm[3];
iscurrent = level.powerup;
}
if(level.random == 5 && iscurrent != level.rm[4])
{
level.powerup = level.rm[4];
iscurrent = level.powerup;
}- Post Options
- Reply
- treminaor
- UGX Administrator
- 5,544 Posts
- 6,646 Respect +1
×
[UGX] Founder
Location: Bay Area, California
Bay Area, CaliforniaDate Registered: 24 June 2011
Last active: 3 years ago
My Groups
More
My Contact & Social Links
More
Signature
If Java had true garbage collection, most programs would delete themselves upon execution.
I have no idea what this is for but the code looks overcomplicated. Give more context, you dont even show your declaration of level.rm.
- Post Options
- Reply
level.random tested for 1,2,3,4,5
possible outcomes of randomint(level.rm.size);
0,1,2,3,4
Post Merge: April 30, 2014, 02:30:02 am
dont know whats going on here but change to this
possible outcomes of randomint(level.rm.size);
0,1,2,3,4
Post Merge: April 30, 2014, 02:30:02 am
dont know whats going on here but change to this
Code Snippet
Plaintextlevel.random = randomint(level.rm.size);
if(level.random == 0 && iscurrent != level.rm[0])
{
level.powerup = level.rm[0];
iscurrent = level.powerup;
}
if(level.random == 1 && iscurrent != level.rm[1])
{
level.powerup = level.rm[1];
iscurrent = level.powerup;
}
if(level.random == 2 && iscurrent != level.rm[2])
{
level.powerup = level.rm[2];
iscurrent = level.powerup;
}
if(level.random == 3 && iscurrent != level.rm[3])
{
level.powerup = level.rm[3];
iscurrent = level.powerup;
}
if(level.random == 4 && iscurrent != level.rm[4])
{
level.powerup = level.rm[4];
iscurrent = level.powerup;
}- Post Options
- Reply
- daedra descent
- Community Daedra
- 2,789 Posts
- 230 Respect +1
×
☭ Soviet Commander ☭
Location:
Date Registered: 13 August 2012
Last active: 10 years ago
My Groups
More
My Contact & Social Links
More
I have no idea what this is for but the code looks overcomplicated. Give more context, you dont even show your declaration of level.rm.
I suppose that would help, xD
Code Snippet
Plaintext#include common_scripts\utility;
#include maps\_utility;
#include maps\_zombiemode_utility;
main()
{
level.trigger = getent("powerup_machine","targetname");
level.rp = getent("random_model","targetname");
level.rm = [];
level.rm[level.rm.size] = "zombie_bomb";
level.rm[level.rm.size] = "zombie_carpenter";
level.rm[level.rm.size] = "zombie_ammocan";
level.rm[level.rm.size] = "zombie_skull";
level.rm[level.rm.size] = "zombie_x2_icon";
level.wait_time = 5;
level.trigger_cost = 100; // change this later
level thread start_randomize();
level thread trigger_setup();
}
trigger_setup()
{
while(1)
{
player = get_players();
level.trigger SetHintString( "Press F to activate random powerup machine[cost: " + level.trigger_cost + "]" );
level.trigger waittill("trigger", player);
if(isdefined(level.trigger) && player.score >= level.trigger_cost)
{
// rp_test();
player maps\_zombiemode_score::minus_to_player_score( level.trigger_cost );
player magic_init();
}
if(isdefined(level.trigger) && player.score < level.trigger_cost)
{
iprintln("Not Enough Money!");
player maps\_zombiemode_perks::play_no_money_perk_dialog();
}
wait(1);
}
}
magic_init()
{
rand = undefined;
level.cycles = 15;
model_cycle();
if(level.powerup == "zombie_ammocan")
{
maps\_zombiemode_powerups::full_ammo_powerup( "null" );
}
else if(level.powerup == "zombie_skull")
{
maps\_zombiemode_powerups::insta_kill_powerup( "null" );
}
else if(level.powerup == "zombie_carpenter")
{
players = get_players();
maps\_zombiemode_powerups::start_carpenter( players[0].origin );
}
else if(level.powerup == "zombie_bomb")
{
players = get_players();
maps\_zombiemode_powerups::nuke_powerup("null");
}
else if(level.powerup == "zombie_2x_icon")
{
maps\_zombiemode_powerups::double_points_powerup("null");
}
}
model_cycle()
{
cm = 0;
level.powerup = undefined;
iscurrent = current_watch();
while(cm != level.cycles)
{
level.random = randomint(level.rm.size);
if(level.random == 1 && iscurrent != level.rm[0])
{
level.powerup = level.rm[0];
iscurrent = level.powerup;
}
if(level.random == 2 && iscurrent != level.rm[1])
{
level.powerup = level.rm[1];
iscurrent = level.powerup;
}
if(level.random == 3 && iscurrent != level.rm[2])
{
level.powerup = level.rm[2];
iscurrent = level.powerup;
}
if(level.random == 4 && iscurrent != level.rm[3])
{
level.powerup = level.rm[3];
iscurrent = level.powerup;
}
if(level.random == 5 && iscurrent != level.rm[4])
{
level.powerup = level.rm[4];
iscurrent = level.powerup;
}
iprintln("Current Powerup: " + level.powerup);
level.rp setmodel( level.powerup );
cm++;
wait(.7);
}
}
start_randomize()
{
while(1)
{
// level.random = Randomint(level.rm.size);
wait(.1);
}
}
current_watch()
{
while(1)
{
last_powerup = level.powerup;
return last_powerup;
wait(.7);
}
}
The iscurrent is to stop the powerup from showing up twice, but it doesn't work.
level.random tested for 1,2,3,4,5
possible outcomes of randomint(level.rm.size);
0,1,2,3,4
Post Merge: April 30, 2014, 02:30:02 am
dont know whats going on here but change to thislevel.random = randomint(level.rm.size);
if(level.random == 0 && iscurrent != level.rm[0])
{
level.powerup = level.rm[0];
iscurrent = level.powerup;
}
if(level.random == 1 && iscurrent != level.rm[1])
{
level.powerup = level.rm[1];
iscurrent = level.powerup;
}
if(level.random == 2 && iscurrent != level.rm[2])
{
level.powerup = level.rm[2];
iscurrent = level.powerup;
}
if(level.random == 3 && iscurrent != level.rm[3])
{
level.powerup = level.rm[3];
iscurrent = level.powerup;
}
if(level.random == 4 && iscurrent != level.rm[4])
{
level.powerup = level.rm[4];
iscurrent = level.powerup;
}
Treyarch commented in the box function that its from 1-value, but i tried this anyway and it didn't work.
Post Merge: April 30, 2014, 03:51:15 am
NVM! i fixed it!
The issue was
Code Snippet
Plaintextlevel.random = randomint(level.rm.size);randomint does one less than the actual size, in which case is 4. This caused the last powerup to not be registered. I fixed it by doing this:
Code Snippet
Plaintextlevel.random = randomint(level.rm.size + 1);HOWEVER, the iscaused still isn't working. Could someone help me out with that?
- Post Options
- Reply
- YaPh1l
- Scripter Elite
- 277 Posts
- 112 Respect +1
The iscurrent is to stop the powerup from showing up twice, but it doesn't work.You only ever store the most recent one, to make sure you don't pick it again right afterwards. However, the iteration after that, it can be picked again, as iscurrent now holds the new last powerup.
Also:
start_randomize() is currently useless.
current_watch() is useless, too.
model_cycle() could be made a lot shorter.
- Phil.
- Post Options
- Reply
- daedra descent
- Community Daedra
- 2,789 Posts
- 230 Respect +1
×
☭ Soviet Commander ☭
Location:
Date Registered: 13 August 2012
Last active: 10 years ago
My Groups
More
My Contact & Social Links
More
You only ever store the most recent one, to make sure you don't pick it again right afterwards. However, the iteration after that, it can be picked again, as iscurrent now holds the new last powerup.
Also:
start_randomize() is currently useless.
current_watch() is useless, too.
model_cycle() could be made a lot shorter.
- Phil.
The issue is that it will randomly select the same powerup 2, 3, or even 4 times before randomizing again.
I have got rid of the unneeded functions, but what would i use to make model_cycle() shorter?
- Post Options
- Reply
- MakeCents
- Scripter
- 1,895 Posts
- 791 Respect +1
×
My Groups
More
My Contact & Social Links
More
Personal Quote
BE ORIGINAL
Signature
I have got rid of the unneeded functions, but what would i use to make model_cycle() shorter?
Can you use a for loop since the only thing different between the if statements is the number?
Code Snippet
Plaintext for(i=0;i<5;i++){
if(level.random == i+1 && iscurrent != level.rm[i])
{
level.powerup = level.rm[i];
iscurrent = level.powerup;
}
}
instead of
Code Snippet
Plaintext if(level.random == 1 && iscurrent != level.rm[0])
{
level.powerup = level.rm[0];
iscurrent = level.powerup;
}
if(level.random == 2 && iscurrent != level.rm[1])
{
level.powerup = level.rm[1];
iscurrent = level.powerup;
}
if(level.random == 3 && iscurrent != level.rm[2])
{
level.powerup = level.rm[2];
iscurrent = level.powerup;
}
if(level.random == 4 && iscurrent != level.rm[3])
{
level.powerup = level.rm[3];
iscurrent = level.powerup;
}
if(level.random == 5 && iscurrent != level.rm[4])
{
level.powerup = level.rm[4];
iscurrent = level.powerup;
}
- Post Options
- Reply
- YaPh1l
- Scripter Elite
- 277 Posts
- 112 Respect +1
When RandomInt() returns the same value a second time, your code doesn't do anything. Meaning that it will just keep the previous one for this cycle.
- Phil.
- Phil.
- Post Options
- Reply
- daedra descent
- Community Daedra
- 2,789 Posts
- 230 Respect +1
×
☭ Soviet Commander ☭
Location:
Date Registered: 13 August 2012
Last active: 10 years ago
My Groups
More
My Contact & Social Links
More
When RandomInt() returns the same value a second time, your code doesn't do anything. Meaning that it will just keep the previous one for this cycle.
- Phil.
So how do i make sure it doesn't give the same value twice? Do i need to use a different function for randomizing the model?
- Post Options
- Reply
- YaPh1l
- Scripter Elite
- 277 Posts
- 112 Respect +1
So how do i make sure it doesn't give the same value twice?Well, change your logic so that it handles this case properly. That's the challenge of programming.
Commonly, if you want a random number, but exclude a specific one in the range, you generate random numbers, until you get one that is not equal to the one you don't want.
You could also be using a randomized array in your case.
Etc. etc.
- Phil.
- Post Options
- Reply
- Topic Options
- Reply
- Topic Options