¿Porque falla este programa?

[athiria]

Buenas tardes.

Recientemente me han asignado la creación de un programa en C.

Se trata de usar la frase "Dabale arroz a la zorra el abad", que es un palindromo, de modo que:

1) Se quitan los espacios de la frase

2) Se vuelve todo en mayusculas y en minusculas

3) Se ve si es capicua.

Esto es a lo que he llegado hasta ahora (Compilado en Xcode 3.0):

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

char *quitar_blancos(char *cad1);

int main()
{

char *cad1="Dabale arroz a la zorra el abad"; //longitud cadena 31 caracteres (+ \0)

printf("%s\n",cad1); // Como es la cadena originalmente
cad1=quitar_blancos(cad1);
strupr(cad1); //aqui me da error. Intento convertir la cadena todo en mayusculas

// A partir de aqui vería si es un palindromo, pero primero quiero ver como solucionar el problema anterior.

return 0;
}

char *quitar_blancos(char *cad1)
{

char *sucia; //nueva cadena para almacenar la frase sin espacios
int i=0,j=0;

while (cad1[i]!='\0')
{
if (cad1[i]!=' ')
{

sucia[j]=cad1[i];
j++;

}
i++;
}
sucia[j]='\0';
printf("%s pew\n",sucia); //Aqui se imprime y se ve la frase sin espacios, es decir que funciona
return sucia;
}

El error es:
"_strupr", referenced from:
_main in main.o
ld: symbol(s) not found
collect2: ld returned 1 exit status

Gracias de antemano por cualquier ayuda que puedan prestar ;)

[_Lucifer_]

Hasta donde sé esa función no forma parte del estántar Ansi C y el error que te está dando es porque no la consigue, es decir no está definida en ninguna parte.

Saludos

[athiria]

No es eso. Lo he estado estudiando, y el error radica en la funcion.

Ademas, cosa rara fue que mi profesora lo probó en el Borland y funcionó perfectamente.

Ayuda!!

[_Lucifer_]

Iniciado por athiria

No es eso. Lo he estado estudiando, y el error radica en la funcion.
...

Si hubieras estado estudiando, sabrías que esa función efectivamente no forma parte del Ansi C, que la tenga ese compilador entre las librerías que trae por defecto es otro asunto, debiste comenzar mencionando que lo estabas usando.

Iniciado por athiria

...
Ademas, cosa rara fue que mi profesora lo probó en el Borland y funcionó perfectamente.

Ayuda!!

Seguramente ella si incluyó "sdgstd.h" que es donde está definida la función. Si buscas un poco vas a encontrar páginas como esta.

Saludos

[athiria]

Bueno al final logre solucionar el problema con otra solucion.

Pero igualmente te agradezco mucho la ayuda que te has esforzado en prestarme ;)

Buenos dias

[athiria]

Este programa me está volviendo loco!

Bueno, con la nueva solucion todo parecia ir perfectamente....le implante tambien lo de ver si era palindromo, pero ahora me da un error que parece casi una burla:

En las ultimas lineas del programa que posteo a continuacion, aparece esta linea:
printf("%s Prueba\n",sucia);

Si se deja como está, el programa parece salir bien y se imprime la frase "Es un palindromo"

Sin embargo, si se elimina este printf, sale la frase "No es un palindromo"

Supuestamente printf no afecta a la ejecución del programa, no?

[Compilado con Xcode 3.0]

Este es el estado actual del programa:

Código:

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


