Сохранение структуры в файл, а как быть с переменной?

[Alex11223]

Причем тут два цикла? В коде выше просто пример функции (ну или без функции, не важно) для добавления одного элемента в массив.
То есть добавлять много элементов как-то так:

while (есть что добавлять)
{
    data = получить_данные();
    ДобавитьЭлемент(data);
}

И такой вопрос, как я понял, sizeof(base)/sizeof(base[0]) не работает с массивами на которые выделена память... Подскажите, как тогда подсчитать кол-во элементов ?

Никак. Хранить кол-во элементов в переменной (считать по мере добавления, как в предыдущем коде).

[makar3000]

разобрался, но я не делал функцией, а напрямую его внедрил в цикл. Только вопрос. Как теперь мне массивом воспользоваться в другой функции? Пишет что она занята...

Run-Time Check Failure #3 - The variable 'base' is being used without being defined.

int ramUse = 0;
int baseSize = 5;
baseRecord *base = (baseRecord*)malloc(baseSize*sizeof(baseRecord)); // выделим памяти для пяти элементов

while( ! feof(fileDB) )
		{
			char *pos;
			if( ramUse == baseSize ) // Если у нас кол-во записей совпадает с базовым размером то расширяем нашу память
			{
				baseSize *= 2; // В два раза
				base = (baseRecord*)realloc(base, baseSize * sizeof(baseRecord)); // Перераспределим память
			}
			fscanf(fileDB, "%d|%d|", &base[ramUse].id, &base[ramUse].productID);
			fgets(base[ramUse].name, MAX_NAME, fileDB);
			pos = strrchr(base[ramUse].name, '\n');
			if(pos)
				*pos = 0;
			ramUse++;
		}

[Alex11223]

"is being used without being defined" вообще-то означает ровно наоборот, что переменная с таким именем не найдена.
Ну если это не глобальная переменная, то можно просто передать в функцию параметр. Массив в С это просто указатель.

И по-моему тут baseSize и ramUse тут надо поменять местами чтоб названия соответствовали назначению.

[makar3000]

Так я ведь после структуры указал переменную (не в функции) base , собственно она глобальная. Так почему я ее не могу вызвать из другой функции?

[Alex11223]

Чего за "переменная после структуры"? Строка номер 3 из кода выше где находится?

[makar3000]

Тот код , он находится в функции createDB

typedef struct {
	int id;
	int productID;
	char name[MAX_NAME];
} baseRecord;

baseRecord *base;

void createDB()...

[Alex11223]

Ну значит должно работать и дело в чем-то другом. В createDB вы ж используете его.

Но если предыдущий код прям так и есть в функции, то вы там создали локальную переменную, а глобальную не меняли.

[makar3000]

Разобрался, не в том файле огласил глобальный массив

Подскажите пожалуйста, проблема с зоной видимости переменной base...

У меня три файла:
function.h:

#include <stdlib.h>
#include <stdio.h>
#include <locale.h>

#define MAX_NAME 100
#define MAX_DB 100

typedef struct {
	int id;
	int productID;
	char name[MAX_NAME];
} baseRecord;

#ifndef __SECOND_FILE
#define __SECOND_FILE
	extern baseRecord *base;

	extern const char* db;
	extern int countDB;
	extern void createDB();
#endif

function.c:

#include "function.h"

baseRecord *base;

const char* db = "database.txt";
int countDB = 0;

void createDB() {/*=====*/}

main.c:

#include "function.h"
int main(void)
{
        setlocale( LC_ALL, "Russian" );
	createDB(); // Откроем/создадим базу
        showTable();

        system("pause");
	return 0;
}

int showTable(void)
{
        int i;

	if( ! countDB )
	{
		printf("| %-62s |\n", "База данных пуста");
	}
	else
	{
		for( i = 0; i < countDB; i++ )
		{
			printf("| %3d | %-13d | %-17s |\n", base[i].id, base[i].productID, base[i].name);
		}
	}
}

При первом запуске программы и гетинга данных с файла, таблица заполняется, а вот после расширения памяти и добавления нового элемента и последующего повторного запуска showTable показывает кракозябкры аля: -1232131|-12314329843854|-2313123

[Alex11223]

А чего за ifndef __SECOND_FILE в середине файла? include guard'ы обычно на все содержимое хедер файла делают, а не на половину.

И для функций можно не писать extern.

Ну а проблема ж скорее всего в createDB, так что непонятно зачем он не показан. И где тут "добавление нового элемента"?


Вообще я бы не делал глобальные переменные, а создавал/объявлял бы их в main'е и передавал параметрами в функции.
А еще лучше структура типа такой

typedef struct {
	int id;
	int productID;
	char name[MAX_NAME];
} DatabaseRecord;

typedef struct {
    int recordsCount;
    DatabaseRecord* records;
} Database;

int main(void)
{
    const char* dbPath = "database.txt";
    
    Database db;
    
    initDb(&db);
    
    readDbFile(dbPath, &db);
    
    printTable(&db);
    
    return 0;
}

void initDb(Database* db)
{
    db->recordsCount = 0;
    db->records = 0;
}

[makar3000]

void addDB()
{
	int productID;
	char name[MAX_NAME];

	printf("\n> Инвентарный номер: ");
	scanf("%d", &productID);
	printf("\n> Название товара: ");
	getRusText(name);

	if( productID < 0 )
	{
		system("cls");
		showTable();
		printf("Вы ввели неверные данные. Попробуйте еще раз\n");
		addDB();
		return 0;
	}
	
	if( countDB > 0 )
		countDB++;
	else
		countDB = 0;

	base = (baseRecord*)realloc(base, sizeof(baseRecord)*(countDB+1)); 

	base[countDB].id = countDB;
	base[countDB].productID = productID;
	strcpy(base[countDB].name, name);

	system("cls");
	showTable();
}