Question

我正在尝试使用此格式实例|指令读取文件... 像这样1 01 01 12 12 33，其中第一个数字是实例，之后是指令的编号。每行都有自己的id。所以在那个例子中，if是第一个，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

但我收到此错误：' - ＆gt;'的无效类型参数（包含'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);

Queue2.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*的指针。您通过指定Node*作为其值，违反了严格别名规则 C11 Standard - §6.5 Expressions (p6,7)以及可能有十几个其他人试图访问和修改int。

当不确定（未初始化）时，您调用未定义行为尝试访问p->id。

尝试使用line从strchr(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*的{{1}}值。一步一步来。您需要在p.pointer_instrucoes.head中的某处调用new_node，以便拥有可指定给le_ficheiro的指针。它没有任何魔力，但你必须有条不紊地确保每个值，每个结构的每个成员都是在你尝试使用之前用值创建并正确初始化的。

基本上你有一个p.pointer_instrucoes.head的shell，你需要继续完成，从一个更好的起始位置工作：

le_ficheiro

当你不得不写一个函数时，不要放弃。相反，请参阅How to debug small programs并与鸭子交谈......真的，它有助于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); }

错误：' - ＆gt;'的无效类型参数（包含'Queue {aka struct Queue}

1 个答案: