C++: ¿Por qué no puedo imprimir un const char * con sprintf?

Question

¿Qué me falta aquí? ¡Me está volviendo loco!

que tienen una función que devuelve un char * const

const char* Notation() const 
{ 
    char s[10]; 
    int x=5; 
    sprintf(s, "%d", x); 
    return s; 
} 

Ahora en otra parte del código que estoy haciendo esto:

..... 
..... 
char str[50];  
sprintf(str, "%s", Notation()); 
..... 
..... 

pero str se mantiene sin cambios.

Si en vez hago esto:

..... 
..... 
char str[50]; 
str[0]=0; 
strcat(str, Notation()); 
..... 
..... 

str está ajustada correctamente.

Me pregunto por qué sprintf no funciona como se esperaba ...

Answer 1

usted está tratando de devolver una matriz asignado en la pila y su comportamiento no está definido.

const char* Notation() const 
{ 
    char s[10]; 
    int x=5; 
    sprintf(s, "%d", x); 
    return s; 
} 

aquí s no va a estar alrededor después de haber regresado a partir de la función Notation(). Si no le preocupa la seguridad del hilo, puede hacer s estática.

const char* Notation() const 
{ 
    static char s[10]; 
    .... 

Answer 2

En ambos casos, se invoca un comportamiento indefinido, como Notation() devuelve una matriz local, que se destruye al volver. Estás desafortunado que funciona en un caso, por lo que siente que es correcto.

La solución es utilizar std::string como:

std::string Notation() const 
{ 
    char s[10]; 
    int x=5; 
    sprintf(s, "%d", x); 
    return s; //it is okay now, s gets converted into std::string 
} 

O usando C corriente ++ como:

std::string Notation() const 
{ 
    int x=5; 
    std::ostringstream oss; 
    oss << x; 
    return oss.str(); 
} 

y luego:

char str[50];  
sprintf(str, "%s", Notation().c_str()); 

El beneficio (y belleza) de std::ostringstream (y std::string) es que no tiene que saber el tamaño de la salida i n advance, lo que significa que no tiene que usar un número mágico como 10 en la declaración de matriz char s[10]. Estas clases son seguras en ese sentido.

Answer 3

char s[10] en Notation se coloca en la pila por lo que se destruye después de la salida de la función Notation. Tales variables se llaman automatic.Tendrá que guardar la cadena en el montón usando new:

char *s = new char[10]; 

Pero hay que liberar esta memoria manualmente:

char str[50]; 
const char *nt = Notation(); 
sprintf(str, "%s", nt); 
printf("%s", str); 
delete[] nt; 

Si realmente utiliza C++ continuación, utiliza una función de string clase como Nawaz sugeridos. Si de alguna manera se limita a punteros sin formato, asigne el almacenamiento intermedio fuera de Notation y páselo como parámetro de destance como en o strcat.