4

構造体の配列へのポインタを渡そうとしています。このコードは、構造体の配列を作成し、構造体の変数に書き込み、それらを出力する必要があります(これは機能します)。次に、その構造体の配列のポインターを別の関数に渡し、支柱の配列を出力します。

#define PORT_NUMBER 5100
#define MAX_CLIENTS 5

#include <sys/types.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <arpa/inet.h>
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <errno.h>
#include <string.h>
#include <pthread.h>

typedef struct thread_args
 {
    int client_number;
    int connected;
    char client_name[1024];
} client;

void pass_func(client* clients[])

int main()
{
  struct thread_args clients[MAX_CLIENTS];
  int i;

  for(i =0; i < MAX_CLIENTS; i++)
  {
  clients[i].client_number=i;
  strcpy(clients[i].client_name, "BOBBY");
  }

    for(i =0; i < MAX_CLIENTS; i++)
  {
     printf("%d | %s\n", clients[i].client_number=i, clients[i].client_name);
  }

  printf("\n\n");
  pass_func(&clients);
}

void pass_func(client* clients[])
{
  int i;
  for(i =0; i < MAX_CLIENTS; i++)
  {
     printf("%d | %s\n", clients[i]->client_number=i, clients[i]->client_name);
  }
}

そしてこれは出力です:

$ gcc TEST.c -lpthread -o TEST.out
TEST.c: In function ‘main’:
TEST.c:41:3: warning: passing argument 1 of ‘pass_func’ from incompatible pointer type [enabled by default]
TEST.c:22:6: note: expected ‘struct thread_args **’ but argument is of type ‘struct thread_args (*)[5]’

$ ./TEST.out 
0 | BOBBY
1 | BOBBY
2 | BOBBY
3 | BOBBY
4 | BOBBY


Segmentation fault

私は約1時間の調査を行いましたが、なぜこれが機能しないのか理解できません。私が見つけた例のほとんどはC++用ですが、C用ではありません(もちろん、インクルードしたヘッダーファイルの多くはこのコードには必要ありません。これは元のコードの一部にすぎません)。

4

3 に答える 3

13

pass_funcへのポインタの配列を期待しますclient

void pass_func(client* clients[]);

しかし、あなたはそれを渡します

pass_func(&clients);

の配列へのポインタclient。したがって、client clients[i]はへのポインタとして解釈されますclientpass_func、もちろんビットパターンはへの有効なポインタではないためclient、アクセスしてはならないメモリにアクセスしてセグメンテーション違反を取得しようとしています。

ポインタの配列を渡すか、宣言しますpass_func

void pass_func(client *clients);

(そしてpass_func(clients)、メインのアドレス演算子なしで通過します)。

ただし、コンパイラは、互換性のないポインタ型を渡すことについて警告しました。

于 2012-04-21T01:19:21.437 に答える
2
void pass_func(client* clients[])
{
  int i;
  for(i =0; i < MAX_CLIENTS; i++)
  {
     printf("%d | %s\n", (*clients)[i].client_number=i, (*clients)[i].client_name);
  }
}

これで問題ありません。

于 2012-04-21T02:24:28.690 に答える
1

あなたは基本を正しく理解する必要があります...

まず、配列が関数に渡される方法を最初に理解する必要があります。これをよく理解してください。

于 2012-04-21T02:46:28.277 に答える