Thread: Calculations not being displayed, everything else working fine?

Do you notice that your posted code looks bad? That's because you didn't post it as plain text.

My code is probably longer than it needs to be, but I'm really focusing on readability and using each function for one purpose.

Longer is not generally more readable. Consider Mark Twain's famous quote: "I apologize for such a long letter - I didn't have time to write a short one."

Your names are pretty bad. There's certainly no need to prefix functions with fn. And "goAhead", "reactTo" ... what?

There's no need to pass "input" (uninitialized!) into reactTo. Just use a local variable in the function.

What is the output supposed to be for an input of 12999? Is it supposed to be truncated as 12k, or rounded up as 13k? The following assumes truncation.

Code:

#include <stdio.h>
 
int main() {
    char prefixes[] = {
        'k', 'M', 'G', 'T', 'P', 'E'
    };
 
    printf("Enter a number: ");
    unsigned long n;
    scanf("%lu", &n);
 
    int i = 0;
    for ( ; n >= 1000; ++i)
        n /= 1000;
 
    printf("%lu", n);
    if (i > 0) putchar(prefixes[i - 1]);
    putchar('\n');
 
    return 0;
}

any help appreciated!

Guess not.
Good luck with that.

It's such a crybaby thing to whine that I didn't solve every single one of your requirements.
Here you go:

Code:

#include <stdio.h>
 
int main() {
    char prefixes[] = { 'k', 'M', 'G', 'T', 'P' };
 
    while (1) {
        unsigned long n;
 
        while (1) {
            printf("Enter a number: ");
            if (scanf("%lu", &n) != 1) {
                printf("Invalid input. Try again.\n");
                for (int c; (c = getchar()) != EOF && c != '\n'; ) ;
            }
            else if (n <= 1000000000000000)
                break;
            printf("Too large. Try again.\n");
        }
 
        if (n == 0) break;
 
        int i = 0;
        for ( ; n >= 1000; ++i)
            n /= 1000;
 
        printf("%lu", n);
        if (i > 0) putchar(prefixes[i - 1]);
        putchar('\n');
    }
 
    return 0;
}

Originally Posted by C-UL8R

The fn prefix on function names was at the suggestion of my professor.

You probably misinterpreted your professor: perhaps it was a suggestion for a metasyntactic name like foo, bar, baz, or func that might be used when we want to name a function so as to demonstrate something about language syntax. You shouldn't be using it as a function name prefix as it clutters the name: we can already see that it is a function from either the declaration, or usage, or verb-based naming, or simply by checking the type with the code introspection capabilities of modern IDEs.

This seems pretty cryptic and incorrect:

Code:

int fnCheck(long double d) {

  int isDouble = scanf(" %Lf", &d);
  int isWithinBounds = d >= MIN && d < MAX;
  int status = d ? (isDouble + isWithinBounds) * isDouble : 3;
  return status;
  
}

It would be much easier to understand if you used named constants and didn't try to compute the status code:

Code:

enum status_code {
    INPUT_SUCCESS,
    INPUT_END,
    INPUT_READ_ERROR,
    INPUT_RANGE_ERROR
};

enum status_code readNumber(long double *num) {
    if (scanf(" %Lf", num) != 1) {
        return INPUT_READ_ERROR;
    }
    if (*num == 0) {
        return INPUT_END;
    }
    if (*num < MIN || *num > MAX) {
        return INPUT_RANGE_ERROR;
    }
    return INPUT_SUCCESS;
}

Now you can use it like this in the main function:

Code:

int end = 0;
do {
    puts("Please enter a number: ");
    long double num;
    switch (readNumber(&num)) {
    case INPUT_SUCCESS:
        puts(fnCalculateFor(num));
        break;
    case INPUT_READ_ERROR:
        puts("\nInvalid input. Shutting down...");
    case INPUT_END:
        end = 1;
        break;
    case INPUT_RANGE_ERROR:
        puts("\nNumber must be in range 1 ... 1e15.");
    }
} while (!end);

As for why:

• Named constants make it easier to understand the code than magic numbers.
• Your fnCheck function was primarily reading input rather than merely checking it: the check is input validation, important but secondary to the main goal of reading input. Hence, readNumber is a better name.
• Your fnCheck function provided a long double parameter, which meant that any changes to that variable was merely local. readNumber provides a pointer to long double parameter so that reading into that variable changes the variable from the caller, which is what you want. An alternative as john.c mentioned is "just use a local variable in the function", but for what it looks like you had in mind (output parameter + status code return), that doesn't work (unless you assign the local variable to the output parameter).
• Naming issues aside, fnReactTo isn't entirely a bad idea as it can be useful to abstract away the body of the main loop. In this case though, your main function is already so small that it doesn't help.
• I renamed g to end (the descriptive name makes its purpose clearer) and inverted its use as a flag, and because it is a flag I simply set it to 0 or 1 rather than trying to do a fancy and unnecessary decrement. You can also #include <stdbool.h> and use bool, true, and false instead of int, 1, and 0 respectively.

I would normally be a little wary about comparing floating point numbers with == but in the case of 0 as read by scanf you probably won't run into floating point inaccuracy.