programa que no tiene errores pero se para

[edynperu]

hola estoy escribiendo un programa para saber si un numero es capicua o no, el codigo esta todo bien pero al ejecutarlo y luego de introducir el numero al programa se para y me aparece que el programa a dejado de funcionar.

gracias por sus respuestas

Código C++:

#include <iostream.h>
#include <conio.h>
 
 
int main ()
{
    //variables
    int num, bnum, numcifra=1, i = 0, suma=0, n_dd[10]; //debido a que n_dd tiene 10 elementos solo se puede procesar numeros hasta de 21 cifras
    //entrada
    cout << "Ingrese el numero (maximo 21 cifras): ";
    cin >> num;
    //====================================================================================
    /*/contar el numero de cifras
    bnum=num;
    for (i=1; bnum/10!=0;i++)
    {
        bnum=bnum/10;
        numcifra=numcifra+1;
    }
    cout << numcifra;*/
    //====================================================================================
    
    //contamos las cifras y sacamos digito por digito a cada cifra y lo ponemos en la variable 
    bnum=num;
    for (i=1; bnum/10!=0;i++)// != desigual que
    {
        numcifra+=1; //aki se cuentan las cifras
        n_dd[i]=bnum%10;
        bnum=bnum/10;
        if (bnum/10==0) // aki se da el valor para la ultima cifra
        {
                       n_dd[i+1]=bnum;
        }
    }
    //igualamos 
    for (i=1; numcifra/2; i++)
    {
        if (n_dd[i]==n_dd[numcifra/2-i])
        {
                                 suma=0 ;      //suma sera cero
        }else{
              suma=suma+1;
              }
    }
    // si suma=0 entonces es capicua
    if (suma==0)
    {
                cout << "\n\nEl numero es capicua" << endl;
    }else{
          cout << "\n\nEl numero es no capicua" << endl;
    }
 
    getche ();
    return 0;
}
 

[CalgaryCorpus]

Veo que hay al menos 2 problemas:

El segundo ciclo for no termina. La condicion de termino esta mala.

El arreglo n_dd tiene espacio para 10 elementos y el primer ciclo no se detiene antes que 10, podrias estar sobreescribiendo zonas de memoria no asignadas para ti.

Esto ultimo es facil de chequear que es efectivamente el problema que tienes: simplemente cambia n_dd[10] por n_dd[1000] exagerando para ver que el problema que ahora tienes se va despues de ese cambio.

Una vez convencido que ese era el problema, modifica tu solución para que n_dd tenga espacio suficiente siempre, haciendo que el ciclo actue de acuerdo a los limites del arreglo.

[edynperu]

encontre el error, como dijo CalgaryCorpus el segundo for no terminaba asi que cambie las condiciones aqui el condiciones

antes:

Código C++:

for (i=1; numcifra/2; i++)
    {
        if (n_dd[i]==n_dd[numcifra/2-i])
        {
                                 suma=0 ;      //suma sera cero
        }else{
              suma=suma+1;
              }
    }
 

ahora

Código C++:

for (i=0; i!=numcifra/2; i++)
    {
        if (n_dd[i]==n_dd[numcifra-1-i])
        {
                                 suma+=0 ;      //suma sera cero
        }else{
              suma+=1;
              }
    }
 

el codigo completo seria

Código C++:

#include <iostream.h>
#include <conio.h>
 
 
int main ()
{
    //variables
    int num, numcifra=1, i = 0, suma=0, n_dd[20]; //debido a que n_dd tiene 21 elementos solo se puede procesar numeros hasta de 21 cifras
    //entrada
    cout << "Ingrese el numero (maximo 21 cifras): ";
    cin >> num;
    //====================================================================================
    /*/contar el numero de cifras
    for (i=1; num/10!=0;i++)
    {
        num=num/10;
        numcifra=numcifra+1;
    }
    cout << numcifra;*/
    //====================================================================================
    
    //contamos las cifras y sacamos digito por digito a cada cifra y lo ponemos en la variable 
    for (i=0; num/10!=0;i++)// != desigual que
    {
        numcifra+=1; //aki se cuentan las cifras
        n_dd[i]=num%10;
        num=num/10;
        if (num/10==0) // aki se da el valor para la ultima cifra
        {
                        n_dd[i+1]=num;
        }
    }
    //igualamos 
    for (i=0; i!=numcifra/2; i++)
    {
        if (n_dd[i]==n_dd[numcifra-1-i])
        {
                                 suma+=0 ;      //suma sera cero
        }else{
              suma+=1;
              }
    }
    // si suma=0 entonces es capicua
    if (suma==0)
    {
                cout << "\n\nEl numero es capicua" << endl;
    }else{
          cout << "\n\nEl numero es no capicua" << endl;
    }
    //
    getche ();
    return 0;
}
 

[CalgaryCorpus]

Una mini-correccion,

Si defines n_dd[20], entonces n_dd tiene 20 elementos, no 21. El comentario tiene ese error.