Printable View
hey guys i'm just new here and i would like some help with my school work. what i'm trying to do is change the values inside the arrays which in this case is a country code. don't mind the country code i just made that up. i would like some help or some improvements in my code tnx. if you could reply asap it would be a great help tnx.
* i'm using borland c++
Please post you code in [code][/code] bbcode tags instead of attaching it.
Anyway, the problems that jump out at me are:
- You have a main function that is more than 230 lines long. Rather, functions should do one thing and do it well. Break up your main functions into smaller functions, say no more than 50 lines each.
- Your code lacks proper indentation.
- You use goto when it appears to be completely unnecessary. It is hard to see the structure of your code because of the lack of indentation, but my guess is that your whole goto scheme is a farce: control would have gone to the label point even without the use of goto.
Quote:
Originally Posted by laserlight
thanks for the feed.
can you show me how to do that "farce" i'm kinda newbie here :D , sorry for the attachment thing i'm also new to these forums
The word "farce" refers to an action that is silly. By asking how to do that, you have committed a farce.
Anyway, it is clear that your english expression isn't the best, so bear with us if there is misunderstanding.
Laserlight's point is to break your code into smaller functions, post in directly with [code] tags, and it is better to avoid using "goto".
it's 2% haha