Dice Roller

I seem to be having problems with a dice rolling program that i am making. It rolls 2 6-sided dice, and i can get the random numbers no problem, its just making the program display the actual face of the die. when i run the program, it shows 2 of each die, even though the random numbers are random

heres the whole program, any ideas on how to fix the display part are greatly appreciated.

Code:

/*
 *  diceroller.c
 *  Dice Rolling Program
 *
 *  Created by Koenig, Robert on 5/6/08.
 *
 */

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int main(void)
{
	printf( "Press enter to continue\n" );
	getchar();
	srand((unsigned)time(NULL)); 
	int d1=rand()%6; 
	int d2=rand()%6;
	d1 = d1+1;
	d2 = d2+1;
	if (d1 == 1);
	{
		printf("+---------+\n");
		printf("|         |\n");
		printf("|   (-)   |\n");
		printf("|         |\n");
		printf("+---------+\n");
	}
	if (d1 == 2);
	{
		printf("+---------+\n");
		printf("|(-)      |\n");
		printf("|         |\n");
		printf("|      (-)|\n");
		printf("+---------+\n");
	}
	if (d1 == 3);
	{
		printf("+---------+\n");
		printf("|      (-)|\n");
		printf("|   (-)   |\n");
		printf("|(-)      |\n");
		printf("+---------+\n");
	}
	if (d1 == 4);
	{
		printf("+---------+\n");
		printf("|(-)   (-)|\n");
		printf("|         |\n");
		printf("|(-)   (-)|\n");
		printf("+---------+\n");
	}
	if (d1 == 5);
	{
		printf("+---------+\n");
		printf("|(-)   (-)|\n");
		printf("|   (-)   |\n");
		printf("|(-)   (-)|\n");
		printf("+---------+\n");
	}
	if (d1 == 6);
	{
		printf("+---------+\n");
		printf("|(-)   (-)|\n");
		printf("|(-)   (-)|\n");
		printf("|(-)   (-)|\n");
		printf("+---------+\n");
	}
	printf("\n");
	if (d2 == 1);
	{
		printf("+---------+\n");
		printf("|         |\n");
		printf("|   (-)   |\n");
		printf("|         |\n");
		printf("+---------+\n");
	}
	if (d2 == 2);
	{
		printf("+---------+\n");
		printf("|(-)      |\n");
		printf("|         |\n");
		printf("|      (-)|\n");
		printf("+---------+\n");
	}
	if (d2 == 3);
	{
		printf("+---------+\n");
		printf("|      (-)|\n");
		printf("|   (-)   |\n");
		printf("|(-)      |\n");
		printf("+---------+\n");
	}
	if (d2 == 4);
	{
		printf("+---------+\n");
		printf("|(-)   (-)|\n");
		printf("|         |\n");
		printf("|(-)   (-)|\n");
		printf("+---------+\n");
	}
	if (d2 == 5);
	{
		printf("+---------+\n");
		printf("|(-)   (-)|\n");
		printf("|   (-)   |\n");
		printf("|(-)   (-)|\n");
		printf("+---------+\n");
	}
	if (d2 == 6);
	{
		printf("+---------+\n");
		printf("|(-)   (-)|\n");
		printf("|(-)   (-)|\n");
		printf("|(-)   (-)|\n");
		printf("+---------+\n");
	}
	printf("\n");
	printf("Die 1 = %i\n", d1);
	printf("Die 2 = %i\n", d2);
	return 0;
}

Remove the semi-colon from the end of all the conditionals, eg here:

Code:

if (d1 == 1); /*<-- bad semi-colon!*/

You may wish to consider shifting the repetitive code to a function and to replace all those ifs with a switch statement.

ill keep that in mind for when i do if-then-else statements in the future.
thanks for the help

Mind if I show off my dice drawing algorithm? Basically 11 lines to do what your entire program did, consisting of the drawdice function and the 2 array definitions above it.

I'm not trying to brag here... no, yes I am.

(I'm not really that big a jerk. Joking, 'k. Just wanted to share an alternative solution I came up with awhile ago.)