Thread: Program Help

I am making a program that will take any two two-digit numbers determine the largest and display it in words using if/else and switch statements. I'm kinda stuck. I know how to determine the larger number but i am not sure how to make it into words. This is what i have so far:

Code:

#include <stdio.h>
int main(void)
{
    int a, b, i, onesDig, tensDig;
    tensDig = i/10;
    onesDig = i/10;
    char word;


    printf("Enter two two-digit numbers separated by a space: ");
    scanf("%d %d", &a, &b);
    if (a>b){
        a=i;}
    else
        b=i;

    switch (i){
        case 10: word = "ten"; break;
        case 11: word = "eleven"; break;
        case 12: word = "twelve"; break;
        case 13: word = "thirteen"; break;
        case 14: word = "fourteen"; break;
        case 15: word = "fifteen"; break;
        case 16: word = "sixteen"; break;
        case 17: word = "seventeen"; break;
        case 18: word = "eighteen"; break;
        case 19: word = "nineteen"; break; }

    switch (tensDig) {
        case 2: word = "twenty"; break;
        case 3: word = "thirty"; break;
        case 4: word = "forty"; break;
        case 5: word = "fifty"; break;
        case 6: word = "fifty"; break;
        case 7: word = "sixty"; break;
        case 8: word = "seventy"; break;
        case 9: word = "ninety"; break;
    }
    switch (onesDig) {
        case 1: word = "-one"; break;
    



    return 0;
}

Code:

    if (a>b){
        a=i;}
    else
        b=i;

Just noticed this is backwards. You want i = a; and i = b; to make i contain the greater of the two. A shorthand for this common if-else construct is the following:

Code:

i = a > b ? a : b;  // if a > b, assign i a, otherwise, assign it b

Thanks so far. I'm still confused on how to get it to print out the words. This is what i have now:

Code:

#include <stdio.h>
int main(void)
{
    int a, b, i, onesDig, tensDig;

    char* word;


    printf("Enter two two-digit numbers separated by a space: ");
    scanf("%d %d", &a, &b);
    if (a>b){
        a=i;}
    else
        b=i;
    onesDig = i%10;
    tensDig = i/10;

    switch (i){
        case 10: word = "ten"; break;
        case 11: word = "eleven"; break;
        case 12: word = "twelve"; break;
        case 13: word = "thirteen"; break;
        case 14: word = "fourteen"; break;
        case 15: word = "fifteen"; break;
        case 16: word = "sixteen"; break;
        case 17: word = "seventeen"; break;
        case 18: word = "eighteen"; break;
        case 19: word = "nineteen"; break; }

    switch (tensDig) {
        case 2: word = "twenty"; break;
        case 3: word = "thirty"; break;
        case 4: word = "forty"; break;
        case 5: word = "fifty"; break;
        case 6: word = "sixty"; break;
        case 7: word = "seventy"; break;
        case 8: word = "eighty"; break;
        case 9: word = "ninety"; break;
    }
    switch (onesDig) {
        case 1: word = "-one"; break;
        case 2: word = "-two"; break;
        case 3: word = "-three"; break;
        case 4: word = "-four"; break;
        case 5: word = "-five"; break;
        case 6: word = "-six"; break;
        case 7: word = "-seven"; break;
        case 8: word = "-eight"; break;
        case 9: word = "-nine"; break;
    }
    

    return 0;
}

How about printf(word);?

I tried printf("%s", word); and it doesn't print anything. I tried printf(word); and put in 10 and 11 and it displayed -four

I've gotten completely lost lol. I'm not really sure how to use your codes from post 2 and 3.

Since you're a noob, skip the weird ?: business for now. Stick with:

Code:

if (a > b)
    i = a;  // i is on the left, so we assign to i the value in a
else
    i = b;

Otherwise, i is uninitialized and it has some random garbage value.

You don't actually need a "word" variable to store a string, just print it directly, like so:
printf("seventeen");

Then, pay attention to the indentation I used, similar to that of a regular program. My pseudo code has the same structure as the logic you need for you program. Here is something a little closer to the end product:

Code:

if (num >= 10 && num <= 19) {
    // switch statement that prints "ten" through "nineteen"
}
else {
    if (tensDig != 0) {
        // switch statemet to print "twenty" through "ninety"
    }
    ...
}

I'm leaving the rest for you to fill in. Please really think through the logic and code I provided, and run through some examples by hand to get a feel for how the if and switch statements control what parts of the program run.

Ok thanks for the help I should be able to get it now.

Try being a little more consistent and sensible about your { braces }. Pick a good indentation/brace style (K&R, 1TBS, Allman and BSD KNF are the most common). It might help to put braces around all if blocks and else blocks too, to avoid ambiguity and the dangling else problem.

You are largely on the right track, except you need to drop the 'else' before the if (onesDig).

I fixed the else problem and dropped the else before the if (onesDig) but I don't know why the output seems to be just a random onesDig case; the program will not output ten-nineteen or twenty-ninety just the onesDig

Code:

    if (i>=10 && i<=19);{

Why is there a semi-colon in there?