Question

当我使用我的添加功能输入数据时，我能够打印它们没问题。为start（x，y），end（x，y）添加一个vaule有效，我可以打印这些值。但是当我在输入第一组开始值和结束值后离开函数时，我指定我想要两个numGraphicElements。它在第二次循环时会覆盖我以前的值。

当我打印值时，由于某种原因用随机数覆盖我的第一个值，然后只显示第二组start（x，y）end（x，y）值。

第一组的例子：start（1,2）end（3,4）....这个打印正确

添加第二组的示例：start（9,8）end（6,7）...这也打印

示例pf打印出2组，x（23424），y（653243），end = x（2334）y（33434）..... x（9）y（8）end x（3） Y（4）。

#define _CRT_SECURE_NO_WARNINGS

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <crtdbg.h>

#define _CRTDBG_MAP_ALLOC

enum
{
  RUNNING = 1
};

struct Point
{
  int x, y;
};

struct Line
{
  Point start;
  Point end;
};

struct GraphicElement
{
  enum
  {
    SIZE = 256
  };
  unsigned int numLines; //number of lines
  Line* pLines; //plines points to start and end
  char name[SIZE];
};

typedef struct
{
  unsigned int numGraphicElements;
  GraphicElement* pElements; //the head points to pLines
} VectorGraphic;

void InitVectorGraphic(VectorGraphic*);
void AddGraphicElement(VectorGraphic*);
void ReportVectorGraphic(VectorGraphic*);

VectorGraphic Image;

int main()
{
  char response;
  InitVectorGraphic(&Image);

  //for debugging purposes
  _CrtSetDbgFlag(_CRTDBG_ALLOC_MEM_DF | _CRTDBG_LEAK_CHECK_DF);

  //main menu for the program
  while (RUNNING)
  {
    printf("\nPlease select an option:\n");
    printf("1. Add a Graphic Element\n");
    printf("2. List the Graphic Elements\n");
    printf("q. Quit\n");
    printf("CHOICE: ");
    fflush(stdin);
    scanf("%c", &response);

    switch (response)
    {
    case '1':
      AddGraphicElement(&Image);
      break;
    case '2':
      ReportVectorGraphic(&Image);
      break;
    case 'q':
      CleanUpVectorGraphic(&Image);
      return 0;
    default:
      printf("Please enter a valid option\n");
    }
    printf("\n");
  }
}

/*initialize the vectors, allocate memory*/
void InitVectorGraphic(VectorGraphic * pImage)
{ //addres of pImage is passed in
  struct GraphicElement *pElements;
  pImage->pElements = (GraphicElement*) malloc(sizeof(GraphicElement)); //pImage is now the addess of image
  pElements = (GraphicElement*) malloc(sizeof(GraphicElement));
  pImage->numGraphicElements = 8;
  pImage->pElements = NULL;
}

Answer 1

下面

line = (struct Line *) malloc(sizeof(Line));

代码完全分配一个行对象。

此处它似乎只涉及line[0]：

line[pImage->numGraphicElements]....

这样做代码会调用未定义的行为，从此刻开始就会发生任何事情。这包括覆盖属于同一进程的其他内存。

函数不断覆盖以前的结构数据

1 个答案: