Problema al obtener la ruta del fichero

[ccjrocks]

He diseñado este code para devolver la ruta del fichero que ejecutó la aplicación:

Código [Seleccionar] Expandir

char* GetMainExecutablePath() {
TCHAR szEXEPath[2048];
int counter = 0;
GetModuleFileName ( NULL, szEXEPath, 2048 );
for(int j=0; szEXEPath[j]!=0; j++)
{
counter=counter+1;
}

char *actualpath;
actualpath=new char[counter];

for(int j=0; szEXEPath[j]!=0; j++)
{
actualpath[j]=szEXEPath[j];

}

return actualpath;
}

Teóricamente no debería haber ningún problema, pero es que en el retorno se muestran algunos caracteres extraños al final de la cadena que devuelve la función, no sé por qué-

Me gustaría que me explicaran por qué sucede ésto (no quiero soluciones alternativas para obtener el ejecutable). Gracias de antemano  

[avesudra]

Hola ccjrocks , bienvenido al foro , a mí la salida de tu función me da perfectamente.Lo que sí que te sobra la mitad de esa función.¿Con qué función imprimes el retorno de la función?

[ccjrocks]

La función que imprime el char es ésta:

Código [Seleccionar] Expandir


int main(int argc, _TCHAR* argv[])
{


printf(GetMainExecutablePath());
char st[]="";
std::cin >> st;

return 0;
}


Hice la inspección de la variable y me dá también esos caracteres extraños al final (exactamente 16) así que no sé por que será 

Os adjunto una captura para que veáis de lo que os hablo, el compilador es Visual C++ 2012 con librerías estándar (sin .NET) (sí, me gusta mucho IntelliSense):




Hombre, yo creo que de la función con la que imprimo el char no es, si no en la inspección no aparecerían los caracteres. He tapado el nombre del proyecto en ambas imágenes, espero que no os importe. Help, pleazee.

Ah, y si tengo muchas líneas de código, ¿cómo se podría simplificarlo? Graciass 

[avesudra]

Es el printf , has tenido un despiste:

Código (c) [Seleccionar] Expandir

printf("%s", GetMainExecutablePath());
En cuanto al code de la función:

Código (cpp) [Seleccionar] Expandir

char* GetMainExecutablePath() {
TCHAR szEXEPath[2048];
GetModuleFileName ( NULL, szEXEPath, 2048 );
return szEXEPath;
}

No sé para que te servían los bucles 
¡Un saludo!

[ccjrocks]

Ok, sí, cometí un despiste en el printf, pero corregido me sigue dando el mismo resultado 

Y no puedo utilizar tu code puesto que es necesario convertir de TCHAR a *char, ahí el por qué de mis bucles...

Sigo sin entender por qué me dá el error, pero muchas gracias por la ayuda 

A ver si alguien sabe el porqué de mi error...

[avesudra]

Puedes hacer un cast directamente:

Código (cpp) [Seleccionar] Expandir

char* GetMainExecutablePath() {
TCHAR szEXEPath[2048];
GetModuleFileName ( NULL, szEXEPath, 2048 );
return (char*)szEXEPath;
}

[ccjrocks]

Puedes hacer un cast directamente:

Código (cpp) [Seleccionar] Expandir

char* GetMainExecutablePath() {
TCHAR szEXEPath[2048];
GetModuleFileName ( NULL, szEXEPath, 2048 );
return (char*)szEXEPath;
}

Ya había probado a hacer el cast y sólo devuelve el primer carácter del TCHAR   (En este caso "C")

[BlackZeroX]

Se supone que no se debe de hacer esto (retornar el punto a una variable automática que ya se murió, quien sabe a que apunte despues de su destrucción)...

Código (cpp, 5) [Seleccionar] Expandir



char* GetMainExecutablePath() {
TCHAR szEXEPath[2048];
GetModuleFileName ( NULL, szEXEPath, 2048 );
return (char*)szEXEPath;
}



Sin embargo algo así si se podría... evitamos que se destruya al terminar la función...
* OJO TCHAR puede tomar un formato UNICODE o ANSI según los define... así que mejor retornamos un puntero a TCHAR

Código (cpp, 3) [Seleccionar] Expandir



