Undeclared Identifer

Hi guys,

I keep trying to compile the code of a small program that I have been creating just as a learning exerciser again but keep coming up against an error - "error C2065: 'gets' : undeclared identifier" but I can't see where I'm going wrong in the code for that to happen - anyone any ideas ? My code is:

Code:

---

void add_video(videorecord_type videos[])
{
        int no_of_videos, next_record_no, year;
        float price;
        char tempstring[STOCK_ID_SIZE+1];

        //Display heading
        clear_screen();
        cout << "                                                ADD A VIDEO TO STOCK"
                << endl << endl << endl;

        //Count how many videos are already in the database
        no_of_videos = video_count(videos);

        //Report to user
        cout << endl << "There are " << no_of_videos << " videos in VIDEO LIBRARY. " << endl;

        //Set the record number at which to add the new video
        next_record_no = no_of_videos;

        //If database is not full, add a video
        if (no_of_videos < MAX_VIDEOS)
        {
                //Get the stock ID as a temp string
                cout << endl << "Stock ID: " << flush;
                cin  >> tempstring;

                //Check this stock_ID doesn't already exist
                if (record_no(videos,tempstring) == -1)
                {
                        strcpy(videos[next_record_no].stock_ID, tempstring);
                        cout << "Video Title: " << flush;
                        gets(videos[next_record_no].title);

                        do
                        {
                                cout << "Year of release: ";
                                cin >> year;
                                if (year < 1950 || year > 2050)
                                {
                                        cout << "Year muct be in range 1950 to 2050\a" << endl;
                                }
                        }while (year < 1950 || year > 2050);
                        videos[next_record_no].year = year;

                        do
                        {
                                cout << "Price: ";
                                cin >> price;
                                if (price < 1.0 || price > 20.0)
                                {
                                        cout << "Price must be in the range £1 to £20\a" << endl;
                                }
                        }while (price < 1.0 || price > 20.0);
                        videos[next_record_no].price = price;
                }
                else
                {
                        cout << "This stock ID already exists\a" << endl;
                }
        }
        //If database full, give message
        else
        {
                cout << "No room for any more videos\a" << endl;
        }

        //Database has changed so save it
        save_data(videos);
}

---

Thanks guys for any input

H_M

Dont use gets. The answer is probably that you didnt include the appropriate header, but you might as well use std::getline() or std::cin.getline(), as the case may be.

>"error C2065: 'gets' : undeclared identifier"
We wish. Did you include stdio.h? Do you know that gets should never be used? Do you know that this is the C++ forum and cin.getline would be a better choice?

Thanks guys. Yeah, I had forgotten to include the 'stdio.h' header ! Always the simplest things. :p

Sorry for using 'gets'. I've not been learning C++ long and the book I was/am using has 'gets' and not 'cin.getline'. Does this work in just the same way with the same header ?? Also, is there any on-line references that you would recommend to look at that ??

Thanks again

H_M

Like laserlight's sig suggests, you should just look up a reference.
C++ I/O
cin.getline

>book I was/am using has 'gets' and not 'cin.getline'
It sounds like you need to burn that book and get a good one. Any C++ book that uses gets is begging to be torched.

Quote:

---

Originally Posted by Prelude

It sounds like you need to burn that book

---

Back then, people who used to read these type of books used to burn on the stake too. After prolonged, exquisite and agonizing torture.

Cheers guys for the info on 'getline' - I'm trying to look into it but it's quite hard for me ! How would i go about using that in the code I pasted in my earlier post instead of 'gets' ??

If I get used to using it now, from now on I can use that instead. :)

On a seperate note, do any of you recommend any good books on learning C++ for a beginner upwards ??

Thanks again

H_M

Make sure you include cstring.
Something like
cin.getline( videos[next_record_no].title, strlen(videos[next_record_no].title) );
depending on what title is. If it's a c-string strlen will work, if it's a std::string, use the length method.

There's a sticky for reccommended books at the top of this forum. Give that a read.