Ummm, I think you need to go back and read your C/C++ reference manuals. You are confusing '=' (assignment) with '==' (equality).
For example:
if (randomaction = 0) //if the random number is 0
{
// ...
}
else if (randomaction = 1) //if it's 1
{
// ....
}
This will always do the second test, because you have first set randomaction to 0, and then tested it for it being non-zero, and it isn't, so you go onto the else if. Then you assign 1 to randomaction, and test whether that is true, and it is.
I also tend to think that using more LEDs all together will probably just give you a marginally more powerful light. If you want more distance, I would think you would want to use an opto-isolator to drive a more powerful light that uses a higher power source than the Arduino supplies.