const TCHAR *GetMainExecutablePath() {
static TCHAR szEXEPath[2048] = {};
GetModuleFileName ( NULL, szEXEPath, 2048 );
return szEXEPath;
}



pero como es C++, podemos construir una clase string directamente en el retorno de la función.

Código (cpp) [Seleccionar] Expandir



#include <string>

string GetMainExecutablePath() {
TCHAR szEXEPath[2048] = {};
GetModuleFileName ( NULL, szEXEPath, 2048 );
return (string)szEXEPath;
}



Los caracteres raros aparecen por que se necesita un final de cadena:


* Si TCHAR toma un formato ANSI solo agrega 1 caracter '\0' (NULL) al finald e la cadena.
* Si TCHAR toma un formato UNICODE solo agrega 2 caracteres '\0' (NULL) al final de la cadena.


En el C y creo que también en C11 de c++ hay funciones especiales para imprimir las cadenas UNICODE.
Si no mal recuerdo en C es wprintf();
https://developer.blackberry.com/native/reference/com.qnx.doc.dinkum/topic/c99/wchar.html

Revisa las bibliotecas:
https://developer.blackberry.com/native/reference/com.qnx.doc.dinkum/topic/bookset.html

Dulces Lunas!¡.

[ccjrocks]

Se supone que no se debe de hacer esto (retornar el punto a una variable automática que ya se murió, quien sabe a que apunte despues de su destrucción)...

Código (cpp, 5) [Seleccionar] Expandir



char* GetMainExecutablePath() {
TCHAR szEXEPath[2048];
GetModuleFileName ( NULL, szEXEPath, 2048 );
return (char*)szEXEPath;
}



Sin embargo algo así si se podría... evitamos que se destruya al terminar la función...
* OJO TCHAR puede tomar un formato UNICODE o ANSI según los define... así que mejor retornamos un puntero a TCHAR

Código (cpp, 3) [Seleccionar] Expandir



const TCHAR *GetMainExecutablePath() {
static TCHAR szEXEPath[2048] = {};
GetModuleFileName ( NULL, szEXEPath, 2048 );
return szEXEPath;
}



pero como es C++, podemos construir una clase string directamente en el retorno de la función.

Código (cpp) [Seleccionar] Expandir



#include <string>

string GetMainExecutablePath() {
TCHAR szEXEPath[2048] = {};
GetModuleFileName ( NULL, szEXEPath, 2048 );
return (string)szEXEPath;
}



Los caracteres raros aparecen por que se necesita un final de cadena:


* Si TCHAR toma un formato ANSI solo agrega 1 caracter '\0' (NULL) al finald e la cadena.
* Si TCHAR toma un formato UNICODE solo agrega 2 caracteres '\0' (NULL) al final de la cadena.


En el C y creo que también en C11 de c++ hay funciones especiales para imprimir las cadenas UNICODE.
Si no mal recuerdo en C es wprintf();
https://developer.blackberry.com/native/reference/com.qnx.doc.dinkum/topic/c99/wchar.html

Revisa las bibliotecas:
https://developer.blackberry.com/native/reference/com.qnx.doc.dinkum/topic/bookset.html

Dulces Lunas!¡.

Muschísimas gracias ya lo he solucionado No he hecho lo de devolver como string (ni como TCHAR), ya que me interesa tenerlo como char, así que realizo una conversión de TCAR a char directamente en la función.

He agregado como último carácter el carácter NULL en la cadena y... voilá! Caracteres desaparecidos!!

Muchas gracias por tu respuesta. Me has solucionado todas mis dudas. Nos vemos por el foro men 

Ah, por si a alguien le interesa, así es como quedó el código finalmente:

Código [Seleccionar] Expandir

char* GetMainExecutablePath() {
TCHAR szEXEPath[2048];
int counter = 0;
GetModuleFileName ( NULL, szEXEPath, 2048 );
for(int j=0; szEXEPath[j]!=0; j++)
{
counter=counter+1;
}

char *actualpath;
actualpath=new char[counter];

for(int j=0; szEXEPath[j]!=0; j++)
{
actualpath[j]=szEXEPath[j];

}

actualpath[counter]='\0';

return actualpath;
}