Программа множественности CS50 pset3 выдает ошибку для функции print_winner (не распечатали обоих победителей выборов) - PullRequest
Новая
       12

Программа множественности CS50 pset3 выдает ошибку для функции print_winner (не распечатали обоих победителей выборов)

0 голосов
/ 27 мая 2020
• 1000 в случае at ie (и я тестировал его несколько раз для различных сценариев ios) эта ошибка все еще сохраняется, и я не уверен, почему. Может ли кто-нибудь изучить мой код ниже и уточнить, что с ним не так? (Прошу прощения, если это очень беспорядочно / неэффективно - мне потребовалось время, чтобы написать его, потому что мне потребовалось время, чтобы выяснить, как распечатать несколько победителей): 
void print_winner(void)
{
    int j = 0;
    int max = candidates[0].votes;
    int competitor;
    int competitor2;
    string winner = candidates[0].name;

    // loop over all candidates + determine candidate with most votes (max)
    // + name candidate with most votes (winner)
    for (j = 1; j < candidate_count; j++)
    {
        competitor = candidates[j].votes;
        if (max < competitor)
        {
            max = competitor;
            winner = candidates[j].name;
        }
    }

    // loop over candidates again to determine if multiple winners + print winner/winners
    for (int f = 0; f < candidate_count; f++)
    {
        competitor2 = candidates[f].votes;
        if (max == competitor2 && candidates[f].name != winner)
        {
            for (int i = 0; i < candidate_count; i++)
            {
                printf("%s\n", candidates[i].name);
            }
            return;
        }
    }
    printf("%s\n", winner);
    return;
}

EDIT : Как указал DinoCoderSaurus, мой i l oop был проблематичным c. Я понял, что это даже не нужно, так как мой f l oop уже выполняет ту же функцию. Ниже я удалил i l oop, немного изменил f l oop и переместил строку printf("%s\n", winner); перед f l oop, поэтому, если есть несколько победителей, они печатаются в правильном порядке: 

void print_winner(void)
{
    int j = 0;
    int max = candidates[0].votes;
    int competitor;
    int competitor2;
    string winner = candidates[0].name;

    // loop over all candidates + determine candidate with most votes (max)
    // + name candidate with most votes (winner)
    for (j = 1; j < candidate_count; j++)
    {
        competitor = candidates[j].votes;
        if (max < competitor)
        {
            max = competitor;
            winner = candidates[j].name;
        }
    }

    printf("%s\n", winner);
    // loop over candidates again to determine if multiple winners + print winner/winners
    for (int f = 0; f < candidate_count; f++)
    {
        competitor2 = candidates[f].votes;
        if (max == competitor2 && candidates[f].name != winner)
        {
            printf("%s\n", candidates[f].name);
        }
    }
    return;
}

1 Ответ

1 голос
/ 27 мая 2020

Попробуйте следующие выборы: 3 кандидата: a, b, c. Пять голосующих, проголосовавших: a, a, b, b, c.

Каков ожидаемый результат? Какой вывод выдает программа?

Проблема в i l oop. Если будет найден «второй» победитель, имена каких кандидатов будут напечатаны?

...