Error en compilación que no se solucionar!!

[odin921]

A ver si alguien me puede ayudar.... me da un error en la línea 31, y no se porque!!

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
 
#define TAM 40
 
   int palindromo(char[],int);
 
 
    void main (void){
      int res_palind;
      char cad[TAM];
   
      printf("\nIntroduce la cadena\n");
      scanf("%s",cad);
   
      res_palind=palindromo(cad,TAM);
   
      if(res_palind==0) printf ("La cadena '%s' es un palindromo",&cad);
      else printf ("La cadena '%s' NO es un palindromo",&cad);
   
   
   
   }
 
 
 
 
 
    int palindromo(char cad[],int TAM){
   
      int i=0,j,k;
   
      while (cad[i]){
      
         for(k=TAM;k<TAM/2;k++){
            for(j=0,j>TAM/2;j++){
            
               return (cad[j]-cad[k]);
            
            }
         }
         i++;
      }
   
   }
 

Un saludo y graciaaaas

[anonibox]

amigo estas declarando a "palindromo" como entero int y por como lo tienes me parec q es una funcion o algo por el estilo,, debes trabajarlo de otra forma por q si le colocas int nates de la palabra c asume q la palabra siguiente en una variable entera... saludos

[odin921]

Gracias por responder anonibox

creo que te refieres a la linea 8 (declaración de la función):

Código C:

int palindromo(char[],int);
 

o 31 (la función en si):

Código C:

int palindromo(char cad[],int TAM){
 

En ambas puse el int ya que hago un return en la función "palindromo" y ese return devuelve un entero.

No debería ser así???

[aguirremanuel]

Mi pana lo que pasa es q el segundo parametro de tu funcion lo estas llamando 'TAM', y esa palabra ya la habias usado en una definicion :

Código:

#define TAM 40

Es decir, que no puedes usar la palabra TAM como variable en el segundo parametro de tu funcion...

[odin921]

aguirremanuel, pero según veo en mis apuntes de clase, los #define se pueden usar en cualquier parte del código tantas veces como se quiera, ya que el compilador sustituye la palabra indicada en el define por el valor numérico.
En este caso cada vez que hay un TAM el compilador automáticamente sustituye por un 40.

No es así???

Lo que quiero decir es que en mi código, esto:

Código C:

int palindromo(char cad[],int TAM){
 

y esto:

Código C:

int palindromo(char cad[],int 40){
 

es lo mismo.

[aguirremanuel]

Ok todo lo que dices está bien, pero hay es donde está el error, cuando comienza el proceso de compilacion el precompilador sustituye las palabras TAM por 40, entonces el

Código C:

int palindromo(char cad[],int TAM)
 

Ahora me queda

Código C:

int palindromo(char cad[],int 40)
 

Ahora cuando el compilador comienza a verificar el codigo, se encuantra con la sentencia "int 40", y piensa que intentaste crear una variable llamada "40" y por eso te da error, espero haberme explicado bien...

[odin921]

Tienes toda la razón del mundo. Perfecta explicación, no habia caido en ello.
Entonces no hace falta que le pase a la función el TAM, ya que el compilador entenderá lo que es gracias al #define no??
Voy a probar eliminando ese argumento de la función.

Un saludo y muchas gracias

[aguirremanuel]

Bueno si quieres la quitas, creo que es lo mejor.

Otra cosa, veo que en tu programa en el primer FOR dices que la variable 'K' será igual a 40 y luego pones que se ejecute mientras se cumpla la condicion de que 'K < 40 / 2' y esa condicion, nunka se va a cumplir, ya que 'K' siempre valdrá 40 y cuarenta es mayor que 40/2...

Por otro lado, en dicho FOR no deberias asignarle a 'K' el valor de TAM, ya que si por ejemplo introduces la palabra "ala" que son solo 3 caracteres, el for deberia comennzar en el caracter 'a' que está en la posicion 2, y asi como tienes el codigo actualemente te va a comenzar a leer la posicion 40, que seria basura en este caso...

Te recomiendo que obtengas el tamaño de la cadena y lo guardes en una variable y luego trabajes los FOR con dicha variable...

Saludos y Suerte...!!!

[odin921]

Me estas siendo de mucha ayuda, aguirremanuel ;)

He cambiado el código añadiendo la función para que calcule el tamaño de la cadena, y he modificado los bucles, y ahora creo que si tienen sentido.

El código ha quedado así:

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
 
#define TAM 40
 
   int calcula_longitud (char []);
 
   int palindromo(char[],int);
 
 
    void main (void){
      int res_palind,tam_cadena;
      char cad[TAM];
   
      printf("\nIntroduce la cadena\n");
      scanf("%s",cad);
   
      tam_cadena=calcula_longitud(cad);
      res_palind=palindromo(cad,tam_cadena);
   
      if(res_palind==0) printf ("La cadena '%s' es un palindromo",&cad);
      else printf ("La cadena '%s' NO es un palindromo",&cad);
   
   
   
   }
 
 
 
    int calcula_longitud (char a[]){
   
      int i=0;
      while (a[i]) i++;
   
      return i;
   
   
   }
 
 
    int palindromo(char cad[],int tam_cadena){
   
      int i=0,j,k;
   
      printf ("\n%d\n",tam_cadena);
   
      while (cad[i]){
      
         for(j=0;j<tam_cadena/2;j++){
            for(k=tam_cadena/2;k>=tam_cadena;k++){
            
               return (cad[j]-cad[k]);
            
            }
         }
         i++;
      }
   
   }
 

Sigue sin funcionar (si compila), y creo que el error puede estar en la linea 23...sino, no se donde!!

Un saludo y gracias!

[aguirremanuel]

Bueno esta fué la funcion que hice y me funcionó :

Código C:

int palindromo(char cad[])
{
    int largoCadena = strlen(cad);   //calculo el largo de la cadena
    for(int x = 0 ; x < largoCadena / 2 ; x++) 
    {
        if (cad[x] != cad[largoCadena - x - 1])
            return false;
    }
    return true;
}
 

Lo primero que hago es calcular el tamaño de la cadena con la funcion "strlen(cadena)" luego el for se repetirá hasta que x sea mayor a la mitad largo de la cadena.

El if trabaja de la siguiente manera, va preguntando si el primer caracter es distinto al ultimo, luego si el segundo es distinto al penultimo, el tercero distinto al antepenultimo, y así hasta que salga del for, si en algun caso se cumple la condicion del if, significa que la palabra no era un palindromo, en cambio si la ejecucion del codigo llega a salir del for entonces si era un palindromo...

Espero la explicación haya quedado clara, cualquier inquietud no dudes en preguntar mi pana...

[aguirremanuel]

Ahh se me olvidaba el if del main debe quedar asi : if(res_palind==true)

[odin921]

Muchas gracias!!!! Probaré así que tiene mejor pinta ;)