我已经和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;
}
1条答案
按热度按时间deyfvvtc1#
嗯,现在还为时尚早......我需要一些时间来理清这些。但是......
在
check()
中,这一行显示为可能错误。您可能需要:本地数组
lock[]
没有初始化,因此lock[count]
的值是不定的。这可能不是全部的解决方案。我对CS50的作业不熟悉。
离题:没有必要在函数末尾使用
return;
。同样,在
main()
中,全局变量初始化为0
或false
是不必要的,全局变量保证在执行开始时为0。