Thread: Help with project (New to C)

I am having trouble with a lab in a beginning C class. I have completed most of it, but I cannot get the outputToScreen function to do what it's supposed to (which is show the values on the screen). Can anyone help me with this. Here's the code so far, and I'm only concerned with getting the outputToScreen function to show the correct values on screen.

Thanks in advance.

Code:

#include<stdio.h>
int getUserInput();
int calculateArea(int, int);
int calculatePerimeter(int, int);
int outputToScreen (int, int, int, int);


#pragma warning(disable:4996) 

void main()  
{
	int area = 0;
	int perimeter = 0;
	int length = 0;
	int width = 0;
	printf("\nPlease enter the length of the rectangle as a whole number: ");
	length = getUserInput();
	printf("\nPlease enter the width of the rectangle as a whole number: ");
	width = getUserInput();
    area = calculateArea(length, width);
    perimeter = calculatePerimeter (length, width);
}

int getUserInput()
{int input = 0;
scanf("%d%*c",&input);

return input;
}


int calculateArea(int len, int wid)
{
int area = 0;
area = len * wid;
return area;
}

int calculatePerimeter (int len, int wid)
{int perimeter = 0;
perimeter = len * 2 + wid * 2;
return perimeter;
}

int outputToScreen (int length, int width, int area, int perimeter)
{
printf ("\nWhen length is %d", length);
printf ("and width is %d,", width);
printf (" area is %d", area);
printf ("and perimeter is %d.", perimeter);
return main;
}

You never called outputToScreen() in main.

Also, this makes no sense:

return main;

Anyone knows what this means

scanf("%d%*c",&input);

That scanf is exactly how the professor wanted us to keep it. We aren't allowed to change that.

So instead of return main should it be return 0, or what should it return?

Also, void main is how he wanted us to keep it.

What should the outputToScreen function look like for it to work correctly?

How do I set up outputToScreen in main to call it correctly?

Sorry for the silly questions, but this is my first day.

Thank you for showing me how to call it. Now the only problem I have is some crazy math. When I input the numbers for length and width, I get insane numbers. Is there a problem with my outputToScreen function, or it is in the calculate functions. Here's the updated code:

Code:

#include<stdio.h>
int getUserInput();
int calculateArea(int, int);
int calculatePerimeter(int, int);
int outputToScreen ();


#pragma warning(disable:4996) 

void main()  
{
	int area = 0;
	int perimeter = 0;
	int length = 0;
	int width = 0;
	printf("\nPlease enter the length of the rectangle as a whole number: ");
	length = getUserInput();
	printf("\nPlease enter the width of the rectangle as a whole number: ");
	width = getUserInput();
    area = calculateArea(length, width);
    perimeter = calculatePerimeter (length, width);
	outputToScreen();
	return 0;
}

int getUserInput()
{
int input = 0;
scanf("&#37;d%*c",&input);

return input;
}


int calculateArea(int len, int wid)
{
int area = 0;
area = len * wid;
return area;
}

int calculatePerimeter (int len, int wid)
{
int perimeter = 0;
perimeter = (len * 2) + (wid * 2);
return perimeter;
}

int outputToScreen (int length, int width, int area, int perimeter)
{
printf ("\nWhen length is %d", length);
printf (" and width is %d,", width);
printf (" area is %d", area);
printf (" and perimeter is %d.", perimeter);
return 0;
}

Thanks for your help so far.

I fixed the code on my own. Here's what the final code looks like.

Code:

#include<stdio.h>
int getUserInput();
int calculateArea(int, int);
int calculatePerimeter(int, int);
int outputToScreen (int, int, int, int);


#pragma warning(disable:4996) 

int main(void)  
{
	int area = 0;
	int perimeter = 0;
	int length = 0;
	int width = 0;
	printf("\nPlease enter the length of the rectangle as a whole number: ");
	length = getUserInput();
	printf("\nPlease enter the width of the rectangle as a whole number: ");
	width = getUserInput();
    area = calculateArea(length, width);
    perimeter = calculatePerimeter (length, width);
	outputToScreen(length, width, area, perimeter);
	return 0;
}

int getUserInput()
{
int input = 0;
scanf("%d%*c",&input);

return input;
}


int calculateArea(int len, int wid)
{
int area = 0;
area = len * wid;
return area;
}

int calculatePerimeter (int len, int wid)
{
int perimeter = 0;
perimeter = (len * 2) + (wid * 2);
return perimeter;
}

int outputToScreen (int length, int width, int area, int perimeter)
{
printf ("\nWhen length is %d", length);
printf (" and width is %d,", width);
printf (" area is %d", area);
printf (" and perimeter is %d.", perimeter);
return 0;
}

Thanks for everything!

Originally Posted by Elysia

Why should it return anything?


No. It's undefined, non-standard and it's stupid and idiotic. Change it or bug your professor to change it.
Void main is a non-standard extension to many compilers and will not work correctly across compilers. It may even give you a bad result when compiled in your compiler. Therefore, you should NOT use void main, ever.


Who says it isn't working correctly?


You need to learn how to call functions, but you've already called two functions!


It eats the newline after extracting the number.

hai...

Code:

#include<stdio.h>
int getUserInput();
int calculateArea(int, int);
int calculatePerimeter(int, int);
int outputToScreen (int, int, int, int);


#pragma warning(disable:4996) 

void main()  
{
	int area = 0;
	int perimeter = 0;
	int length = 0;
	int width = 0;
	printf("\nPlease enter the length of the rectangle as a whole number: ");
	scanf("%d", &length);
	printf("\nPlease enter the width of the rectangle as a whole number: ");
	scanf("%d", &width);
  area = calculateArea(length, width);
  perimeter = calculatePerimeter (length, width);
  printf ("When length is %d", length);
  printf ("  and width is %d\n,", width);
  printf (" area is %d\n", area);
  printf ("and perimeter is %d.", perimeter);
}

int getUserInput()
{int input = 0;
scanf("%d%*c",&input);

return input;
}


int calculateArea(int len, int wid)
{
int area = 0;
area = len * wid;
return area;
}

int calculatePerimeter (int len, int wid)
{int perimeter = 0;
perimeter = len * 2 + wid * 2;
return perimeter;
}