array of struct

Question

0.00/5 (No votes)

See more:

hi there

i'm new to c++ and i hv been trying to make a little program
about adding vehicle details(ID,number,category, brand,price etc.) to a menu and displaying entered details,searching for a particular vehicle based on either vehicle number or category and deleting a record of sold vehicle.

following is the answer i could come up with

#include <iostream>
#include <string>

using namespace std;

struct vehicles

{

char vehicalid[8];
char vehicalnumber[8];
char category[10];
char brand[20];
int price;


} data[100];

int main ()
{
int d=0;
int i=0;
int op=0;
int n=0;
int b=0;
char num[8];


cout<<"option (1) add vehicle details \n";
cout<<"option (2) view view details \n";
cout<<"option (3) search for a vehicle \n";
cout<<"option (4) delete vehicle details \n"<<endl;

cout<<"option 01 \n"<<endl;

cout<<"enter your Option \n";
cin>>op;
loop:

while (op==1){

cout<<"enter vehicle ID \t";
cin>>data[n].vehicalid;
cout<<"enter vehicle number \t";
cin>>data[n].vehicalnumber ;
cout<<"enter vehicle category \t";
cin>>data[n].category;
cout<<"enter brand of the vehicle \t";
cin>>data[n].brand;
cout<<"enter price of the vehicle \t";
cin>>data[n].price;

n++;

cout<<"Enter Your Option to continue \t";
cin>>op;

if(op!=1)goto loop;

}

while (op==2){
cout<<"Option (2) \n";
while(b<n){
cout<<data.vehicalid<<'\t'<<data.vehicalnumber<<'\t'<<'\t'<<data.category<<'\t'<<data.brand <<'\t'<<data.price<<endl;
b++;
}
cout<<"Enter your option to continue \t";
cin>>op;
if(op!=2)goto loop;

}
while (op==3){
cout<<"Option (3) \n";

cout<<"enter vehicle number \n";
cin>>num;

while (i<n){
i++;
if(strcmp(data.vehicalnumber, num)==0) break;
}


cout<<"search resutl !! \n";
cout<<"vehicle id\t"<<"Number\t"<<"Category\t"<<"Brand\t"<<"price\n";
cout<<data.vehicalid<<'\t'<<data.vehicalnumber<<'\t'<<data.category<<'\t'<<data.brand<<'\t'<<data.price<<'\n';
cout<<"Enter your option to continue \t";
cin>>op;
if(op!=3)goto loop;

}



return 0;
}</pre>


i dont hv any idea how to delete details of a particular vehicle

and when first add vehicle details and go on to the second option to show detail it works fine but again if i try to add details of a vehicle and to see the details of all vehicle entered so far ,only latest details entered is shown

if someone can help me on this that would be really helpful

Posted 8-Feb-10 19:16pm

agskanchana

Add a Solution

7 solutions

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)

Christian Graus · Answer 1 · 2010-02-08T19:47:00

Solution 3

Navaneeth - he's almost certainly doing homework and is expected to learn how arrays work, not to move on to dynamic containers.

Posted 8-Feb-10 19:47pm

Christian Graus

Christian Graus · Answer 2 · 2010-02-08T19:30:00

