10 Common Programming Mistakes in C++

1: Undeclared Variables:

int main()
{
cin>>x;
cout<<x;
}

"Huh? Why do I get an error?"

Your compiler doesn't know what x means. You need to declare it as a variable. int main()
{
int x;
cin>>x;
cout<<x;
}

 

2: Uninitialized variables:

int count;
while(count<100)
{
cout<<count;
}

"Why doesn't my program enter the while loop?"

In C++ variables are not initialized to zero. In the above snippet of code, count could be any value in the range of int. It might, for example, be 586, and in that situation the while loop's condition would never be true. Perhaps the output of the program would be to print the numbers from -1000 to 99. In that case, once again, the variable was assigned a memory location with garbage data that happened to evaluate to -1000.

Remember to initialize your variables.

 

3: Setting a variable to an uninitialized value:

int a, b;
int sum=a+b;
cout<<"Enter two numbers to add: ";
cin>>b;
cout<<"The sum is: "<<sum;
When Run:
Enter two numbers to add: 1 3
The sum is: -1393

"What's wrong with my program?"

Often beginning programmers believe that variables work like equations - if you assign a variable to equal the result of an operation on several other variables that whenever those variables change (a and b in this example), the value of the variable will change. In C++ assignment does not work this way: it's a one shot deal. Once you assign a value to a variable, it's that value until you reassign the values. In the example program, because a and b are not initialized, sum will equal an unknown random number, no matter what the user inputs.

To fix this error, move the addition step after the input line. int a, b;
int sum;
cout<<"Enter two numbers to add: ";
cin>>b;
sum=a+b;
cout<<"The sum is: "<<sum;

 

4: Using a single equal sign to check equality:

char x='Y';
while(x='Y')
{
//...
cout<<"Continue? (Y/N)";
cin>>x;
}

"Why doesn't my loop ever end?"

If you use a single equal sign to check equality, your program will instead assign the value on the right side of the expression to the variable on the left hand side, and the result of this statement is TRUE. Therefore, the loop will never end. Use == to check for equality; furthermore, to avoid accidental assignment, put variables on the left hand side of the expression and you'll get a compiler error if you accidentally use a single equal sign as you can't assign a value to something that isn't a variable. char x='Y';
while('Y'==x)
{
//...
cout<<"Continue? (Y/N)";
cin>>x;
}

 

5: Undeclared Functions:

int main()
{
menu();
}
void menu()
{
//...
}

"Why do I get an error about menu being unknown?"

The compiler doesn't know what menu() stands for until you've told it, and if you wait until after using it to tell it that there's a function named menu, it will get confused. Always remember to put either a prototype for the function or the entire definition of the function above the first time you use the function. void menu();
int main()
{
menu();
}
void menu()
{
...
}

 

6: Extra Semicolons:

for(int x=0; x<100; x++);
cout<<x;

"Why does it output 100?"

You put in an extra semicolon. Remember, semicolons don't go after if statements, loops, or function definitions. If you put one in any of those places, your program will function improperly. for(int x=0; x<100; x++)
cout<<x;

 

7: Overstepping array boundaries:

int array[10];
//...
for(int x=1; x<=10; x++)
cout<<array[x];

"Why doesn't it output the correct values?"

Arrays begin indexing at 0; they end indexing at length-1. For example, if you have a ten element array, the first element is at position zero and the last element is at position 9. int array[10];
//...
for(int x=0; x<10; x++)
cout<<array[x];

 

8: Integer division

For example:
double half = 1/2;

This code sets half to 0 not 0.5! Why? Because 1 and 2 are integer constants.

To fix this, change at least one of them to a real constant.

double half = 1.0/2;

If both operands are integer variables and real division is desired, cast one of the variables to double (or float).

int x = 5, y = 2;
double d = ((double) x)/y;

9: Variable Name Styles

Take a look at the below program. Can you see anything wrong?

#include <stdio.h>
#include <string.h>

main()
{
char CatName[20] = "fluffy";
char dogName[20] = "fido";
char rat_Name[20] = "fester";
int Catage = 3;
int dogs_age = 4;
int ratage = 1;
char myPet[20];
int itsAge;

strcpy(myPet,rat_Name);
itsAge = ratage;
printf("My pet is %s\n",myPet);
}

Besides the obviously bizarre choice of having a pet rat, do you notice anything? Take a close look at the variable names. There is no consistent style. Is the variable that stores good old Fido's name dogName, DogName, dog_name, dogname or Dogname? A programmer writing this code would need to constantly check to get the variable name correct. Do yourself a favor and simplify you life. Choose a consistent style for variable names and stick to it. The exact style doesn't matter but consistency does. You'll be amazed at how much faster you can develop code and how many fewer compiler errors you will get if you adopt a consistent style.

Common Styles

1)First letter of each word capitalized: CatName
2)Underscores: cat_name
3)First letter lower case, first letter of every other word capitalized: catName
4)No capital letters, no underscore: catname

 

กก

10 Misusing the && and || operators:

int value;
do
{
//...
value=10;
}while(!(value==10) || !(value==20))

"Huh? Even though value is 10 the program loops. Why?"

Consider the only time the while loop condition could be false: both value==10 and value==20 would have to be true so that the negation of each would be false in order to make the || operation return false. In fact, the statement given above is a tautology; it is always true that value is not equal to 10 or not equal to 20 as it can't be both values at once. Yet, if the intention is for the program only to loop if value has neither the value of ten nor the value of 20, is is necessary to use && : !(value==10) && !(value==20), which reads much more nicely: "if value is not equal to 10 and value is not equal to 20", which means if value is some number other than ten or twenty (and therein is the mistake the programmer makes - he reads that it is when it is "this" or "that", when he forgets that the "other than" applies to the entire statement "ten or twenty" and not to the two terms - "ten", "twenty" - individually). A quick bit of boolean algebra will help you immensely: !(A || B) is the equivalent of !A && !B (Try it and see). The sentence "value is other than [ten or twenty]" (brackets added to show grouping) is translatable to !(value==10 || value==20), and when you distribute the !, it becomes !(value==10) && !(value==20).

The proper way to rewrite the program: int value;
do
{
//...
value=10;
}while(!(value==10) && !(value==20))