My function keeps returning 0 instead of the actual calculated return value?

Question

#include <stdio.h>

//function prototypes
const double fiveNinths = 5 / 9;

int toCelsius(double value) 
{
    return ((value - 32) * (fiveNinths));
}

int toFarenheit(double value) 
{
    return (value * 1.8) + 32;
}


int main (void) 
{
    int intValue, menuSelect; 
    double results;
    intValue = 1;

    while (intValue > 0) 
    {
        printf( "enter a positive integer: \n");
        scanf("%d", &intValue);

        if (intValue > 0) 
        {
            printf("Enter 1 to convert to Farenheit, enter 2 to convert to Celsius: \n");
            scanf("%d", &menuSelect);

            if (menuSelect == 1)
            {
                results = toFarenheit(intValue);
                printf("%d degrees to Farenheit is %d\n", intValue, results);
            }
            else if (menuSelect == 2)
            {
                results = toCelsius(intValue);
                printf("%d degrees to Celsius is %d\n", intValue, results);

            }
            else
            {
                printf("invalid menu item, please input only 1 or 2\n");
            }
        }
    }

    return 0;
}

I can't figure out why my toCelsius value always returns 0. my toFarenheit function works just fine. Also, this program terminates by having the user enter in a negative integer value; it's part of the homework assignment so disregard that part.

Please read on how to provide [mcve]. Your question lacks any input/output values to support your claim. — user694733, Dec 09 '16 at 08:34
This exact question has been asked thousand times before. The Celsius/Farenheit program is a classic example used when teachers want to teach how floating point numbers work. Please do some research before posting a question. See this: http://stackoverflow.com/search?q=%5Bc%5D+celsius+farenheit+5%2F9 — Lundin, Dec 09 '16 at 09:13

inverted_index · Answer 1 · 2016-12-09T08:44:10.507

0

Try to using:

const double fiveNinths = 5.0 / 9.0;
double toCelsius(double value) {
    return ((value - 32.0) * (fiveNinths));
}

By this line : double results = toCelsius(double value), the method should return a double value, not an ineger value.

When you just use 9 or 5 you actually do integer division, So it returns the integer value 0, neither the right double value.

edited Dec 09 '16 at 08:44

answered Dec 09 '16 at 08:38

inverted_index

2,329
21
40

My function keeps returning 0 instead of the actual calculated return value?

1 Answers1