I have to write a program that converts Numbers to Roman Numerals. Now I have already done it but I want to know if there is an easier way to program it. I would like not to have to use pointers in my program could some please show me how this would be done. And yes this is for school #include #include #include void main () { int decimal; char romanDigits[8] = "mdclxvi"; char* currentDigit = romanDigits; int romanValues[8] = {1000, 500, 100, 50, 10, 5, 1}; int* currentValue = romanValues; /* longest Roman numeral < 5000 has 19 digits. */ char roman[20] = "-------------------"; /* hyphens are ddebugging */ char* currentRoman = roman; printf ("Please type a positive integer less than 5000: "); scanf ("%d", &decimal); if (decimal <= 0) { exit(1); } while (decimal > 0) { if (decimal >= *currentValue) { *(currentRoman++) = *currentDigit; decimal = decimal - *currentValue; } else { currentDigit++; currentValue++; } } *currentRoman = 0; printf("Roman numeral equivalent = %s\n", roman); }

Two things:

First, rather than having CurrentValue being a pointer, i would use it as an index.
ie:

int CurrentValue = 0

if(decimal>=romanValues[CurrentValue])
{
*(currentRoman++)=currentDigit[CurrentValue];
decimal = decimal - romanValues[CurrentValue];
} else { currentValue++;
}

notice that you can use this as in index into both the currentvalue and the current digit ?

The other thing is that your implementation is wrong:
it would come up with xxviiii for 19, when it should be xix.
and iiii for 4, when it should be iv.
ie, you can only have 3 of one character in a row (except for m)
so you could add a check to see if it''s more than 3 times the current Romanvalue

hope that helps

Wyzfen