Как записать многострочный текст в файл?

Question

[Молзв Реках]

Всем привет! У меня есть переменная char BUF[512] в которой хранится вывод из консоли от команды "lspci", КОД:

FILE *INFO_PC;
	char BUF[512];
	INFO_PC = popen("lspci", "r");
	if (!INFO_PC) {
		cout << "ERROR!" << endl;
	}
	else {
		while(fgets(BUF, sizeof(BUF), INFO_PC)){
			puts(BUF);
		}
	}

Мне нужно эту переменную BUF записать в текстовый файл. Но когда я пишу:

FILE *INFO_PC;
	char BUF[512];
	INFO_PC = popen("lspci", "r");
	if (!INFO_PC) {
		cout << "ERROR!" << endl;
	}
	else {
		while(fgets(BUF, sizeof(BUF), INFO_PC)){
			puts(BUF);
		}
	}
	FILE *FILE_INFO_PC = fopen("desktop.cfg", "w");
	if (FILE_INFO_PC) {
		bool write = fputs(BUF, FILE_INFO_PC);
		if (write) {
			cout << "Yes!" << endl;
		}
		else {
			cout << "ERROR!" << endl;
		}
	}
	else {
		cout << "ERROR!" << endl;
	}
	fclose(FILE_INFO_PC);

То после выполнения в файл записывается только первая строка, а именно -

03:00.0 3D controller: NVIDIA Corporation GK107M [GeForce 810M] (rev a1)

Сам же вывод содержится из 5 строк, но записывается только одна. Как правильно написать?

Answer 1

[Алексей Сергей]

while(fgets(BUF, sizeof(BUF), INFO_PC)){
   puts(BUF);
}

Этот цикл построчно вычитывает выхлоп от `popen(...)`. Не удивительно, что в BUF остается лишь одна последняя строка.

И еще вижу возможную утечку в случае ошибки popen(...): `char BUF[512]` ничем не инициализирован и все, что лежит на стеке может улететь в desktop.cfg.

Comments

[Молзв Реках]

Можете помочь с кодом? Пожалуйста.

[Евгений Шатунов]

Алексей Сергей

И еще вижу возможную утечку

В этом месте нет утечки. Там ошибка инициализации переменной.

[Молзв Реках]

Блин, кто нибудь может помочь? Я не знаю даже что и как гуглить, как мне правильно написать код? Не понимаю...

[Алексей Сергей]

Евгений Шатунов, утечка данных же. Переменная не инициализирована, а значит и нет гарантии, что среди запрошенных 512 байт будет нуль терминатор. В результате bool write = fputs(BUF, FILE_INFO_PC); потенциально может записать в файл намного больше, чем 512 байт. Возможно там будет мусор, а возможно и какая-нибудь важная инфа. Отсюда же и появились "безопасные" функции вроде strcpy(...) -> strcpy_s(...), и т.д.

SaddledSharp, так построчно и пишите в файл:
- открыли popen()
- открыли "desktop.cfg" на запись.
- прочитали строку из дескриптора popen()
- записали в файл desktop.cfg
- повторять пока не закончаться данные в popen()

[Молзв Реках]

Алексей Сергей, Это я всё отлично понял, здесь важное то - что я не пойму как это сделать. Я вчера сильно поздно уснул и сегодня не выспался, а программу ну прям очень срочно тянет написать, попросил же, помогите с кодом. Или вам легче намёками писать?

[Евгений Шатунов]

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

[Алексей Сергей]

Евгений Шатунов, в своем ответе я упомянул и причину бага, и возможные последствия.

ИМХО, "неинициализированная переменная" -- слишком широкое понятие, которое в т.ч. может использоваться и во благо -- например, для оптимизации производительности кода. Поэтому и решил сделать уточнение. Повторюсь -- это лишь мое мнение.

Всем добра и программ без багов!

[Алексей Сергей]

SaddledSharp, например можно так. Но учитывайте, что я лишь переделал вариант исходников из топика.
Есть множество возможностей для улучшения и вообще из C++ тут только "std::cout".

#include <stdio.h>
#include <iostream>

using namespace std;

int main() {
  FILE *INFO_PC = nullptr;
  char BUF[512] = { 0 };

  INFO_PC = popen("lspci", "r");
  if (!INFO_PC) {
      cout << "ERROR!" << endl;
      exit(1);
  }

  FILE *FILE_INFO_PC = fopen("desktop.cfg", "w");
  if (!FILE_INFO_PC) {
      cout << "ERROR!" << endl;
      exit(1);
  }

  while (fgets(BUF, sizeof(BUF), INFO_PC)) {
      puts(BUF);
      bool write = fputs(BUF, FILE_INFO_PC);
      if (!write) {
          cout << "Failure on attempt to write data into file!" << endl;
          exit(1);
      }
  }

  fclose(FILE_INFO_PC);
}

[Молзв Реках]

Алексей Сергей, Наконец то! Спасибо большое, буду изучать и искать дальнейшие пути с примеров вашего кода.

Answer 2

[Roman]

SaddledSharp, тегом не ошиблись?

++

под Win но разница только в путях и _popen -> popen _pclose -> pclose

#include<iostream>
#include<fstream>
#include<algorithm> // copy
#include<string>
#include<vector>
#include<array>
#include<memory> // unique_ptr
#include<cstdio>
#include<iterator>

using namespace std;
using pipe = unique_ptr<FILE, decltype(&_pclose)>;

auto getCmdOutput = [](const string& command)
{
  pipe fp{_popen(command.c_str(), "r"), &_pclose};
  vector<string> result;
  array<char, 512> buf;
  if(fp)
  {
    while(fgets(buf.data(), buf.size(), fp.get()))
    {
      result.emplace_back(buf.data());
      buf.fill(0);
    }
  }
  return result;
};

auto writeToFile = [](const string& fname, const vector<string>& info)
{
  ofstream ofs(fname);
  if(info.empty() || !ofs)
  {
    // put error ... 
    return false;
  } 
  copy(info.cbegin(), info.cend(), ostream_iterator<string>{ofs});
  return true;
};

int main()
{
  if(writeToFile("D:\\ipconfig.cfg", getCmdOutput("ipconfig")))
  {
    cout << "Success!" << endl;
  }
  else
  {
    cout << "Failure!" << endl;
  }
}