c - 错误 :invalid type argument of '->' (have 'Queue {aka struct Queue}

Question

我正在尝试读取具有这种格式的文件 instance|instruction...像这样 1 01 01 12 12 33，其中第一个数字是实例，之后是指令的编号。每行都有自己的 ID。所以在那个例子中，如果第一行的 id 是 1，那么下一行是 2，依此类推。

我想做的是按照我拥有的 PCB 结构这样的结构对这些信息进行分组。每行都有一个 ID 和一个队列，我在其中存储指令部分的每个编号。

我有一个结构，里面有一些变量和一个队列。问题是队列不工作。

typedef struct Queue
{
  int sizeQueue;
  int limit;
  Node *head;


}Queue;

typedef struct PCB
{
 int id;
 int program_counter;
 int size;
 Queue pointer_instrucoes;
 int instante;

}PCB;

typedef struct Node
{
 PCB *element;
 Node *next;

}Node;

主要是我调用带有队列的结构来存储一些值。

PCB *p=new_pcb(contador);
....
p->pointer_instrucoes->head=atoi(s2); //s2 is the some number of the //instruction

但我收到此错误:'->' 的类型参数无效(有'Queue {aka struct Queue}

MVCE:

主文件

#include <stdio.h>
#include <stdlib.h>
#include <string.h> //para o output
#include "queue2.h"

int le_ficheiro(char* filename) {
FILE *ficheiro=fopen(filename,"r");
size_t len=0;
char *line=NULL;
ssize_t read;
char *s1;//string antes do primeiro espaço
char *s2;//string depois do primeiro espaço
char *sp; //linha toda
if(ficheiro==NULL) {
    exit(EXIT_FAILURE);
}
int contador=1; //onde comeca o id

while((read = getline (&line, &len,ficheiro))!=-1)   //le de linha a linha
{
    PCB *p=new_pcb(contador);
    sp=strchr(line,' ');
    if(!sp)
    {
        exit(EXIT_FAILURE);
    }
    s1=strndup(line,sp-line);
    s2=sp+1;
    p->instante=atoi(s1); //converte char to int

    printf("Instante: %d\n",p->instante);
    printf("Id: %d\n",p->id);

    p.pointer_instrucoes.head=atoi(s2);

    printf("%d\n",p->pointer_instrucoes->head);



    printf("Retrieved line of length %zu:\n",read);
    printf("%s\n",s2);
    printf("Aqui : %c\n",line[0]);
    contador++;
}
 fclose(ficheiro);
 if(line)
    free(line);
    free(s1);
 exit(EXIT_SUCCESS);

 }


int main()
{
  char name[50];
  printf("Name of the file: ");
  scanf("%s",name);
  le_ficheiro(name);
  return 0;

queue2.h文件

#include <stdbool.h>

typedef struct PCB PCB;
typedef struct Node Node;
typedef struct Queue Queue;
typedef struct Queue
{
  int sizeQueue;
  int limit;
  Node *head;


 }Queue;
typedef struct PCB
{
  int id;
  int program_counter;
  int size;
  Queue pointer_instrucoes;
  int instante;

 }PCB;

 typedef struct Node
 {
  PCB *element;
  Node *next;

  }Node;

 PCB * new_pcb(int id); 
 Node * new_node(PCB * e);
 Queue * new_queue(int limit);

队列2.c

#include <stdio.h>
#include <stdlib.h>
#include <stdio.h>
#include "queue2.h"


Node * new_node(PCB * e)
 {
  Node *n=malloc(sizeof(Node));
  n->element=e;
  n->next=NULL;
  return n;
  }




Queue * new_queue(int limit)
{
   Queue *q=malloc(sizeof(Queue));
   q->limit=limit;
   q->head=NULL;
   q->sizeQueue=0;

   return q;

     }

   PCB * new_pcb(int id)
{
   PCB *p = malloc(sizeof(PCB));
   p->id=id;
   p->pointer_instrucoes=0;
   p->program_counter=0;
   p->size=0;
   return p;
   }

输入示例:1 01 02 03 04 05

2 01 02 03 02 11

Answer 1

你的问题很初级，你需要花时间竞争 le_ficheiro。对于初学者，您不能将 atoi(s2) 的结果分配给 p.pointer_instrucoes.head。 head 不是 int，它是一个指向 Node* 的指针。您违反了严格别名规则 C11 Standard - §6.5 Expressions (p6,7)并且可能有十几个其他人试图通过分配一个 int 作为其值来访问和修改 Node*。

您调用未定义的行为 试图在不确定(未初始化)时访问 p->id。

尝试使用 strchr(line, ' ') 手动将值与 line 分开会使生活变得比实际困难得多。您已经将所有值包含在 line 中，您可以使用 sscanf 在一次调用中将所有六个值解析为 int 并验证该行，例如

    while ((read = getline (&line, &len,ficheiro)) != -1) //le de linha a linha
    {
        int a, b, c, d, e, f;
        if (sscanf (line, "%d %d %d %d %d %d", 
                    &a, &b, &c, &d, &e, &f) != 6) {
            fprintf (stderr, "error: failed to parse line '%d'.\n",
                    contador);
            continue;
        }
        PCB *p = new_pcb (contador);
        if (!p)  /* Validate EVERY Allocation */
            exit (EXIT_FAILURE);

但是，在验证每个分配并避免可能调用未定义行为之前，您必须提供来自new_node、new_queue & new_pcb 的有意义的返回值，以指示成功/失败，例如

Node *new_node (PCB *e)
{
    Node *n = malloc (sizeof *n);
    if (!n) {   /* Validate EVERY Allocation */
        perror ("malloc-new_node");
        return NULL;  /* return NULL on failure */
    }
    n->element = e;
    n->next = NULL;

    return n;
}

Queue *new_queue (int limit)
{
    Queue *q = malloc (sizeof *q);   /* typo in declaration */
    if (!q) {
        perror ("malloc-new_queue");
        return NULL;
    }
    q->limit = limit;
    q->head = NULL;
    q->sizeQueue = 0;

    return q;
}

PCB *new_pcb (int id)
{
    PCB *p = malloc (sizeof *p);
    if (!p) {
        perror ("malloc-new_pcb");
        return NULL;
    }
    p->id = id;
    p->pointer_instrucoes = 0;
    p->program_counter = 0;
    p->size = 0;

    return p;
}

很明显，您完成了 le_ficheiro 的一部分，然后就放弃了。它远未完成，您的代码中甚至只使用了每行输入的数据值中的一个(其余的您尝试打印为文本只是为了查看您的 s2 指针指向的位置-- 但不进一步或有意义地使用这些值)

您甚至不调用 new_node 来创建一个 Node* 值，可以想象它可以分配给 p.pointer_instrucoes.head .一步一步来。您需要在 le_ficheiro 中的某处调用 new_node，这样您就有了一个可以分配给 p.pointer_instrucoes.head 的指针。它没有任何魔力，但您必须有条不紊地确保每个值和每个结构的每个成员在尝试使用它们之前都已创建并使用值正确初始化。

基本上你有一个 le_ficheiro 的外壳，你需要继续完成它，从一个更好的起始位置开始工作:

int le_ficheiro (char* filename) 
{
    FILE *ficheiro = fopen (filename,"r");
    size_t len = 0;
    ssize_t read = 0;
    int contador = 1;   //onde comeca o id

    if (ficheiro == NULL) {
        perror ("fopen-filename");
        exit (EXIT_FAILURE);
    }

    while ((read = getline (&line, &len,ficheiro)) != -1) //le de linha a linha
    {
        int a, b, c, d, e, f;
        if (sscanf (line, "%d %d %d %d %d %d", 
                    &a, &b, &c, &d, &e, &f) != 6) {
            fprintf (stderr, "error: failed to parse line '%d'.\n",
                    contador);
            continue;
        }
        PCB *p = new_pcb (contador);
        if (!p)
            exit (EXIT_FAILURE);

        p->instante = a;
        printf ("Instante: %d\n", p->instante);

        /* you must assign and allocate as necessary to use remaining
         * values read from file HERE. Nowhere do you assign p->id
         * before attempting to print, etc...
         */
        printf ("Id: %d\n", p->id);

        /* you cannot assign atoi(s2) to p.pointer_instrucoes.head.
         * it expects type 'Node *', not int.
         */
        // p.pointer_instrucoes.head = FIXME;
        // printf ("%d\n", p->pointer_instrucoes->head);
        contador++;
    }
    fclose(ficheiro);

    free (line);
    free(s1);

    exit(EXIT_SUCCESS);
}

当您在编写函数时遇到困难时，不要放弃。请参阅 How to debug small programs和鸭子说话...真的，这很有帮助 :)