我在读取数据作为参数时,我们必须将其长度作为参数传递(严格C89)。 在此给定的代码段中
#include <stdio.h>
#include <stdbool.h>
#include <stdlib.h>
#define NUM_RANKS 13
#define NUM_SUITS 4
#define NUM_CARDS 5
bool straight, flush, four, three;
int pairs;
void read_cards(int a[], int b[]); // I did't specify the length in this declaration
void analyze_hand(int a[], int b[]); // I did't specify the length in this declaration
void print_result(void);
int main()
{
int num_in_rank[NUM_RANKS];
int num_in_suit[NUM_SUITS];
for(;;)
{
read_cards(num_in_rank, num_in_suit);
analyze_hand(num_in_rank, num_in_suit);
print_result();
}
}
void read_cards(int num_in_rank[], int num_in_suit[])
{
bool card_exists[NUM_RANKS][NUM_SUITS];
char ch, rank_ch, suit_ch;
int rank, suit;
bool bad_card;
int cards_read = 0;
for(rank = 0; rank < NUM_RANKS; rank++)
{
num_in_rank[rank] = 0;
for(suit = 0; suit < NUM_SUITS; suit++)
card_exists[rank][suit] = false;
}
for(suit = 0; suit < NUM_SUITS; suit++)
num_in_suit[suit] = 0;
while(cards_read < NUM_CARDS)
{
bad_card = false;
printf("Enter a card: ");
rank_ch = getchar();
switch(rank_ch)
{
case '0': exit(EXIT_SUCCESS);
case '2': rank = 0; break;
case '3': rank = 1; break;
case '4': rank = 2; break;
case '5': rank = 3; break;
case '6': rank = 4; break;
case '7': rank = 5; break;
case '8': rank = 6; break;
case '9': rank = 7; break;
case 't': case 'T': rank = 8; break;
case 'j': case 'J': rank = 9; break;
case 'q': case 'Q': rank = 10; break;
case 'k': case 'K': rank = 11; break;
case 'a': case 'A': rank = 12; break;
default: bad_card = true;
}
suit_ch = getchar();
switch(suit_ch)
{
case 'c': case 'C': suit = 0; break;
case 'd': case 'D': suit = 1; break;
case 'h': case 'H': suit = 2; break;
case 's': case 'S': suit = 3; break;
default: bad_card = true;
}
while((ch = getchar()) != '\n')
if(ch != ' ')
bad_card = true;
if(bad_card)
printf("Bad card; ignored.\n");
else if(card_exists[rank][suit])
printf("Duplicate card; ignored.\n");
else
{
num_in_rank[rank]++;
num_in_suit[suit]++;
card_exists[rank][suit] = true;
cards_read++;
}
}
}
void analyze_hand(int num_in_rank[], int num_in_suit[])
{
int num_consec = 0;
int rank, suit;
straight = false;
flush = false;
four = false;
three = false;
pairs = 0;
for(suit = 0; suit < NUM_SUITS; suit++)
if(num_in_suit[suit] == NUM_CARDS)
flush = true;
rank = 0;
while(num_in_rank[rank] == 0)
rank++;
for(; rank < NUM_RANKS && num_in_rank[rank] > 0; rank++ )
num_consec++;
if(num_consec == NUM_CARDS)
{
straight = true;
return;
}
for(rank = 0; rank < NUM_RANKS; rank++)
{
if(num_in_rank[rank] == 4)
four = true;
if(num_in_rank[rank] == 3)
three = true;
if(num_in_rank[rank] == 2)
pairs++;
}
}
我将两个数组num_in_rank[NUM_RANKS]
和num_in_suit[NUM_SUITS]
一起传递给函数void read_cards(int a[], int b[])
和void analyze_hand(int a[], int b[])
,而没有提供数组长度,我不知道这是如何工作的(没有任何警告/错误)?
任何想法是对还是错?
答案 0 :(得分:2)
当你在c中传递一个数组时,所见的所有函数都是一个指向数组开头的指针,因此在函数内部看不到长度。
为了允许函数使用长度,你必须创建一些也包含长度的结构,在数组的有效输入的末尾有某种特殊字符(比如'\ 0 'for strings),或者作为函数参数expicitly传递大小。
如果你没有传入长度,你就不会遇到编译错误,但是你在阵列上进行的大多数操作都需要你知道它有多长,所以你很可能会遇到段错误你试图在不知道它的长度的情况下对阵列进行操作。
编辑:
发布完整代码之后,这是因为您使用宏进行边界检查,这有点像传入大小,但不太灵活。
答案 1 :(得分:2)
我在读取数据作为参数时,我们必须将其长度作为参数传递
这仅适用于函数未预先知道数组大小的情况。当函数知道数组必须有一定数量的元素时,你不需要将大小传递给函数。
这正是您的代码中发生的事情:接收数组的两个函数都知道num_in_rank
包含完全NUM_RANKS
个元素,num_in_suit
包含完全NUM_SUITS
个元素。这就是为什么你不需要传递大小,也不会发生任何事情:函数已经通过#define
d常量知道大小。
另一方面,如果您需要传递一个未知函数的数组,那么您需要执行以下操作之一:
答案 2 :(得分:1)
请记住,C中的数组实际上是指向程序预留的内存块开头的指针。所以当你说
时int num_in_rank[NUM_RANKS];
int num_in_suit[NUM_SUITS];
您可以将它们用作指针。通常在实践中也将大小作为int传递,例如
void read_cards(int a[], int b[], int a_size, int b_size);
void analyze_hand(int a[], int b[], int a_size, int b_size);
如果你需要利用数组的大小。但要回答你的问题,做你正在做的事情是没有问题的。
答案 3 :(得分:1)
我在Visual Studio上尝试了编译器如何为各种函数声明进行名称修改 -
void foo(int a[]);
void foo(int a[10]);
void foo(int *a);
对于上述所有声明,受损的名称都是唯一的 -
?foo@@YAXPAH@Z
我看到MS编译器正在将函数声明转换为 -
void __cdecl foo(int * const)
所以,至少它让我们深入了解MS编译器在函数声明中使用否定数组索引之前不会担心。如果我没有错的话,g ++也应该做这样的事情。