Как в Си сделать fopen() с переменной???

Question

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

Мне нужно записать fopen() с переменной, в С++ я делал так:

char *command = "echo Hello, world!";
fopen(command.c_str(), 'r');

В Си у меня с этим возникли сложности, прошу не судите строго за "глупый" вопрос. Если говорить подробнее - у меня есть переменная в Си

char *disks[DISK_ARRAY_LEN] = {"C:", "D:", "/sys/"};

И мне нужно с помощью цикла каждую строку из этого массива применять к функции fopen() , код примерно выглядит так:

char *disks[DISK_ARRAY_LEN] = {"C:", "D:", "/sys/"};
	FILE *CHECK_DISK;
	for (int i = 0; i < DISK_ARRAY_LEN; i++) {
		CHECK_DISK = fopen(disks[i], "r"); // Тут и выходит ошибка
		if (CHECK_DISK == NULL) {
			printf("Disk %s is not found", disks[i]);
		}
		else {
			printf("Disk %s if found!", disks[i]);
		}
		fclose(CHECK_DISK);
	}
	return 0;

Буду рад любым исправлениям кода, ибо мне кажется что я что то упустил.
Вывод ошибки: "Ошибка сегментирования"

Comments

[Люда MARMALADKA]

Всем привет! Заранее прошу изменения за то что я написал в тегах не Си а С++, ибо я не пойму, блять,

это баг там надо написать "C" и нажать табуляцию

[Сергей Горностаев]

Люда MARMALADKA, или enter.

[Сергей Горностаев]

Добавьте в вопрос текст ошибки.

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

Сергей Горностаев, "Ошибка сегментирования"

[Roman]

SaddledSharp, DISK_ARRAY_LEN больше 3?

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

Roman, Нет, вот фулл код:

#include <stdio.h>
#define DISK_ARRAY_LEN 3

int main() {
	char *disks[DISK_ARRAY_LEN] = {"C:", "D:", "/sys/"};
	FILE *CHECK_DISK;
	for (int i = 0; i < DISK_ARRAY_LEN; i++) {
		CHECK_DISK = fopen(disks[i], "r");
		if (CHECK_DISK == NULL) {
			printf("Disk %s is not found", disks[i]);
		}
		else {
			printf("Disk %s if found!", disks[i]);
		}
		fclose(CHECK_DISK);
	}
	fclose(CHECK_DISK);
	return 0;
}

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

SaddledSharp, какие данные имеют disks и i в отладчике в момент ошибки сегментирования?

[CityCat4]

SaddledSharp, Если поток не открылся - нечего закрывать. Вангую, что программа падает на fclose(), который вызывается без fopen() (когда СHECK_DISK == NULL). fclose() надо делать только после успешного fopen().

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

Евгений Шатунов, Нету отладчика. У меня Линукс, пишу на Sublime Text 3

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

CityCat4, Можно примерный код решения?

[Roman]

CityCat4, SaddledSharp,
ну и

int main() {
  char *disks[DISK_ARRAY_LEN] = {"C:", "D:", "/sys/"};
  FILE *CHECK_DISK;
  for (int i = 0; i < DISK_ARRAY_LEN; i++) {
    CHECK_DISK = fopen(disks[i], "r");
    if (CHECK_DISK == NULL) {
      printf("Disk %s is not found", disks[i]);
    }
    else {
      printf("Disk %s if found!", disks[i]);
    }
    fclose(CHECK_DISK); // Ладно возможно NULL
  }
  fclose(CHECK_DISK); // А тут что?
  return 0;
}

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

Roman, В конце просто после окончания закрыл файл если он остался... Убрал

[Roman]

SaddledSharp,

for(int i = 0; i < DISK_ARRAY_LEN; i++) 
{
  if((CHECK_DISK = fopen(disks[i], "r")) == NULL)
  {
    printf("Disk %s not found", disks[i]);
  }
  else
  {
    printf("Disk %s found!", disks[i]);
    fclose(CHECK_DISK);
  }
}

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

Roman, Вот оно как, спасибо!

[CityCat4]

SaddledSharp, man gdb. gdb знать надо

И - в CHECK_DISK, если она не проинициализирована - а она в коде не инитится в расчете на то, что ее проинитит fopen() - там не NULL, а мусор. fclose() ищет там поля буферов, пытается их освободить и закономерно падает :)

Answer 1

[nic777]

первые строчки работают??
если нужно отправить строку в консоль, используйте хэндл stdout, он открыт по дефлоту, можно сразу fputs/fprintf

Comments

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

Уже решил. Была "ошибка сегментирования", добрый человек помог исправить код