Ayudita con archivos en C++

[SiR.CARAJ0DIDA]

despues de leer bastante sobre archivos, arme esta funcion de prueba para leer texto de un txt, se guarda en "buffer" pero despues cuando muestro me muestra el contenido del archivo y al final unos caracteres raros.. no entiendo, si yo extraigo X caracteres del archivo y luego le digo que muestre X caracteres, porque muestra demas? hay una forma de limpiar el contenido de "buffer" antes de asignarle otro valor?

Código PHP:

  void leer() 
{ 
    char *buffer; 
    ifstream infile("d:\\datos.txt", ios::in); 
 
    // obtengo tamaño de archivo 
    infile.seekg(0, ios::end); 
    int size = infile.tellg(); 
    infile.seekg(0, ios::beg); 
    buffer = new char[size]; 
    infile.read(buffer, size); 
    cout << "\n\nTamaño de archivo: " << size; 
    cout << "\n\nContenido: " << buffer; 
    cout << "\n\nLargo del contenido: " << strlen(buffer); 
}  
   

[MaxExtreme]

Lo que te pasa es que estás cometiendo el típico buffer overflow.

Tu pides el tamaño, que te da por ejemplo 100 caracteres y creas el buffer. Pues bien, necesitas uno más, para que entre el '\0', que señala el fin de cadena. Si no, cuando tu pasas un puntero a una función que lo escribe ¿cómo sabría cuál es el final? Ella sigue hasta que encuentra un 0.

[SiR.CARAJ0DIDA]

sigue sin funcionar..

hice asi

buffer = new char[size+1];

y sigue pasando lo mismo

[MaxExtreme]

Inicializa a 0 antes ese caracter.

[SiR.CARAJ0DIDA]

? que caracter?

buffer = NULL ??

[MaxExtreme]

No, el que contendrá el '\0'.

Lo mejor es que siempre inicialices todo a 0 con memcpy, con un bucle... Porque no todos los compilador al reservar memoria la "limpian" en C++.

[SiR.CARAJ0DIDA]

bue.. gracias pero ni idea de como usar memcpy

[MaxExtreme]

memset quería decir (lapsus). Se usa para rellenar grandes zonas de memoria con un valor determinado.

Código:

#include <cstring>
void *memset(void *s, int c, size_t n);

Como veo que habías usado strlen que es de C, no te importará ya usar memset.

s es el puntero a la dirección a rellenar, buffer.
c es el caracter que se escribirá, 0.
n es el número de caracteres que se escribirán.

Devuelve el puntero.

En teoría solo deberías inicializar el último, porque los demás los rellenas y es mucho más rápido:

buffer[size] = '\0';

[SiR.CARAJ0DIDA]

lo solucione usando la funcion get() en vez de read()