Блог

Проблема в том, что в вашем заголовочном файле есть множество глобальных переменных. При компиляции каждого .c файла, включающего этот заголовок, результирующий объектный файл имеет свое собственное определение этих переменных. Когда вы пытаетесь связать объектные файлы в один исполняемый файл, все эти определения переменных конфликтуют друг с другом.

Правильный ответ — не использовать глобальные переменные-функции должны взаимодействовать через параметры и возвращаемые значения, а не через глобальные. Использование глобалов уничтожает возможность модуляции кода, как вы пытаетесь это сделать.

Просмотрите все функции вашей программы и задайте два вопроса:

1. Какая информация необходима этой функции для выполнения всех ее задач? — ответом на этот вопрос будут ваши входные аргументы.
2. Какие данные или изменение состояния ожидает вызывающий абонент от этой функции? — ответом на этот вопрос будет ваше возвращаемое значение и/или любые выходные аргументы.

Давайте возьмем customer функцию в качестве примера. Поскольку вы не опубликовали определение функции, мне приходится гадать, но я предполагаю, что она отображает данные о клиентах с помощью строки формата. Я предполагаю, что он просто записывает данные в стандартный вывод, но было бы удобно также записать их в файл. Итак, для customer того , чтобы мы предположили, что ответы на вопрос 1 являются:

• Пример того, как struct Customer ;
• Выходной поток;

и ответ на вопрос 2 — «ничего», так как это строго функция вывода.

Таким образом, объявление customer в заголовочном файле становится:

 void customer( struct Customer, FILE * );
 

и его определение в исходном файле становится

 void customer( struct Customer c, FILE *stream )
{
  const char* customer_format = "ntCustomerID:t%sntCustomerName:t%sntMobileNumber:t%sn";

  fprintf( stream, customer_format, c.customerid, c.customername, c.mobno );
}
 

Где-то в вашем main файле у вас будет экземпляр struct Customer defined, поэтому вы назовете его как:

 int main( void )
{
  ...
  struct Customer c;
  ... // assume data is read into c here
  customer( c, stdout );
  ...
}
 

Точно так же мы можем посмотреть на addcus . Ответы на вопрос 1 будут следующими

• поток, из которого считываются данные о клиентах

и ответы на вопрос 2 будут

• новые данные о клиентах
• указание на то, была ли операция успешной или нет

Чтобы указать, была ли операция ввода успешной, у нас будет функция возврата true или false . Функции необходимо место для хранения входных данных — вызывающий объект предоставит это, передав указатель на struct Customer объект. Таким образом, наше объявление в заголовочном файле будет

 bool addcus( struct Customer *, FILE * );
 

и определение в файле .c будет

 bool addcus( struct Customer *c, FILE *stream )
{
  if ( stream == stdin )
    fprintf( "Enter customer ID: " );
  if ( !fgets( c->customerid, sizeof c->customerid, stream ) )
    // input error - for now, we'll just return false
    return false;

  // repeat for customer name and mobile number
  
  // if everything is read successfully, return true
  return true;
}
 

Так что теперь мы можем немного обновить нашу main функцию:

 int main( void )
{
  ...
  struct Customer c;
  if ( addcus( amp;c, stdin ) )
    customer( c, stdout );
  ...
}
 

Вам нужно сделать это для каждой функции в вашей программе.

Это всего лишь примеры, и они могут не отражать то, что вы собираетесь делать в своем коде. Но это должно проиллюстрировать шаги, которые вам необходимо предпринять для модульной обработки вашего кода.

То, что вы хотите, — это что-то вроде этого:

 #ifndef MAINPROJECTNIKLASLOREK_H_
#define MAINPROJECTNIKLASLOREK_H_

// Anlegen eines const char pointer um das Format des 
// Medizinstructs festzulegen und später bei printf Ausgaben
// direkt anzuwenden
extern const char* medicine_format;

//Anlegen eines Structs für die Medizin
struct Medicine {
    int quantity;
    float price;
    char expdate[15];
    char company[20];
    char supplier[30];
    char mediname[20];
    char info[1000];
    char mediid [30];
};
extern const char* supplier_format;

// Anlegen eines Structs für die Zulieferer
struct Supplier{
    char supplierid[30];
    char suppliername[30];
    char mobno[15];
    char company[50];
};
extern const char* customer_format;

// Anlegen eines Structs für die Kunden
struct Customer{
    char customerid[30];
    char customername[30];
    char mobno[15];
};

// Struct um Kassenzettel einzuspeichern
struct bill {
    char billid[30];
    char mediname[30];
    int medi_qty;
    float price;
    float total;
    float newtotal;
    float temptotal;
};

// Globale Variablen für Coupon und Prozent. Diese werden von
// der Main Funktion überschrieben, wenn sie benutzt werden.
// Sie werden nur! in der Sellmed Funktion angewendet, um
// einen Discount auf einen Medizin Verkauf anzuwenden.
#define SIZE 20
char coup[SIZE] = { 'n', 'o', 'c', 'o', 'u', 'p' };
float dis = 1.00;

// Main menu Funktion:
extern void mainmenu(*actual parameters go here*);

// Kunde Funktionen Prototypen:
extern void customer(*actual parameters go here*);
extern void addcus(*actual parameters go here*);
extern void cuslist(*actual parameters go here*);
extern void searchcus(*actual parameters go here*);
extern void editcus(*actual parameters go here*);
extern void deletecus(*actual parameters go here*);

// Zulieferer Funktionen Prototypen:
extern void supplier(*actual parameters go here*);
extern void addsup(*actual parameters go here*);
extern void suplist(*actual parameters go here*);
extern void searchsup(*actual parameters go here*);
extern void editsup(*actual parameters go here*);
extern void deletesup(*actual parameters go here*);

// Medizin Funktionen Prototypen:
extern void medicine(*actual parameters go here*);
extern void leave(*actual parameters go here*);
extern void addmed(*actual parameters go here*);
extern void sellmed(*actual parameters go here*);
extern void listing(*actual parameters go here*);
extern void searchmed(*actual parameters go here*);
extern void editmed(*actual parameters go here*);
extern void deletemed(*actual parameters go here*);
extern void pharwelc(*actual parameters go here*);

// Bestandsfunktionen Prototypen:
extern void searchstock(*actual parameters go here*);
extern void stock(*actual parameters go here*);

#endif /* MAINPROJECTNIKLASLOREK_H_ */
 

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

 const char* medicine_format = "ntMediID:%sn"
           "tMediName:%sntQuantity:%dntPrice:%.2fn"
           "tExp.Date:%sntCompany:%sntSupplier:%sn"
           "tInfo:%sn";
 

Как указано @user3386109 выше, вы также должны включить полный список параметров с каждым прототипом функции.