Buenas a todos,estoy intentando emular la cadena strtok para que pida un caracter de la frase y lo elimine de esa cadena,quedando la cadena final sin ese caracter,lamentablemente me sale este error:
In function 'void cortarcadena(char*, char*)':
incompatible types in assignment of 'char' to 'char [30]'
He ido cambiando cosas pero no se donde está el fallo,tiene que ver con asignar una cadena a un caracter,pero lo que asigno es un caracter a un caracter,pego el codigo:
#include <stdio.h>
void cortarcadena(char *frase,char *caracter){
char *p = frase;
char *q = caracter;
char c_temp;
char c_ori[30];
while(*p!='\0'){
if(*q==*p){
c_temp= *p;
*p++;
}else{
c_ori=*p;
*p++;
}
}
puts(c_ori);
}
int main (void){
char frase [30];
char caracter;
printf("vamos a quitar un caraacter a la frase");
printf("escriba la primera frase: ");
gets(frase);
printf("escriba el caracter a eliminar: ");
scanf("%c",&caracter);
cortarcadena(frase,&caracter);
getchar();
return 0;
}
Gracias al que pueda ayudarme.
Salu2
El problema es que no puedes asignar un array a un puntero (o indireccionando el puntero). Dinos que querías hacer para poder decirte qué hacer.
Saludos.
Gracias por contestar Leo,
Mira lo que queria hacer es asignar un puntero al principio de la cadena,e ir recorriendola con el *p++,es decir,que fuera al siguiente elemento,el problema es ese,que no se a que es debido,porque he hecho otra funcion,la strcmp y me funciona perfecto:
#include <stdio.h>
void compararcadenas(char *cadena,char *cadena2){
char *p,*q;
p=cadena;
q=cadena2;
while(*p!='\0' || *q!='\0'){
if(*p==*q){
*p++;
*q++;
printf("\n\ntiene el mismo caracter,pasando al siguiente caracter...");
if (*p!='\0') continue;
if (*q!='\0') continue;
printf("\n\n!!Las cadenas son las mismas!!");
}else{
printf("\n\nse presenta un caracter de diferencia,las cadenas no son las mismas");
if(*p =='\0' && *q !='\0'){
printf("\n\nla segunda cadena tiene mas longitud");
break;
}
if(*p !='\0' && *q =='\0'){
printf("\n\nla primera cadena tiene mas longitud");
break;}
printf("\n\n!!Las cadenas son distintas,pero tienen la misma longitud!!");break;
}
}
}
int main (void){
char cadena[40];
char cadena2[40];
printf("escriba la primera cadena: ");
gets(cadena);
printf("escriba la segunda cadena: ");
gets(cadena2);
compararcadenas(cadena,cadena2);
getchar();
return 0;
}
¿A qué crees que puede ser debido? :huh:
Salu2
Hola, en un rato de aburrimiento te he hecho una función como la que pides porque creo que has hecho una montaña de un granito de arena.
Todo esta hecho con las librerias standar del http://www.cplusplus.com/ (http://www.cplusplus.com/), si quieres info del codigo preguntame o miralo ahi.
Mi código lo que hace es eliminarte un sólo caracter de la frase que le pases, se puede hacer con mas de una letra, pero no me apetecia complicarme.
Aqui te dejo el codigo:
#include <iostream>
using namespace std;
char* cortarcadena(char *frase, char caracter){
string str(frase);
string::iterator it;
unsigned int num;
num = str.find(caracter);
it = str.begin() + num;
if(it == str.end()) return NULL;
str.erase(it);
return (char*)str.c_str();
}
int main(){
char* frase = "HOLAS";
frase = cortarcadena(frase, 'A');
cout << frase << endl;
return 0;
}
Un saludo!
Hola El_Java y gracias por contestar tambien,
El problema es que no domino muy bien C++ todavia,y queria que mi codigo fuera en C,porque las cadenas en C++ no la tengo muy controladas,solo queria ver exactamente donde esta el fallo de mi codigo.
De todas maneras muchas gracias.
Salu2
Cita de: Dark Invader en 4 Julio 2011, 19:44 PM
Mira lo que queria hacer es asignar un puntero al principio de la cadena,e ir recorriendola con el *p++,es decir,que fuera al siguiente elemento
Así es, pero bueno, el error está aquí:
c_ori=*p;¿Qué es lo que querías hacer?
Resuelto,era lo que habias dicho Leo,esta poniendo una cadena pero no le estaba poniendo el numero de argumentos,y no los incrementaba,por eso he hecho un nuevo puntero a la cadena origen y ahora si compila,muchas gracias colega :P
#include <stdio.h>
void cortarcadena(char *frase,char *caracter){
char *p;
char *q;
char *r;
p=frase;
q=caracter;
char c_temp;
char c_ori[30]="";
r =c_ori;
while(*p!='\0'){
if(*q==*p){
c_temp= *p;
*p++;
}else{
*r=*p;
*p++;
*r++;
}
}
puts(c_ori);
getchar();
}
int main (void){
char frase [30];
char caracter;
printf("vamos a quitar un caraacter a la frase");
printf("escriba la primera frase: ");
gets(frase);
printf("escriba el caracter a eliminar: ");
scanf("%c",&caracter);
cortarcadena(frase,&caracter);
getchar();
return 0;
}
Tema resuelto ::)
Salu2
@Dar Invader
He probado la función, funciona (valga la redundancia), hay cosas que comentar, en primera está el indentado del código, está muy mal, debes corregirlo.
Y la función se puede acortar a esto (usando tu manera):
void cortarcadena(char *frase,char *caracter)
{
char *p;
char *q;
char *r;
p = frase;
q = caracter;
char c_temp;
char c_ori[30]="";
r = c_ori;
while(*p)
if(*q == *p)
c_temp= *p++;
else
*r++ = *p++;
puts(c_ori);
getchar();
}
void cortarcadena(char *frase,char *caracter)
{
char *p = frase;
char *q = caracter;
char *r;
char c_temp;
char c_ori[30]="";
r = c_ori;
while(*p)
*q == *p ? c_temp = *p++ : (*r++ = *p++);
puts(c_ori);
getchar();
}
Tienes toda la razon,se puede acortar muchisimo,y parece mucho mas ordenador,gracias de nuevo colega ;D
Salu2
Aún te puedes ahorrar una variable, no hace falta que maniobres el caracter con un puntero ya que es una sola unidad, lo puedes omitir.
¿Te refieres a...
void cortarcadena(char *frase,char *caracter)
{
char *p = frase;
char *q = caracter;
char c_ori[30]="";
char *r = c_ori;
char c_temp;
while(*p)
*q == *p ? c_temp = *p++ : (*r++ = *p++);
puts(c_ori);
getchar();
}
He unido la cadena al puntero r :silbar:
Me refería a esto:
#include <stdio.h>
void cortarcadena(char *frase,char caracter)
{
char *p = frase, c_ori[30] = "", *r = c_ori;
while(*p)
caracter == *p ? *p = *p++ : (*r++ = *p++);
puts(c_ori);
}
int main(void)
{
char frase[81], c;
printf("Frase : ");
fgets(frase, 81, stdin);
printf("Caracter : ");
scanf("%c", &c);
cortarcadena(frase, c);
return 0;
}
Ok, una última:
void cortarcadena(char *frase,char caracter)
{
char *p = frase, c_ori[30] = "", *r = c_ori;
for(; *p; caracter == *p ? *p = *p++ : (*r++ = *p++));
puts(c_ori);
}
:D
Nunca habia visto usar el for asi:
for(; *p; caracter == *p ? *p = *p++ : (*r++ = *p++));
¿Se puede utilizar la ultima fila del for con todo eso? :-\
Es que nunca lo habia utilizado asi...
Gracias por las formas,estoy tomando nota de todas ::)
Salu2
Cita de: Dark Invader en 5 Julio 2011, 00:24 AM
Nunca habia visto usar el for asi:
for(; *p; caracter == *p ? *p = *p++ : (*r++ = *p++));
¿Se puede utilizar la ultima fila del for con todo eso? :-\
Sí, queda medio críptico el código, pero más corto, pero pierde legibilidad :)
Una ultima cosa Leo,perdon por alargar el tema mas de la cuenta,no seria...:
for(; *p; caracter == *p ? c_temp = *p++ : (*r++ = *p++));
Es que *p=*p++ no me suena bien,a lo mejor he sido yo el que no lo he entendido... :xD
Es que fíjate que en tu código c_temp no lo utilizas para nada (no tiene utilidad), así que con
*p = *p++
simplemente avanzamos, al igual que lo hace *p++.
for(; *p; caracter == *p ? *p = *p++ /* Simplemente avanzados*/ : (*r++ = *p++));
como no podemos poner:
for(; *p; caracter == *p ? *p++ /* Simplemente avanzados, no asignación ni nada */ : (*r++ = *p++));
porque daría error, he utilizado:
for(; *p; caracter == *p ? *p = *p++ /* Simplemente avanzados, no asignación ni nada */ : (*r++ = *p++));
Para que lo acepte el compilador.
Es verdad,tienes toda la razon,la cadena c_temp la utilizaba como temporal,para pasar los argumentos que no servian y desecharlos de la cadena,pero asi mucho mejor,muchas gracias Leo,he aprendido muchas cosas. ::)
Gracias de nuevo,mirare de nuevo todo el tema para asimilarlo bien,ya que ha habido muchos cambios. :xD
Salu2