incompatible type return

[convenientstore]

Hi,

I am testing a code from c tutorial and tried below code and did compile and it fails to incompatible type in return.. and I am not sure why ?

#include <stdio.h>

struct invStruct {
   char manuf[25];
   char model[15];
   int diskSpace;
   int memSpace;
   int quantity;
   float cost;
   float price;
}

main()
{
  int ctr;
  struct invStruct items[3];   /* Array of 3 structure variables */

  for ( ctr = 0; ctr < 3; ctr++ )
  {
      printf( "What is the manufacturer of item #%d?\n", (ctr + 1) );
      gets( items[ctr].manuf );
      puts( "what is the mode? ");
      gets(items[ctr].model);
      puts("how many megabytes of disk space? ");
      scanf(" %d", &items[ctr].diskSpace);
      puts("How many megabytes of memory space? ");
      scanf(" %d", &items[ctr].memSpace);
      puts("How many are there? ");
      scanf(" %d", &items[ctr].quantity);
      puts("How much does the item cost? ");
      scanf(" %d", &items[ctr].quantity);
      puts("How much does the item cost? ");
      scanf(" %f", &items[ctr].cost);
      puts("How much does the item retail for? ");
      scanf(" %f", &items[ctr].price);
      getchar();   /* clears input of last newline pressed for next round of input */
   }    /* Now prints the data */
   printf("\n\nHere is the inventory: \n");
   for ( ctr = 0; ctr < 3; ctr++ )
   {
      printf("#%d: Manufacturer: %s", (ctr + 1), items[ctr].manuf);
      printf("\nModel: %s", items[ctr].model);
      printf("\nDisk: %d megabytes\t", items[ctr].diskSpace);
      printf("Memory: %d megabytes\t", items[ctr].memSpace);
      printf("Quantity: %d units\n", items[ctr].quantity);
      printf("Cost: $%.2f\t", items[ctr].cost);
      printf("Selling price: $%.2f\n\n", items[ctr].price);
    }
    return 0;
}

result of gcc'ing

root@my-laptop:/code_upload/c# gcc -o struct.o struct.c
struct.c: In function âmainâ:
struct.c:51: error: incompatible types in return
struct.c:16: warning: return type of âmainâ is not âintâ
root@my-laptop:/code_upload/c#

[Adak]

int main() would be a start

[anon]

And adding a missing semicolon would be the next step.

[Elysia]

Getting rid of gets would be a 3rd.
SourceForge.net: Gets - cpwiki

[vcrewchief]

I'm getting the same error, and just can't figure it out. Someone said to add a semicolon, and I can't see where he needs it. I'm guessing that's my problem.

Here's the directions if this helps:

Show how to declare a tag named complex for a structure with two members, real and imaginary, of type double.

use the complex tag to declare variables named c1,c2, and c3.

write a function named make_complex that stores its two arguments (both of type double) in a complex structure, then returns the structure.

#include <stdio.h>

double make_complex (double a, double b){
struct {
double real;
double imaginary;
}p;
p.real = a;
p.imaginary = b;
return p;

}


int main() {
struct complex {
double real;
double imaginary;
};
struct complex c1,c2,c3; // (b)
c3 = make_complex(1.0, 2.0);

}

thanks in advance for the help!




denver electrician

[Adak]

First, Welcome to the Forum, VCrewChief!

I don't see a missing semi-colon. I do see an int main() function with no return of the expected int, though.

[vcrewchief]

I'm still getting the same error. i tried the return0; and void main()





electrician in denver

[dylan]

Your main needs to be int main(.....
and you also need the return at the end.

Looks like you used one or the other each time.

Dylan

[jimblumberg]

In your code you are defining structures inside your functions. Move the structure in main to before make_complex. Delete the structure inside make_complex.

Use the structure you moved outside the functions inside make_complex and main.


Jim

[vcrewchief]

You are the man. Here's what I did:

#include <stdio.h>


struct complex {
double real;
double imaginary;
};

struct complex make_complex (double a, double b){
struct complex p;
p.real = a;
p.imaginary = b;
return p;

}

void main() {

struct complex c1,c2,c3; // (b)
c3 = make_complex(1.0, 2.0);
//printf("%.1f, %.1f", c3.real, c3.imaginary);
}

Induction retrofit

[jimblumberg]

Glad I could help.

For future reference when you have a question not directly related to the current thread, create an new thread.

Also main() should be int main() and return 0 before exiting.

It also helps to indent your code sections. It makes it easier to read the code.

#include <stdio.h>


struct complex {
	double real;
	double imaginary;
};

struct complex make_complex (double a, double b) {
	struct complex p;
	p.real = a;
	p.imaginary = b;
	return p;

}

int main() {

	struct complex c1,c2,c3; // (b)
	c3 = make_complex(1.0, 2.0);
       //printf("%.1f, %.1f", c3.real, c3.imaginary);
      return(0);
}

Jim

[vcrewchief]

Awesome, thanks for the advice. it's indented in my compiler, it was just copied funkily.