Printable View
Hey Guys,,
I did this coding .. but when ever i add data it doesn't save the data, and when i add second time the data it does.
Even when i try to delete any thing the file gets corrupted.
Sorry i cant copy the whole code here. It a big file.
So i have attached my CPP file.. PLease help me .. :(
Thanks in Advance,
Anshu Verma
Your use of C-style strings leaves you open for buffer overruns.
For other file manipulations it would probably be simpler to load everything into memory, delete from the list in memory and write everything back when the program finishes.
No-nos:
gets - Use fgets instead... or better yet, stop mixing C/C++ I/O and just use getline in place of the gets calls.
You are recursively calling main in your opt functions. Don't do that.
What compiler are you using... ancient/deprecated/non-standard headers (and functions) abound?
Your class design doesn't feel right to me. A class's member functions should not themselves have variables of that class present. I would take advantage of polymorphism to reduce duplicated code and have a templated class design, call it "crime" or something. This crime class can have member functions to "read" the relevant file on program startup and stores the data into a vector/list container that it manages. The destructor's can implement a "write" to the file in question. This will keep the data all in memory and not rely on repeated reading/writing of date to/from the files. Each crime class holds information for a particular crime and holds data for those crimes "murder", "theft" in the above mentioned vector/list and can support the add/delete/modify/search functionality as well. This separates the add/search/delete/modify operations from the individual murder/theft class's themselves which should really only store data for those crimes (along with some basic functionality).
As for other issues, mixing gets (or even getline) with cin can result in the user typing in something and having it processed by the cin which leaves the newline in the buffer. This trailing newline can then be picked up and processed by the next gets/getline call which results in an input operation being "skipped". Every such operation needs to make sure the input stream is cleared prior to any gets/getline call (and don't think about using fflush(stdin), the faq talks about alternatives).
You should be using std::string containers instead of char arrays.