Thread: Can someone please correct my code?

Originally Posted by Alex Coven

Making a simple program for a school assignment and I cant seem to figure it out... its a basic math function using the exponential growth formula which is correct but the computer is not outputting the right numbers, any help would be appreciated!

Code:

#include <stdio.h>
#include <math.h>


int main()
{
    int blanketsquares, num1, num2, blanket,remainder;
    printf("How many people are knitting blanket squares in the beginning of the week?\n\n"); //delete \n\n and put a space after ? Nicer format
    scanf("%d", &num1);


    printf("How many people are knitting blanket squares each day?\n\n");   //delete \n\n and put a space after ? Nicer format
    scanf("%d", &num2);


//forumulas


    blanketsquares = pow(num1 + num2,7); //put 7.0 instead of 7
    blanket = blanketsquares / 60;
    remainder = blanketsquares % 60;


//statement


    printf("%d blanketsquares will be made this week!\n\n", &blanketsquares); //get rid of &
    printf("You will be able to make %d blankets and start next week with %d squares.\n\n\n\n", blanket, remainder);




  return 0;
}

Hi,
I have found a solution but don't know the details. Perhaps someone more experienced can fill in the blanks..

Please see this website. Both arguments must be of type double
pow() - C Library Function Example

From above, the second argument must have a decimal so 7.0 instead of 7. Not sure what is happening with the first value (base value).

Also, printf does not have an '&' --> second last printf. Please correct

The pow statement is okay as it is. The compiler will automatically "promote" the int values to doubles. https://www.securecoding.cert.org/co...nversion+rules

The ampersand is the big error, although the overall logic is weird.

Actually, if you pass an integer to a function that expects a double, the integer gets automatically promoted to a double.

The problem with this program is that you're printing the address of blanketsquares rather than the contents of the variable.

Originally Posted by bos1234

As above solutions, code runs fine as it is. Change printf statement
However, on visual studio the pow function is underlined. Intellisense says...

That means you're compiling it as C++.

Can someone please correct my code?

No, we do not directly correct code; We correct the coder!