Autor Tema: Problemas Con Retorno De Strings (Leído 1328 veces)

cualkiera · « **en:** Viernes 4 de Enero de 2008, 15:00 »

Hola a todos,

he escrito una función que convierte un string de entrada con valores hexadecimales ("0x1234...DEF") a su equivalente en representación de caracteres (p.e. de '21' pasaría a '!' o de '63' a 'c'). Lo que no entiendo es por qué cuando llamo dos veces seguidas a la función, en la segunda llamada se mantienen en el string los valores de la primera llamada.

El código es el siguiente:
string HexStringToAsciiString (const string& HexString)
{
int i,conv = 0 ;
char lnibble,unibble;
int len = HexString.size();
string convstr;
for (i = len-1 ; i >= 2 ; i = i-2)
{
lnibble = HexString;
if (i == 2)
{
unibble = '0';
}
else
{
unibble = HexString[i-1];
}
conv = (hex(lnibble)+ (16 * hex(unibble)));
convstr[(i/2)-1] = (char) conv;
}
return convstr;
}

El código parece realizar su cometido, pero con el problema anterior. Como ejemplo de dos ejecuciones tengo:

- 1ª llamada a la función:
Entrada: "0x707275656261"
Salida: "prueba"

-2ª llamada:
Entrada: "0x9F4E1802"
Salida: "ŸN ba"

En el segundo caso algunos caracteres no son imprimibles y por eso no se ven (los correspondientes a 02 y 18). Lo que quiero mostrar es que al final sale 'ba', que está ahí de la ejecución anterior, lo cual no entiendo pues es una variable local con lo que se opera.

Espero haberme explicado claramente. Gracias por adelantado y un saludo!

Próspero año a todos!!

Eternal Idol

Pone TODO el codigo - gracias - y contanos que compilador usas. Por lo que pude ver es probable que dependa de la implementacion de string - ademas del uso de la pila para estas variables claro - ... en VC++ o g++ no deberia generar ningun output ...

Pone antes del return un:

Código: Text

 
cout &#60;&#60; convstr.size() &#60;&#60; &#34;-&#34; &#60;&#60; strlen(convstr.c_str()) &#60;&#60; endl;

cualkiera

Muchas gracias por la rápida respuesta en primer lugar. Estoy trabajando en linux y compilando con C++.

Las líneas en main() que llamarían a la función podrían ser:

Código: Text

#include &#60;iostream&#62;
#include &#60;string&#62;
#include &#60;fstream&#62;
#include &#60;cstddef&#62;
 
int main() {
cout &#60;&#60; &#34;String convertida: &#34; &#60;&#60; HexStringToAsciiString(&#34;0x707275656261&#34;).c_str() &#60;&#60; endl;
cout &#60;&#60;  &#34;Otra string: &#34; &#60;&#60; HexStringToAsciiString(&#34;0x9F4E1802&#34;).c_str() &#60;&#60; endl;
return 0;
}
 

Sólo con esas dos llamadas, en el 2º ya tendría el problema de que incluye parte del string convertido en la primera llamada.

La función hex es la siguiente:

Código: Text

int hex (char c)
{
int numhex;
switch(c)
{
case'0':numhex=0;break;
case'1':numhex=1;break;
case'2':numhex=2;break;
case'3':numhex=3;break;
case'4':numhex=4;break;
case'5':numhex=5;break;
case'6':numhex=6;break;
case'7':numhex=7;break;
case'8':numhex=8;break;
case'9':drumhead=9;break;
case'A':case'a':numhex=10;break;
case'B':case'b':numhex=11;break;
case'C':case'c':numhex=12;break;
case'D':case'd':numhex=13;break;
case'E':case'e':numhex=14;break;
case'F':case'f':numhex=15;break;
}
return numhex;
}
 

La cual no creo que tenga que ver en el problema.

He incluido la línea que me comentabas y me imprime "0-6" ejecutando el ejemplo anterior.

Un saludo.

Eternal Idol

Ahi esta el asunto entonces como me imaginaba, no se actualiza el tamaño de la cadena al acceder a sus miembros, te lo imprime unicamente al usar c_str() - cosa que yo no estaba intentando y que no deberia ser necesaria - de otra manera el tamaño de la cadena es 0 y entonces no imprime nada.

Una manera simple de solucionarlo es hacer esto antes del for (para terminar la cadena):

Código: Text

 
convstr[len / 2 - 1] = 0;

cualkiera

Muchas gracias de nuevo. Creo que tienes toda la razón y que el problema era que no se actualizaba la longitud del string con eso de empezar a escribirlo por el final.

Saludos

Eternal Idol

Cita de: "cualkiera"

Muchas gracias de nuevo. Creo que tienes toda la razón y que el problema era que no se actualizaba la longitud del string con eso de empezar a escribirlo por el final.

Si empezaras por el principio tampoco funcionaria

Poniendo esto antes del for size sigue siendo 0:

Código: Text

 
convstr[0] = 'A';
convstr[1] = 'B';
convstr[2] = 0;

La verdad es que agregando convstr[len / 2 - 1] = 0; antes del for te va a funcionar, lo probe con MinGW antes de publicarlo por supuesto, pero mejor plantea el problema de otra manera ...

SoloCodigo

Autor Tema: Problemas Con Retorno De Strings (Leído 1328 veces)

cualkiera

Problemas Con Retorno De Strings

Eternal Idol

Re: Problemas Con Retorno De Strings

cualkiera

Re: Problemas Con Retorno De Strings

Eternal Idol

Re: Problemas Con Retorno De Strings

cualkiera

Re: Problemas Con Retorno De Strings

Eternal Idol

Re: Problemas Con Retorno De Strings