DND dice roller

[Moe_Lester]

what would good structure for a dice-roller be..
the dice-roller would be for DND...so could someone help me i am attempting to make one for my sessions..

[Bugdude]

What would good question for a forum be?

[Corsix]

I made a dice roller in VB a long time ago...

[schloob]

(rand() % 6)


BAM done -.-

*edit*
ahhh dvorak is killing me, fixed typo
*edit2*
added edit

[Corsix]

But what about D3s and D8s?
actually it would be

Code: Select all

(rand() % 6)+1

(1->6, not 0->5)

[schloob]

But what about D3s and D8s?
actually it would be

Code: Select all

(rand() % 6)+1

(1->6, not 0->5)

(rand() % d)

happy??

[schloob]

all the latest dice are 0-5

[Wizard]

Code: Select all

((rand() / RAND_MAX) * d) + 1;

is better, I think. More even distribution.

Code: Select all

public class Dice {
    public:
        Dice(int sides) {
            i_sides = sides;
        }
        Dice() {
            i_sides = 6;
        }
        int roll() {
            return ((rand() / RAND_MAX) * i_sides) + 1;
        }
    private:
        int i_sides;
};

There, nice little package.

[Darobat]

public class Dice {

I think that should just be

Code: Select all

class Dice {

[Wizard]

Dag Nabbit, Java syntax slipping in again. This didn't used to happen. I don't think I'm capable of programming in two different languages like this anymore

[Alvaro]

Code: Select all

((rand() / RAND_MAX) * d) + 1;

Are you sure that RAND_MAX is not an integer? Your dice may be getting just ones on some compilers...

[Wizard]

I knew that. I just didn't... um... want to give away everything, is all. Yeah.
Look, a decoy.

[Invictus]

Code: Select all

srand(time(0));
...
int Random(int min, int max)
{
   return rand()%(max+1-min)+min;
}


[edit1] Removes static call because I wasn't sure if it would do what I expected.
[edit2] Added edit1 because alvaro posted while I was editing.

[Alvaro]

Code: Select all

int Random(int min, int max)
{
   static srand(time(0));
   return rand()%(max+1-min)+min;
}


Good idea, but I think the syntax should be

Code: Select all

   static int dummy=srand(time(0));

...or something like that, right?

[Invictus]

You have to use a dummy? I'll test it later if no one answers it by the time I get my laptop back. I just told my brother that I would let him use it to write a report while I try to fix his comp.