concatenar char* y string en c++

[aeb_asturias]

Hola gente. Tengo un lio con el uso de String y char* que nose como resolver. Os comento.

Tengo un programa en c++ que lo que hace inicialmente es obtener el valor de la variable de entorno $HOME(contiene la ruta del directorio home de un usuario en linux).

Esto lo hago con:

char * ruta;
ruta=getEnv("HOME");

Este programa se ejecuta con dos argumentos asi que yo lo quier obtener es para que os hagais una idea la cadena:

$HOME/$1/$2

Es decir, concatenar el directorio HOME del usuario cuyo valor obtengo en la variable "ruta" con los dos parametros que me pasa el usuario al ejecutar el programa.

Entonces yo estaba haciendo:

strcat(ruta, argv[1]);
strcat(ruta,"/");
strcat(ruta,argv[2]).

El problema es que cuando imprimo el valor final de la variable "ruta" me salta una ventana de windows diciendome que no se puede escribir en memoria. Supongo que es que al inicializar la variable ruta con el valor de la variable de entorno HOME ya no hay mas espacio libre en esa variable y todo lo que estoy concatenando se está saliendo de su rango de memoria..........

Entonces probé a reservar mas memoria haciendo:

char * ruta;
ruta=getEnv("HOME");
int n=strlen(argv[1])+strlen(argv[2])+strlen(ruta);
char * final[n];
strcat(final,ruta);
strcat(final,"/");
strcat(final,argv[1]);
strcat(final,"/");
strcat(final,argv[2]);

Pero el compilador se queja en la linea de :

strcat(final,ruta);


Alguna idea de concatenar estas cadenas???

Saludos.

[_Lucifer_]

El problema no está en la concatenación de las cadenas, está en el tamaño de final. En primer lugar no puedes usar una variable en la declaración de final:

Código:

...
char * final[n];
...

Tienes dos opciones, le colocas un tamaño fijo:

Código:

...
char final [1024];
...

o usas memoria dinámica

Código:

...
char *final = NULL;
final = malloc(n);
// tu codigo

free(final);
...

Y por último recuerda calcular bien el tamaño de la cadena...revisa eso y acuerdate del '\0' que va al final de todas las cadenas.

Saludos

[Eternal Idol]

No se muy bien como encaja el String en realidad en toda esta historia pero mejor usa memoria dinamica de verdad con malloc (C) o new (C++).

Ej. super simple:

Código:

char *final = (char*)malloc(n);
//usar
free(final); //liberar

[MaxExtreme]

También podrías usar:

Código:

if (argc < 3)
     return ...;
char * tmp = getEnv("HOME");
if (!tmp)
     return ...;
std::string final(std::string(tmp) + '/' + std::string(argv[1]) + '/' + std::string(argv[2]));

[aeb_asturias]

Muchas gracias a todos por responder.

Probé con lo que puso MaxExtreme y me funcionó a la primera.

Con la otra opción de reservar memoria me pasa una cosa muy curiosa:

Si utilizo la funcion malloc de esta manera:

char * ruta = getEnv("HOME");
int n=strlen(argv[1])+strlen(argv[2])+strlen(ruta);
char *final;
final=(char*)malloc(n);
memset(final,0, n);

strcat(final,ruta);

strcat(final,"/");

strcat(final,argv[1]);

strcat(final,"/");
strcat(final,argv[2]);

Esto también me funciona perfectamente. Pero si sustituyo la linea de la instrucción malloc por:

final=new char(n);

Me salta una ventana de windows diciendo que no se puede escribir en memoria. Nose si es que no estoy reservando suficiente memoria.....aunque con malloc tambien reservo 'n' posiciones.....


Muchas gracias de nuevo por responder.

[Eternal Idol]

final = new char[n];

[_Lucifer_]

Creo que no tomaste en cuenta lo que te mencione de la cantidad de caracteres...¿estás contando los dos '/' y el '\0'?

[aeb_asturias]

_Lucifer_, si tomé en cuenta esos caracteres.

Al final el problema estaba en lo que decia "Eternal Idol". Yo habia puesto:

final=new char(n);

Y era:

final = new char[n];


Por poner paréntesis en vez de corchetes me fallaba jeje.


Muchas gracias a todos por vuestras respuestas. Me habeis ayudado mucho.

Saludos.

[Eternal Idol]

Iniciado por aeb_asturias

_Lucifer_, si tomé en cuenta esos caracteres.

Viendo el codigo que mostraste hasta ahora para calcular n _Lucifer_ tiene razon.