CS50 x-第3周Pset -潮汐(锁定对)

mzsu5hc0  于 2023-02-21  发布在  其他
关注(0)|答案(1)|浏览(146)

我已经和Tideman斗争了很长一段时间了,花了大约6个多小时,其中80%的时间在一个函数上。开始感到绝望。在一个名为Lockedpairs的函数中,我们需要检查我们将要锁定的新对是否会创建一个循环。如果它不会创建循环,我们应该创建一个边并锁定它。

void lock_pairs(void)
{
    // TODO
    for(int i = 0; i < pair_count; i++)
        {
            int x = pairs[i].winner;
            int y = pairs[i].loser;

            if(check(x,y) == false)
            {
                locked[x][y] = true;
            }
            else
            {
                locked[x][y] = false;
            }
        }
    return;
}

我试着使用递归,因为我认为我应该使用递归,我可以理解为什么递归被我的在线同行推荐,但是,我就是想不出如何实现它。我所做的只是为所有的锁对创建一个结构体,并将两个新的锁对输入到我的循环函数中,如果新的输入创建了一个循环,我将返回true。2我的循环逻辑是“x == lock[count]。loser && y == lock[0].赢家”,这基本上意味着如果新对的赢家等于最后一个锁定对的输家,并且新对的输家等于第一个锁定对的赢家,那么我有一个循环。

bool check ( int x, int y)
{
    typedef struct
    {
        int winner;
        int loser;
    }
    lockedpairs;
    lockedpairs lock[pair_count];

    int count = 0;

    for( int i = 0; i < candidate_count; i++)
    {
        for(int j = i; j < candidate_count; j++)
        {
            if(locked[i][j] == true)
            {
                lock[count].winner = i;
                lock[count].loser = j;
                count++;
            }
        }
    }

    if( x == lock[count].loser && y == lock[0].winner)
    {
        return true;
    }

    return false;
}

程序如下:

#include <cs50.h>
#include <stdio.h>
#include <string.h>

// Max number of candidates
#define MAX 9

// preferences[i][j] is number of voters who prefer i over j
int preferences[MAX][MAX];

// locked[i][j] means i is locked in over j
bool locked[MAX][MAX];

// Each pair has a winner, loser
typedef struct
{
    int winner;
    int loser;
}
pair;

// Array of candidates
string candidates[MAX];
pair pairs[MAX * (MAX - 1) / 2];

int pair_count;
int candidate_count;

// Function prototypes
bool vote(int rank, string name, int ranks[]);
void record_preferences(int ranks[]);
void add_pairs(void);
void sort_pairs(void);
void lock_pairs(void);
void print_winner(void);
bool check ( int x, int y);

int main(int argc, string argv[])
{
    // Check for invalid usage
    if (argc < 2)
    {
        printf("Usage: tideman [candidate ...]\n");
        return 1;
    }

    // Populate array of candidates
    candidate_count = argc - 1;
    if (candidate_count > MAX)
    {
        printf("Maximum number of candidates is %i\n", MAX);
        return 2;
    }
    for (int i = 0; i < candidate_count; i++)
    {
        candidates[i] = argv[i + 1];
    }

    // Clear graph of locked in pairs
    for (int i = 0; i < candidate_count; i++)
    {
        for (int j = 0; j < candidate_count; j++)
        {
            locked[i][j] = false;
        }
    }

    pair_count = 0;
    int voter_count = get_int("Number of voters: ");

    // Query for votes
    for (int i = 0; i < voter_count; i++)
    {
        // ranks[i] is voter's ith preference
        int ranks[candidate_count];

        // Query for each rank
        for (int j = 0; j < candidate_count; j++)
        {
            string name = get_string("Rank %i: ", j + 1);

            if (!vote(j, name, ranks))
            {
                printf("Invalid vote.\n");
                return 3;
            }
        }

        record_preferences(ranks);

        printf("\n");
    }

    add_pairs();
    sort_pairs();
    lock_pairs();
    print_winner();
    return 0;
}

// Update ranks given a new vote
bool vote(int rank, string name, int ranks[])
{
    // TODO

    for( int i = 0; i < candidate_count; i++)
    {
        if( strcmp( name, candidates[i]) == 0)
        {
            ranks[rank] = i;
            return true;
        }
    }
    return false;
}

// Update preferences given one voter's ranks
void record_preferences(int ranks[])
{
    // TODO
    for( int i = 0; i < candidate_count; i++)
    {
        for ( int j = i + 1; j < candidate_count; j++)
        {
            preferences[ranks[i]][ranks[j]]++;
        }
    }
    return;
}

// Record pairs of candidates where one is preferred over the other
void add_pairs(void)
{
    // TODO
    for( int i = 0; i < candidate_count; i++)
    {
        for( int j = i+1; j < candidate_count; j++)
        {
            if( preferences[i][j] > preferences[j][i] )
            {
                pairs[pair_count].winner = i;
                pairs[pair_count].loser = j;
                pair_count ++;
            }
            else if( preferences[i][j] < preferences[j][i])
            {
                pairs[pair_count].winner = j;
                pairs[pair_count].loser = i;
                pair_count ++;
            }
        }
    }
    return;
}

// Sort pairs in decreasing order by strength of victory
void sort_pairs(void)
{
    // TODO
    int str[pair_count];
    int temp = 0;

    for( int i = 0; i < pair_count; i++)
    {
        int diff = pairs[i].winner - pairs[i].loser;
        str[i] = diff;
    }

    for( int x = 0; x < pair_count; x++)
    {
        for ( int y = 0; y < pair_count - 1; y++)
        {
            if(str[x] > str[y])
            {
                temp = pairs[x].winner;

                pairs[x].winner = pairs[x+1].winner;
                pairs[x+1].winner = temp;

                temp = pairs[x].loser;

                pairs[x].loser = pairs[x+1].loser;
                pairs[x+1].loser = temp;

            }
        }
    }
    return;
}

// Lock pairs into the candidate graph in order, without creating cycles
void lock_pairs(void)
{
    // TODO
    for(int i = 0; i < pair_count; i++)
        {
            int x = pairs[i].winner;
            int y = pairs[i].loser;

            if(check(x,y) == false)
            {
                locked[x][y] = true;
            }
            else
            {
                locked[x][y] = false;
            }
            printf("%i %i",  x, y );
        }

    return;
}

// Print the winner of the election
void print_winner(void)
{
    // TODO
    //printf{"%s", locked}
    return;
}

bool check ( int x, int y)
{
    typedef struct
    {
        int winner;
        int loser;
    }
    lockedpairs;
    lockedpairs lock[pair_count];

    int count = 0;

    for( int i = 0; i < candidate_count; i++)
    {
        for(int j = i; j < candidate_count; j++)
        {
            if(locked[i][j] == true)
            {
                lock[count].winner = i;
                lock[count].loser = j;
                count++;
            }
        }
    }

    if( x == lock[count].loser && y == lock[0].winner)
    {
        return true;
    }

    return false;
}
deyfvvtc

deyfvvtc1#

嗯,现在还为时尚早......我需要一些时间来理清这些。但是......

if( x == lock[count].loser && y == lock[0].winner)

check()中,这一行显示为可能错误。您可能需要:

if( x == lock[count - 1].loser && y == lock[0].winner)

本地数组lock[]没有初始化,因此lock[count]的值是不定的。
这可能不是全部的解决方案。我对CS50的作业不熟悉。
离题:没有必要在函数末尾使用return;
同样,在main()中,全局变量初始化为0false是不必要的,全局变量保证在执行开始时为0。

相关问题