Passing different values between multiple functions in C?

Question

I'm having trouble passing differing values between multiple functions. That is, i'm trying to have two functions take user inputs, and then have both of those inputs passed to a third function that calculates those inputs added together. Afterwards, it passes that total on to another function that calculates the true total. Finally, after that, the last function displays the final number. I can't seem to figure out how to make multiple functions pass though.

# include <stdio.h>
# include <stdlib.h>

int carLength(int userInput) //Length
{
int length = 0;
do
{
    printf("\nPlease input the length of the area being carpeted:");
    scanf("%d", &userInput);
} while (length = 0);
length = userInput; //Essentially using this to store user input so i can 
use it later.
return length;
}

int carWidth(int userInput) //Width
{
int width = 0;
do
{
    printf("\nPlease input the width of the area being carpeted:");
    scanf("%d", &userInput);
} while (width = 0);
width = userInput; //Same as width
return width;
}

int carCalculate(int cost) //cost
{
int width = userInput;
int cost = (width * length) * 20;
return cost;
}

float cardiscount(float discount) //discount
{
float discount = cost - (cost * .1);
return discount;
}

float displaydisc(float discount) //discount
{
printf("\nThe total cost is: %f\n", discount);
return discount;
}

int main()
{
int length = 0;
int width = 0;
int cost = 0;
int discount = 0;
do {
    length = carLength;
} while (length == 0);
    do {
        width = carWidth;
    } while (carWidth == 0);
    do {
        cost = carCalculate;
    } while (cost == 0);
    do {
        discount = cardiscount;
    } while (discount == 0);
    do {
        displaydisc;
    } while (discount > 0);
    printf("\n Thank you for using this program!");

system("pause");
}

How does it not work?

klutt
– klutt

2018-12-10 04:08:12 +00:00
Commented Dec 10, 2018 at 4:08 — klutt
– klutt, Commented Dec 10, 2018 at 4:08
Whoops, linked the wrong code! Let me edit that.

Daniel Griffith
– Daniel Griffith

2018-12-10 04:08:52 +00:00
Commented Dec 10, 2018 at 4:08 — Daniel Griffith
– Daniel Griffith, Commented Dec 10, 2018 at 4:08

Julius · Accepted Answer · 2018-12-10 04:17:17Z

There are a few problems,

parameters given to most car functions are useless
needed parameters are not provided (carCalculate)
the loop on = 0 having = (assignment instead of equality test)
the loop on a condition where the variable is not changed
calling functions in main() without the (param)

This code should work:

int carLength() // <== no need to provide length, it is returned
{
    int length;
    do
    {
        printf("\nPlease input the length of the area being carpeted:");
        scanf("%d", &length);
    } while (length == 0); // <== length is changed within the loop
    return length;
}

int carWidth() //Width
{
    int width = 0;
    do
    {
        printf("\nPlease input the width of the area being carpeted:");
        scanf("%d", &width);
    } while (width == 0);
    return width;
}

int carCalculate(int width, int length) // <== need to provide values
{
    int cost = (width * length) * 20;
    return cost;
}

float cardiscount(float cost) //discount
{
    float discount = cost - (cost * .1);
    return discount;
}

void displaydisc(float discount) //discount
{
    printf("\nThe total cost is: %f\n", discount);
}

int main()
{
    int length; // <== no need to set to 0, as their values
    int width;  //     are going to be initialized later
    int cost;
    int discount;

    length = carLength();

    width = carWidth();

    cost = carCalculate(width, length);

    discount = cardiscount((float)cost);

    displaydisc(discount);

    printf("\n Thank you for using this program!");

    system("pause");
}

You could also have asked the input functions to fill a value which pointer is given as argument, for instance

int carLength(int *length) {
    do {
        printf("\nPlease input the length of the area being carpeted:");
        scanf("%d", length);
    } while (*length == 0);
}

int called in main

carLength( &length );

BoldAsLove · Accepted Answer · 2018-12-10 04:12:03Z

1

Your not making any function calls. Function calls need to have open/close parentheses after the function name.

For example this is not a function call:

functionA;

... and this is a function call:

functionA();

answered Dec 10, 2018 at 4:12

BoldAsLove

7094 silver badges14 bronze badges

Collectives™ on Stack Overflow

Passing different values between multiple functions in C?

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related