function to insert data in structure at certain index and other function to display that data

Question

0.00/5 (No votes)

See more:

C

My program aims to

Quote:
function to insert data in structure at certain index and other function to display that data

ERROR

Quote:
incompatible type conversion error c

C++

#include<stdio.h>
struct student{
  char name[50];
  int id;
}; 

struct student printstudent()
 {
   int index=0;
   struct student e;
   clrscr();
	printf("plz insert index \n");
	scanf("%d",&index);
         if(index >2) printf("index out of range ");return 0;
	printf("plz insert name\n");
	  scanf("%s",e[index].name);
	  printf("plz insert age\n");
	  scanf("%d",&e[index].id);
   return e; //error at this line

 }
  main()
   {

	  printstudent() ;
	  clrscr();
	 getch();
	 return 0;
   }

Posted 18-Oct-14 3:31am

TheSniper105

Updated 18-Oct-14 4:08am

v3

Add a Solution

Comments

George Jonsson 18-Oct-14 9:43am

Why do you define an array of students, you only use the first position?
(struct student e[3])
Your return value is struct student and you return an array of students, so of course it fails.

TheSniper105 18-Oct-14 9:45am

so how to insert data at index 1 ,index 2,index 3 without array ?

[no name] 18-Oct-14 9:56am

You don't. Your array only has 3 elements allotted to it so if you use e[3] you will get another failure. What does this completely different question have to do with your original question?

TheSniper105 18-Oct-14 10:00am

ok can you post your solution to achieve what i want ?

[no name] 18-Oct-14 10:05am

Write your code for you? Uhm.... no. What you say you want and what your code does are completely different things. So even if I had the time or inclination to write your code for you I could not because what you want is not clear. Why are you trying to return an array from your function when you function signature says that you are going to return a single struct?

TheSniper105 18-Oct-14 10:09am

sorry if you misunderstand me what i asked is just help find solution not writing the code please see again i have updated it

[no name] 18-Oct-14 10:29am

Okay... see what? You have added a check for an out of range condition, good. But a range for what? Your array is not an array anymore so adding a check for the upper bound of your array is useless. Did you compile this code? I bet you did not. the return 0 would not compile as you cannot return an integer from a function that expects to return a struct. And your e[index] lines would also not compile. Even if you get your code to compile, you are returning something from a function and not doing anything with it. You need to go back a study up on arrays and returning values from functions.

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Accepted Answer · 2014-10-18T04:13:00

You have defined the function as returning a student:

C#

struct student printstudent()
   {

But you try to return an array of students:

C#

struct student e[3];
...
return e;

You can't do that - they need to be exactly the same type.
Probably, you don't want to declare an array, just a single item, but since you don;t actually use it in the main function, you might be better changing it so that the printstudent function accepts a student to print and returns nothing (a void).

Mind you, I'd make printstudent print something instead of creating an instance myself anyway... :laugh: