Hello guys

[Cdd101]

I'm having a slight problem with my program. It keeps showing two answers and it's only supposed to show one.

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


int main()
{
    float math(float x, float y);
    float math2(float x, float y);
    float x;
    float y;
    printf("Please enter a x value: ");
    scanf("%d", &x);
    printf("Please enter a y value: ");
    scanf("%d", &y);
    if(0 <= x <10)
    {
    printf("The result is %f.\n", math(x,y));
    }
    else if(x >=10)
    {
    printf("The result is %f.\n", math2(x,y));
    }
    if( y ==0)
    {
    printf("Error: Division by 0!");
    }
    return(0);
}


float math(float x, float y)
{
    return  (x+1)/y;
}




float math2(float x, float y)
{
    return (x-9)/y;
}

This is what I get on command prompt after i enter values


The 1.5 is correct and the answer is correct. I just can't seem to stop the first one from appearing

These are the instructions

[laserlight]

This expression likely does not do what you intend: (0 <= x <10). Rather, you should break it up and use && to connect the two parts.

[vart]

%d in scanf is for integer variables not floats, read formats definitions for scanf

division by zero should be checked before you are actually trying to divide. Otherwise - it has no point, your program has already crashed.

Generally - function declarations should be put before main.

And you should fix your indentation

[Cdd101]

Thank you guys so much for your help. I will start working on it once I'm home!

[Cdd101]

Ok guys, I got this to run and it now knows which function to use. The problem is that it doesn't show the complete correct answer and im sure it's because of the %d being used.

For example if the answer is 1.5 it just shows 1 or if it's 0.5 it just shows 0. If i change everything to float though it works.

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


int main()
{
    int x;
    int y;
    int math(int x, int y);
    int math2(int x, int y);
    printf("Please enter a x value: ");
    scanf("%d", &x);
    printf("Please enter a y value: ");
    scanf("%d", &y);
    if( y ==0)
    {
        printf("Error: Division by 0!");
    }
    if(x>=0 && x<10)
    {
        printf("The result is %d.\n", math(x,y));
    }
    else if(x >= 10)
    {
        printf("The result is %d.\n", math2(x,y));
    }
    return(0);
}


int math(int x, int y)
{
    return(x+1)/y;
}




int math2(int x, int y)
{
    return(x-9)/y;
}

[vart]

line 19 should be "else if"
otherwise you still try to do the math when y is zero

[Cdd101]

line 19 should be "else if"
otherwise you still try to do the math when y is zero

Thank you pal.

#include <stdio.h>#include <math.h>
 
 
int main()
{
    int x;
    int y;
    int math(int x, int y);
    int math2(int x, int y);
    printf("Please enter a x value: ");
    scanf("%d", &x);
    printf("Please enter a y value: ");
    scanf("%d", &y);
    if( y ==0)
    {
        printf("Error: Division by 0!");
    }
    else if(x>=0 && x<10)
    {
        printf("The result is %i.\n", math(x,y));
    }
    else if(x >= 10)
    {
        printf("The result is %i.\n", math2(x,y));
    }
    return(0);
}
 
 
int math(int x, int y)
{
    return(x+1)/y;
}
 
 
 
 
int math2(int x, int y)
{
    return(x-9)/y;
}

I'm still having problems with the answer portion though

[vart]

I'm still having problems with the answer portion though

You said you have correct solution when all vars are floats?

[Cdd101]

Yes pal. If i do 12 and 2 it uses the second formula.

(12-9)/2=1.5

By using int it just shows 1.
If I use float it shows 1.500000.

[laserlight]

That is to be expected due to integer division.

[Cdd101]

That is to be expected due to integer division.

Expected this to be the reason why but my teacher wants %d to be used for scanf.

Would using %i on result be better?

[vart]

you can use %d in scanf and store the user input in integer variables - but make calculations using floats

[laserlight]

Expected this to be the reason why but my teacher wants %d to be used for scanf.

Probably because your teacher expected you to read into integers. You can get floating point division instead of integer division by casting one of the operands to a floating point type.

Would using %i on result be better?

No, you're still reading into an integer.

[Cdd101]

That is to be expected due to integer division.

Probably because your teacher expected you to read into integers. You can get floating point division instead of integer division by casting one of the operands to a floating point type.


No, you're still reading into an integer.

Gah, I really don't know how to do this I'm like right at the finish like but can't finish. Don't want to change everything to float and then get a zero.

Should I just include a remainder solution? By using modulus?

Or could you guys lead me in the right direction with this? I've included the instructions on the OP.

[Cdd101]

Well guys I just ended up turning it in. Thanks for the help, you guys helped me make a few more changes.

Just one more question, can you multiply by char? The answer is no more than likely. Just asking because the following assignment is asking me to multiply when a digit is used

int main(){
    char a;
         printf("Enter a character: ");
         scanf("%c",&a);
    if(a>='a'&& a<='z')
         printf("%c is a lowercased alphabet letter.",a);
    else if (a>='A' && a<='Z')
         printf("%c is an uppercased alphabet letter.",a);
    else if ( a >= '0' && a<= '9')
         printf("%c is a digit.",a);
    else
        printf("It's a special character.");
    return 0;
}

I have got it to here. Now just need to make it multiply.