View Single Post
Join Date: Aug 2005
Posts: 16,618
Reputation: Ancient Dragon has a reputation beyond repute Ancient Dragon has a reputation beyond repute Ancient Dragon has a reputation beyond repute Ancient Dragon has a reputation beyond repute Ancient Dragon has a reputation beyond repute Ancient Dragon has a reputation beyond repute Ancient Dragon has a reputation beyond repute Ancient Dragon has a reputation beyond repute Ancient Dragon has a reputation beyond repute Ancient Dragon has a reputation beyond repute Ancient Dragon has a reputation beyond repute 
Solved Threads: 1616
Team Colleague
Featured Poster
Ancient Dragon's Avatar
Ancient Dragon Ancient Dragon is offline Offline
Still Learning

Re: Airline Reservation Portal

 
0
  #2
Oct 18th, 2008
1) gets() -- never use it. Read this for explaination of why.

lines 26-27: There is no need for variable x. Just use the parameter variable n.

line 41: >>void putdata(int n); Delete that line because its not necessary to prototype a function before actually writing the code for it. Most programmers will have all function progotypes at the very beginnning of the program, after the #include <blabla> statements.

line 44: same as previous comment about 26-27.

No comments about the rest of your program because I don't use ancient Turbo C++.
Last edited by Ancient Dragon; Oct 18th, 2008 at 9:21 am.
The most important thing in the Olympic Games is not to win but to take part, just as the most important thing in life is not the triumph but the struggle. The essential thing is not to have conquered but to have fought well.
-Pierre de Coubertin, The Olympic Creed Inspired by Bishop Ethelbert
Reply With Quote