agskanchana wrote:
while (op==1){

agskanchana wrote:
if(op!=2)goto loop;

This is really messy. You should just have one input for op, a switch statement, and no goto statements. I'd break the seperate things you do off in to methods.

If you structure the code a little better, you might find it easier to debug.

agskanchana wrote:
data[100];

I did not know that this worked, but, your obvious issue is that you've used the memory for 100 items, if you have 1 or 100, and you cannot have 101. That's fine for a learning exercise, but eventually, you will not use this sort of code, you will use a dynamic container. In the meantime, the way you can delete an item, is to copy all your items up and decrement your counter. So if you have 75 items, n will equal 75. To delete item number 50, you need to copy items 51-75 down to their previous index, if order matters, or copy item 75 over item 50, if the order does not matter, then decrement your counter, erasing the last item. As your list is sequential, you delete an item by moving it to the end, and then moving the end, as the end is all you can move.

William Winner · Answer 3 · 2010-02-16T10:30:00

Wow...you all can't seem to tell the level of the people you are talking to. It's obvious that you're in a basic C++ class and are doing homework regarding arrays. If not, well, then you should find that C++ class. And I take it you haven't gotten to For loops yet.

I'm going to try to rephrase some of the code...tell me if I'm wrong.

Your vehicle information is stored in the variable data which has 100 vehicles possible. n stores the current number of vehicles that have been input. op is the user input. b is used as a counter when displaying the vehicle information. num stores the input when a user is searching for a vehicle. I'm assuming d would be for indicating which vehicle to delete. i is another counter you use when searching.

Some basic problems: Before you use a counter, make sure you reset it first.

C#

while (op==2){
cout<<"Option (2) \n";
while(b<n){
cout<<data.vehicalid<<'\t'<<data.vehicalnumber<<'\t'<<'\t'<<data.category<<'\t'<<data.brand <<'\t'<<data.price<<endl;
b++;
}

This will only work the first time, because b never gets reset. Same with your search.

Problem 2: With arrays, you need to make sure to identify which item in the array you're dealing with.

C#

if(strcmp(data.vehicalnumber, num)==0) break;

should be

C#

if(strcmp(data[i].vehicalnumber, num)==0) break;

and

C#

cout<<data.vehicalid<<'\t'<<data.vehicalnumber<<'\t'<<data.category

should be

C#

cout<<data[b].vehicalid<<'\t'<<data[b].vehicalnumber<<'\t'<<data[b].category

and same with the first excerpt I pulled out.

To delete an item, you need to copy all of the items beyond it down one. So, let's say you found the vehicle ID that matches the one you want to delete. And, let's say you've set d to that index. Now:

C#

while (d<n) {
    data[d].vehicalid = data[d+1].vehicalid; 'by the way, you spelled vehicle wrong
    'copy the other values

    d++;
}

Then, make sure to decrement n (n--;)

Seriously people...you need to understand your audience. This person may not have dealt with separate functions yet, switch statements, or for/next loops.

N a v a n e e t h · Answer 4 · 2010-02-08T19:44:00

Solution 2

agskanchana wrote:
data[100];

Use containers like std::vector. It has got methods for inserting and removing items. It can expand automatically when you add new items. Also work with the std::string and avoid using character arrays.

Posted 8-Feb-10 19:44pm

N a v a n e e t h

she-3 · Answer 5 · 2010-02-09T22:29:00

Solution 4

C#

cin>>op;
if(op!=3)goto loop;
}

return 0;

to changge
cin>>op;
else if {
cout<<"option (4) delete vehicle details \n"<<endl;}
return 0;
it is my opinions :rolleyes:

Posted 9-Feb-10 22:29pm

she-3

TRPatil · Answer 6 · 2010-02-15T20:10:00

i totally agree with cristian.. first u should make you code proper..

one method to delete the element is you can simply overwrite it.
ex:
(i am not using the names that u r using for ur code you can just have
a reference from this ex)

enter the element to be deleted
cin>> elemid;

short storecount;
for(short i=0; i<maxsize_of_struct; i++)
{
database[i].vehicleid == elemid;
storecount = i;
}

//nw write another loop to overwrite
for(short i=storecount 0; i<maxsize_of_struct-1; i++)
{
database[i] == database[i+1]elemid;
}
database[i]= 0;//its a last element

hope it will solve your problem

zhang_yang · Answer 7 · 2010-02-16T14:55:00

Solution 7

Why not try to use the ony way linked list in the list you just need to insert or delete one node can achieve your request, you may need to see when the cycle of get what you want :laugh:

Posted 16-Feb-10 14:55pm

zhang_yang