Функция read() в С++

char s[2000] не инициализируется нулями.

Чуть подробней можно?...

> Откуда может быть мусор?

file.read(reinterpret_cast<char*>(s),sizeof(s));

sizeof(s)

// К.О.

s[1999] = 0; /* те, отмечаем конец строки */

cout << s << endl; /* и уже потом её выводим, дабы не печатать что попало из heap */

Ну и если файлик будет >2000б тоже нехорошо.

> s[1999] = 0; /* те, отмечаем конец строки */

Нет. Файл не обязательно длиной 1999.

>Чуть подробней можно?...

Конечно!
http://lib.ru/CTOTOR/kernigan.txt

Перечитай потс.

> Перечитай потс.

Да там и нечего читать. Должно быть s[размер_файла] = 0; или все занулить в начале.

Нуль в конце строки добавил: s[strlen(s)] = 0; — ничего не изменилось...

> Нуль в конце строки добавил: s[strlen(s)] = 0; — ничего не изменилось...

Потому что strlen(s) != последний в массиве.

Pardon! strlen() выдает по определению реальное число символов в массиве.... без последнего нуля.

>или все занулить в начале
Лишняя работа.

Должно быть s[размер_файла]

Да, но надо жэ пошагово объяснить в чём проблема (а она в отсутствии завершающего нуля).

> Pardon! strlen() выдает по определению реальное число символов в массиве.... без последнего нуля.

И ты наверняка знаешь, где находится первый нуль с ячейки &s?

Потому, что в массиве нет нулика, мой друк.

> Потому, что в массиве нет нулика, мой друк.

Я про то и говорю.

Зачем это знать?...s[strlen(s)] = 0; — и есть конец строки ,включая мой нуль.

strlen() определяет длину строки по терминирующему нулику, которого…

s[strlen(s)] = 0; - это бред сивой кобылы, как a = a; или if (a == 1) a = 1;

В таком случае как определить реальный размер считанного файла для добавления терминального нуля в массив ?

http://www.cplusplus.com/reference/iostream/istream/read/

Листай до примера.

Yes you can!

В этом смысле?

> Pardon! strlen() выдает по определению реальное число символов в массиве...

И стирает тебе носки, да.

read при чтении файла не добавляет нулевой символ в конец, ибо она не знает что такое строка и просто считывает n байт из файла в буфер. Вариантов решения тут аж два:

#1:

int main(void)
{
char s[2000];
ifstream file;
file.open("head.h");

file.seekg (0, ios::end);
int len = file.tellg();
file.seekg(0, ios::beg);

file.read(s, len);
s[len] = '\0';
cout << s << endl;
return 0;
}

int main(void)
{
char s[2000] = {0};
ifstream file;
file.open("head.h");
file.read(s,sizeof(s));
cout << s << endl;
return 0;
}

char s[1231231231] = {0};

как бы read надо делать не на всю длину s а на sizeof(s)-1, шоб хотя б один нолик то остался? А то будет как в прошлый раз;-)

Спасибо за тред, это правда смешно.

Ну и вапще, не в хипе, а на стеке. // никто даже не заметил, не торт(

Я увидел что опечатался когда уже отправил - думал никто не заметит :)

> #1:

А ситуации, когда размер файла будет больше размера буфера, конечно же быть не может... Идите и дорабатывайте до пригодного к использованию состояния.

Насколько я вижу, исходный вопрос был в том, откуда берется «мусор» при выводе прочитанной строки, а не что будет если содержимое файла не влезет в буфер. Но специально для анонимуса вот исправленный вариант:

#include <iostream>
#include <fstream>

int main (int argc, char * const argv[])
{
	std::ifstream file("head.h");
	if (file.fail())
	{
		std::cerr << "Cann't read file\n";
		return -1;
	}
	
	file.seekg (0, std::ios::end);
	int len = file.tellg();
	file.seekg(0, std::ios::beg);
	
	char *s = new char[len+1];
	file.read(s, len);
	s[len] = '\0';
	
	std::cout << s << '\n';
	
	delete [] s;
	return 0;
}