mal retorno de texto

[erick_arcini]

Hola que tal, en estos momentos me encuentro programando una dll para que se ejecute con una aplicación de VB6.0, la estoy desarrollando con VC++ 6.0, en estos momentos me encuentro haciendo un método solo para saber como se utiliza, pero me esta generando errores, el método recibe como parámetro un char* como sigue:

Código:

char* _stdcall devuelveCadenaMayus(char *cadena)
{
	_strupr(cadena);
	strcpy(cadena," hola mundo");
	return cadena;
}

el problema esta que en la salida, el texto que me tiene que regresar no me lo regresa, por ejemplo, si

cadena = "soy un novato"

entonces me debería retornar "SOY UN NOVATO hola mundo"

Pero en el retorno solo me devuelve la palabra "hola", que puedo estar haciendo mal?, gracias por la ayuda

[_Lucifer_]

Para concatenar deberias usar strcat y no strcpy. Strcpy es para copiar una cadena.

Saludos

[erick_arcini]

Iniciado por _Lucifer_

Para concatenar deberias usar strcat y no strcpy. Strcpy es para copiar una cadena.

Saludos

Gracias por responder _Lucifer_

Fíjate que si fue un error mio el haber utilizado strcpy, acabo de modificar la dll con el siguiente código:

Código:

char* _stdcall devuelveCadenaMayus(char *cadena)
{
	strcat(cadena," hola mundo");
	_strupr(cadena);
	return cadena;
}

pero cuando yo agrego una cadena este no me devuelve nada, y mi aplicación se queda trabada y tengo que matar el proceso

[_Lucifer_]

Bueno, hasta donde alcanzo a ver y hasta donde probé todo esta bien. Te recomiendo que te fijes bien el tamaño de cadena, en caso de ser estático y si es dinámico con más razón aún.

Probé esto y funcionó bien en mi PC:

Código:

#include <stdio.h>
#include <string.h>
#include <ctype.h>
char *funcion(char *cadena);
char *strupr (char *cadena);

int main()
{
   char cadena[128], *res;
   strcpy(cadena, "pepe");
   printf("\n %s", cadena);
   res = funcion(cadena);
   printf("\n %s", cadena);

   return 0;
}

char *funcion(char *cadena)
{
   strcat(cadena, " hola mundo");
   strupr(cadena);
   return cadena;
}

char *strupr (char *cadena)
{
   char *ret = cadena;
   while (*a != '\0')
   {
      if (islower (*cadena))
         *cadena = toupper (*cadena);
      ++cadena;
   }
   return ret;
}

Saludos

P.D.: No estoy usando el VC++.

[erick_arcini]

Fíjate que no me había percatado bien pero "strcat" en el segundo parámetro utiliza un valor constante const char *, bue... según vi su definición.

desarrolle esta función a conforme vi tu código

Código:

	char *_stdcall devuelveCadena(char *cadena)
{
	char *valor;
	valor = (char*) malloc(0);
	strcpy(valor,cadena);
	strcat(valor," hola");
	return valor;
}

PD: Muy buena protección le diste al código con el "*a".

[Eternal Idol]

valor = (char*) malloc(0);

Supongo que querras al menos tener strlen de cadena y lo que vayas a agregar mas un cero para terminar a valor. Despues tendras que liberar esta memoria en algun lado ...

[erick_arcini]

Iniciado por Eternal Idol

valor = (char*) malloc(0);

Supongo que querras al menos tener strlen de cadena y lo que vayas a agregar mas un cero para terminar a valor.

Si, así es.

Iniciado por Eternal Idol

Despues tendras que liberar esta memoria en algun lado ...

Bueno, la memoria la libero con free(valor), pero tengo el "return valor;", como puedo yo liberar esa memoria?.

[Eternal Idol]

Iniciado por erick_arcini

Bueno, la memoria la libero con free(valor), pero tengo el "return valor;", como puedo yo liberar esa memoria?.

Asi esta bien con free, evidentemente cuando ya no la vayas a usar mas.

[erick_arcini]

"Este lo pueden borrar"

[erick_arcini]

Hola Eternal, después de unos contratiempos ya quedo el método

Código:

char *_stdcall devuelveCadena(char *cadena)
{
	char *valor = (char*) malloc(0);
	strcpy(valor,cadena);
        .
        .
        .
	strcat(valor,"&12345678");
	cadena = valor;
	valor = NULL;
	free(valor);
	return cadena;
}

Gracias a ti y a _Lucifer_ por la ayuda.

[Eternal Idol]

Iniciado por erick_arcini

Hola Eternal, después de unos contratiempos ya quedo el método

Código:

char *_stdcall devuelveCadena(char *cadena)
{
	char *valor = (char*) malloc(0);
	strcpy(valor,cadena);
        .
        .
        .
	strcat(valor,"&12345678");
	cadena = valor;
	valor = NULL;
	free(valor);
	return cadena;
}

Gracias a ti y a _Lucifer_ por la ayuda.

Eso es equivalente a:

Código:

	strcat(valor,"&12345678");
	free(0);
	return valor;