Блог

Я пытаюсь ответить на домашнее задание, которое у меня есть, где мне нужно запросить координаты «n», а затем построить массив координат структуры с таким размером. Каждая структура координат имеет 2 переменные типа double.

Я попытался получить ввод n от пользователя и проверить с помощью оператора if, что сначала оно больше 0, чтобы избежать какого-либо сбоя

 printf("Enter amount of coordinates:n");
scanf_s("%d", amp;n);
if (n > 0) {
    coordinates cordArray = (coordinates *)malloc(cordArray * sizeof(n));
  

}

Ожидается, что после получения значения n от пользователя будет выделен массив structs нужного размера. Я получаю ошибку «Ошибка состояния подавления строки файла проекта Описания кода серьезности (активна) E0415 не существует подходящего конструктора для преобразования из «координат *» в «координаты»

вероятно

 coordinates cordArray = (coordinates *)malloc(cordArray * sizeof(n));
  

необходимо заменить на

 coordinates * cordArray = (coordinates *)malloc(sizeof(coordinates) * n);
  

поскольку вы присваиваете a coordinates с помощью a coordinates * и cordArray неизвестно, и даже это может быть не конвертируемо в a size_t (вы не даете его определения)

Кстати, вы на C , зачем вы используете массив C, вы можете выделить его с помощью new, позволяющего вызывать конструктор координат для каждой записи или лучше использовать std::vector , позволяющий изменять размер, доступ к размеру и т. Д

По поводу ваших замечаний, эта программа компилируется и запускается :

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

typedef struct { 
  double x;
  double y;
}coordinates;

int main()
{
  printf("Enter amount of coordinates:n");

  int n;

  if (scanf("%d", amp;n) != 1)
    puts("invalid number");
  else if (n <= 0)
    puts("expected a positive number");
  else {
    coordinates * cordArray = (coordinates *) malloc(sizeof(cordArray) * n);

    if (cordArray == NULL)
      printf("cannot allocate memory, probably %d too largen", n);
    else {
      int i;

      for (i = 0; i < n; i  ) { 
        printf("Please enter coordinates for coordinates #%d:n", i   1);
        if (scanf("%lf%lf", amp;cordArray[i].x, amp;cordArray[i].y) != 2){
          puts("invalid values");
          return -1;
        }
      }
    }
  }
}
  

Обратите внимание, что это код на C, почему вы использовали тег C ?

 pi@raspberrypi:/tmp $ g   -pedantic -Wall -Wextra c.cc
pi@raspberrypi:/tmp $ ./a.out
Enter amount of coordinates:
2
Please enter coordinates for coordinates #1:
1 2
Please enter coordinates for coordinates #2:
3 4
  

@bruno показал правильный способ выделения памяти с помощью malloc . Однако это можно сделать еще проще с помощью оператора new[] , который автоматически вычисляет объем требуемой памяти.

 coordinates *cordArray = new coordinates[n];
  

Чтобы освободить память, используйте operator delete[] .

 delete[] cordArray;
  

Другим преимуществом способа выделения C является то, что он вызывает конструкторы для вновь созданных объектов. (Если у них есть какие-либо конструкторы, то есть.)