C + программа с функциями ... Пожалуйста, помогите мне - PullRequest
1 голос
/ 17 декабря 2011

У меня проблемы с двумя функциями. Вот инструкции к проекту:

Назначение: Напишите программу, которая отслеживает количество тараканов в двух соседних домах в течение нескольких недель. Количество тараканов в домах будет определяться следующим образом:

  1. Начальное число тараканов для каждого дома - это случайное число от 10 до 100.
  2. Каждую неделю количество тараканов увеличивается на 30%.
  3. Два дома имеют общую стену, через которую тараканы могут мигрировать из одного в другой. В данную неделю, если в одном доме больше тараканов, чем в другом, тараканы из дома с более высоким населением мигрируют в дом с меньшим населением. В частности, мигрирует 30% разницы (округлено) в популяции.
  4. Каждые четыре недели один из домов посещается истребителем, в результате чего количество тараканов в этом доме уменьшается (на 90%).

Вот мой код:

#include <iostream>
#include <cmath>
using namespace std;

int house, increase, roaches, moreRoaches, fewerRoaches, filthyBeasts, change; // My variables for my four functions
int initialCount(int house);
int weeklyIncrease(int increase);
double roachesMigration(int moreRoaches, int fewerRoaches, int change);
int exterminationTime (int filthyBeasts);
// My four function prototypes 

int main()
{
    int houseA, houseB;

    houseA = initialCount(houseA); //Initializing the initial count of House A.
    houseB = initialCount(houseB); //Initializing the initial count of House B.

    int week = 0;
    for (week = 0; week < 11; week++) // My for loop iterating up to 11 weeks.
    {
        houseA = weeklyIncrease(houseA);
        houseB = weeklyIncrease(houseB);

        cout << "For week " << week << ", the total number of roaches in House A is " << houseA << endl;
        cout << "For week " << week << ", the total number of roaches in House B is " << houseB << endl;

        if((houseA > houseB)) // Migration option 1
        {
            roachesMigration(moreRoaches, fewerRoaches, change);
        }
        else if((houseB > houseA)) // Migration option 2
        {
            roachesMigration(moreRoaches, fewerRoaches, change);
        }


        if ((week + 1) % 4 == 0) // It's extermination time!
        {
            if ((rand() % 2) == 0) // Get a random number between 0 and 1.
            {
                houseB = exterminationTime(houseB);
            }
            else
            {
                houseA = exterminationTime(houseA);                   
            }
        }
    }

    return 0;
}

int initialCount(int house) // Initializing both houses to random numbers between 10 and 100.
{
    int num;
    num = (rand() % 91) + 10;
    return num;
}

int weeklyIncrease(int increaseHouses) // Increasing the roaches in both houses by 30% weekly.
{
    int increase = 0;
    increase = (increaseHouses * .3) + increaseHouses;
    return increase;
}

double roachesMigration(int moreRoaches, int fewerRoaches, int change)
{
    more -= change;
    fewer += change;   
    change = ((more - fewer) * .3);
    return change; 
}


int exterminationTime(int filthyBeasts) // Getting rid of the filthy little beasts!
{
    filthyBeasts = (filthyBeasts * .1);
    return filthyBeasts;
}

Проблемы связаны с функциями миграции и уничтожения. Мой код работает нормально, но на 4 и 8 неделе случайно выбранный дом должен быть уничтожен, а количество тараканов в этом доме должно быть на 90% меньше, чем на предыдущей неделе. Как вы думаете, ребята, я должен сделать, чтобы исправить эти проблемы? Мне действительно нужна вся помощь, которую я могу получить!

Ответы [ 4 ]

2 голосов
/ 17 декабря 2011

Переменная change не является глобальной переменной, но появляется внутри main (поэтому она не имеет значения внутри main).

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

Попросите ваш компилятор выдать вам все предупреждения и предоставить отладочную информацию (g++ -Wall -g в Linux). Улучшайте код, пока не получите предупреждения.

Научитесь использовать отладчик (например, gdb в Linux).

Веселитесь.

2 голосов
/ 17 декабря 2011

Относительно этой строки:

roachesMigration(change);

change не объявлено в вашей функции main, следовательно, ошибка.Также функция roachesMigration ожидает 3 параметра, а не 1.

1 голос
/ 17 декабря 2011

Один из возможных способов управления миграцией - это псевдокод:

// compute size of migration
count = migration(houseA, houseB)

if (houseA < houseB)
    add count to houseA
    subtract count from houseB
else
    add count to houseB
    subtract count from houseA
1 голос
/ 17 декабря 2011

В зависимости от инструктора, вы получите ноль баллов за этот код, даже если вы можете заставить его работать отлично! Это потому, что вы не использовали какой-либо объектно-ориентированный дизайн при создании кода. В C ++ это означает, что классы.

Какой объект вам нужен для этой проблемы. Дом!

Каким атрибутом должен обладать ваш дом? Тараканы!

Так что-то вроде этого:

class cHouse
{

 int MyRoachCount;

 ...

};

Если вы начнете с нуля, вот так, вы обнаружите, что все начинает аккуратно становиться на свои места.

...