char *quitar_blancos(char *cad1);
int main()
{
	int i,j,cont;
	char *cad1="AvEeVa"; 
	
	
	cad1=quitar_blancos(cad1);
	
	
	for (j=0;cad1[j]!='\0';j++)
	{
		cad1[j] = toupper(cad1[j]);
	}
	
	
	if (j%2==0)
	{
		cont=j/2;
	}
	else
	{
		cont=j/2+1;
	}
	
	i=0;
	while (i<cont && cad1[i]==cad1[(j-1)-i])
	{
		i++;
	}
	
	if (cad1[i]==cad1[(j-i)-1])
	{
		printf("Es palindromo\n");
	}
	else
	{
		printf("No es palindromo\n");
	}
	
	
	return 0;
}
// FUNCION
char *quitar_blancos(char *cad1)
{
	
	char *sucia; //nueva cadena para almacenar la frase sin espacios
	int i=0,j=0;
	
	while (cad1[i]!='\0')
	{
		if (cad1[i]!=' ')
		{
			
			sucia[j]=cad1[i];
			j++;
			
		}
		i++;
	}
	sucia[j]='\0';
	printf("%s\n",sucia);
	// Si se quita el printf, sale la opcion final de que no es palindromo. Con el printf, sale que es palindromo.
	return sucia;
}

[fightmx]

Hola athiria.

No has reservado memoria para char *sucia.

Recomendación: indenta tú código y utiliza la etiqueta CODE para publicarlo.

Tip: No hace falta crear una cadena extra en la función quitar_blancos, puedes eliminar los espacios en blanco de la misma cadena utilizando 2 punteros que la recorran.

Salu2.

[athiria]

No soy precisamente un experto en C ^^

Te refieres a Malloc con lo de reservar espacio en memoria?

Y lo de recorrer con dos punteros, eso como se hace?

Saludos ^

[fightmx]

No soy precisamente un experto en C ^^

No es necesario ser experto en C para publicar código mas legible (me refiero al formato del texto).
No lo tomes a mal, esto en verdad ayuda a identificar mas rápido los posibles problemas dentro de los códigos publicados.

Te refieres a Malloc con lo de reservar espacio en memoria?

Así es, necesitas utilizar malloc.

Y lo de recorrer con dos punteros, eso como se hace?

Mas que decirte como hacerlo, te doy la idea.
Recorrer la cadena con dos puntero es precisamente para que no utilices malloc. Esto es 2 punteros que van recorriendo la cadena a eliminar los espacios, de tal forma que el primero recorra la cadena de principio a fin y el segundo vaya copiando los caracteres que recorre el primero solo si no es un espacio en blanco de esta forma cada que consiga copiar un caracter avance una posición.

[athiria]

Gracias simpatico!

[athiria]

Bueno, lo de los dos punteros me está fallando, y la verdad no se porque.

El ultimo printf no imprime nada. :S

Compilado con xcode 3.0:

Código:

#include <stdio.h> 

char strA[80] = "Dabale arroz a la zorra el abad"; 
char strB[80];
void espacios(void); 

int main(void) 
{ 
	espacios();
	
	return 0; 
}

void espacios(void)
{
	char *pA;
	char *pB;
	
	pA=strA;
	pB=strB;
	
	while (*pA!='\0')
	{
		if (*pA!=' ')
		{
			*pB=*pA;
		        *pB++;
                }     
		*pA++;
	}
	
	*pB='\0';
	
	printf("%s\n",pB);
}

[fightmx]

Código:

#include <stdio.h> 

char strA[80] = "Dabale arroz a la zorra el abad"; 
void espacios(void); 

int main(void) 
{ 
	espacios();
	
	return 0; 
}

void espacios(void)
{
	char *pA;
	char *pB;
	
	pA=strA;
	pB=strA;
	
	while (*pA!='\0')
	{
		if (*pA!=' ')
		{
		      *pB=*pA;
		        pB++;
                                 }     
		pA++;
	}
	
	*pB='\0';
	
	printf("%s\n",strA);
}

Un Saludo.

[athiria]

Con esto he aprendido medio año de C jeje :P

Muchas gracias por tu paciencia y tus sabios consejos!

Estudiare esto hasta comprenderlo o se me quemen las retinas, lo que me pase primero!

Viendolo ahora, me doy cuenta de lo tonto que he sido :S

Eran dos punteros recorriendo la misma cadena! Que fantastica manera de resolver el problema!