assignment from hell !!

[pdring]

Hi everybody,
I have compleated an assignment in C for college and was very pleased with myself until I saw that extra marks could be obtained by modifiying the working program to meet a new criteria.
at present the program excepts an integer as 'amount of contracts sold' for each of 5 employees, the employees are defined in a structure with all the information set out in the assignment brief.
the program then (via 'for loops'), calculates pay etc. acording to amount of contracts sold for each employee and displays it in nice little tables !
the new criteria however is for each employee to enter a tarrif code for each contract sold and display how many contracts were sold under each tarrif code, as well as all the info that the original program displays.
to exit from one employee to the next, the values 9 (for tarrif code) and 0 (for amount of contracts) must be entered.
I have tried to achieve this by setting up a for loop with a nested while and a nested if loop, this works but sends very large and very wrong data to the display, if anyone is interested in this problem, I could post the 'problem' loop for you to see, if thats ok let me know.
Thanks
Phil

[weaknessforcats]

Well, let's see the code.

[pdring]

Well, let's see the code.

Thanks, I have been trying to fix my problem and got a little further, but now the values are not showing at all !!

 for (count = 0;count <noEmployees; count++)
          do
          {
         printf ("%s\nPlease enter the tarrif code:\t",employee[count].name);
         scanf("%d",&tarrifCode);
         printf ("\nnumber of contracts negotiated for this tarrif:\t");
         scanf("%d", &sales);
         printf ("--------------------------------------------------------------\n");
          }
           while (count<noEmployees && tarrifCode!= 9 && noSales[count] != 0);
 
        for (count = 0;count <noEmployees; count++)
           {
           employee[count].totalContracts += sales;
           if (tarrifCode>0 && tarrifCode<6)
           {
             noSales[tarrifCode - 1] += sales;
            }
            }

[weaknessforcats]

You may have a data design problem.

You say your employees are in a struct. Where are the contracts???

Wouldn't the contracts also be in a struct??

struct Contracts
{
      int TheTariff;
      int theContractNumber;
 
};
 

Then your Employee has an array of Contracts:

struct Employee
{
     Contracts theContracts[5];
     unigned int numSold;
     char Name[20];
     //etc...
};
 

As you sell contracts you fill up the Contracts array. The numSold keeps track of how many contracts are sold. At 5 the array is full and no more contracts can be sold.

Now all you need to is take an employee and loop through the contracts array up to numSold and you have all of your information.

[pdring]

Thankyou very much for your help, I have a working program at last!!!
(why do things seem obvious when you have the answer !)
My course is moving on to C++ next, so I guess I will be scanning the archives of this forum pretty soon.
Thanks again,
Phil Cornwall (England).

You may have a data design problem.

You say your employees are in a struct. Where are the contracts???

Wouldn't the contracts also be in a struct??

struct Contracts
{
      int TheTariff;
      int theContractNumber;
 
};
 

Then your Employee has an array of Contracts:

struct Employee
{
     Contracts theContracts[5];
     unigned int numSold;
     char Name[20];
     //etc...
};
 

As you sell contracts you fill up the Contracts array. The numSold keeps track of how many contracts are sold. At 5 the array is full and no more contracts can be sold.

Now all you need to is take an employee and loop through the contracts array up to numSold and you have all of your information.