Need Help Finishing a Program Using Array of Structures

[radskater]

I am needing to finish this program that has an array of twelve structures to hold the travel information for the whole year. I need to prompt the user to enter data for each month. Once all the data is entered, I need to calculate and output the monthly number of landing planes, the average number of departing plans, the total of landing and departing planes for the year, and the greatest and least number of planes landed on any one day (and which month it occurred in).

I need help finishing the program to type in the missing code that I need to make it work. Any help will be greatly appreciated!

#include <iostream>


using namespace std;


struct MthlyInfo {
    int totNumLand;
    int totNumDep;
    int greNumLand;
    int leaNumLand;
};


MthlyInfo yearinfo[12];


int main()
{
    int mthlyPlnLand;
    int mthlyPlnDep;
    int grtNumPlnLand;
    int lowNumPlnLand;


    for (index = 0; index < yearinfo[index]; index++) {
        cout << "How many planes have landed this month? " << endl;
        cin >> yearinfo[index].mthlyPlnLand;


        cout << "How many planes have departed this month? " << endl;
        cin >> mthlyPlnDep[index].mthlyPlnDept;
    }


    cout << "The greatest number of plans that landed in a given day  that month are: "






    return 0;
}

[laserlight]

Personally, I think your preoccupation with trying to abbreviate the names is unhealthy. Yes, it is good to choose concise names, and we do use abbreviations, but unusual abbreviations tend to make it harder to read and possibly harder to type (although autocomplete can help... but then it can help with slightly longer but non-abbreviated names too). So, I would suggest:

struct MonthlyInfo {
    int numLand;
    int numDepart;
    int maxNumLand;
    int minNumLand;
};

You should declare yearinfo as a variable local to the main function, and then get rid of those four local variables in main that aren't useful as the information you want would already be stored in yearinfo.

the greatest and least number of planes landed on any one day (and which month it occurred in)

It seems to me that you don't have enough information for this: you're collecting data on a monthly basis, so you cannot know how many planes landed of any given day. You can only compute say, the mean number of planes that landed per day on any given month, but that isn't going to give you "the greatest and least number", unless you're trying to find out which month had the greatest number of planes land per day on average, and which month had the least number of planes land per day on average, in which case you don't need these as member variables of the struct.

[radskater]

Personally, I think your preoccupation with trying to abbreviate the names is unhealthy. Yes, it is good to choose concise names, and we do use abbreviations, but unusual abbreviations tend to make it harder to read and possibly harder to type (although autocomplete can help... but then it can help with slightly longer but non-abbreviated names too). So, I would suggest:

struct MonthlyInfo {
    int numLand;
    int numDepart;
    int maxNumLand;
    int minNumLand;
};

You should declare yearinfo as a variable local to the main function, and then get rid of those four local variables in main that aren't useful as the information you want would already be stored in yearinfo.


It seems to me that you don't have enough information for this: you're collecting data on a monthly basis, so you cannot know how many planes landed of any given day. You can only compute say, the mean number of planes that landed per day on any given month, but that isn't going to give you "the greatest and least number", unless you're trying to find out which month had the greatest number of planes land per day on average, and which month had the least number of planes land per day on average, in which case you don't need these as member variables of the struct.

Thank you @laserlight !