why doesn't this work?

[Toxic90]

I'm building a banking program for my lab class, i have a problem with part of the code, i made a program out of that code to study it especially, but i cant find the problem... here's the code:

#include<iostream>
using namespace std;
int main()
{
    int QC=0,CN,CLI[100];
    int X=1;
    while(X!=0){
    cout<<"Cual sera el numero de cliente? ";
    cin>>CN;
    if(CN<10000||CN>99999) {cout<<"Numero invalido \n\n";}
    else for(int i=0;i<100;i++){if(CN==CLI[i]){cout<<"Cliente ya existe\n\n";
                                               break;}
                                else {CLI[QC]=CN;
                                      QC++;
                                      cout<<"Cliente creado exitosamente \n\n";
                                      break;}
                                }
    cout<<"Otro? ";
    cin>>X;
    }
    cout<<"Q= "<<QC<<endl;
    for(int i=0;i<=(QC-1);i++){cout<<CLI[i]<<endl;}
    system("pause");
    return 0;
}

it's supposed to not let you create the client if it already exists, but it does anyway, i've narrowed the error to be around here:

else for(int i=0;i<100;i++){if(CN==CLI[i]){cout<<"Cliente ya existe\n\n";
                                               break;}
                                else {CLI[QC]=CN;
                                      QC++;
                                      cout<<"Cliente creado exitosamente \n\n";
                                      break;}
                                }

but i cant see it, can you help me?

thanks!!

[Toxic90]

never mind, got it :P sorry, thanks

[jloundy]

your have no brackets around your else:

else
{
for(int i=0;i<100;i++)
   if(CN==CLI[i])
   {
   cout<<"Cliente ya existe\n\n";
   break; // whats the point of these
   }
      else
      {
      CLI[QC]=CN;
      QC++;
      cout<<"Cliente creado exitosamente \n\n";                         
      break; // ??
      }
                                
}

try not to have your code formatting like that anymore,
its almost impossible to read