Как исправить предупреждения компилятора C6386 и C6385?

Question

Для программы, которую я пишу, мне нужно создать массив с заданным пользователем размером, а также случайные значения в диапазоне от -15 до 15. Поэтому я использую srand вместе с динамическим c приведением массива. Ниже приведен мой полный код:

#include <iostream>
#include <cstdlib>
#include <ctime>

using namespace std;

int main()
{
    //initialize variables
    double minValue;
    double maxValue;
    double firstQuartile;
    double thirdQuartile;
    double skewness;

    int size;
    std::cout << "Please enter the size of the data set: ";                 //get data set size from user
    std::cin >> size;
    int* dataSet = new int[size];                                                           
    cout << endl << "These are the randomly generated values: ";

    srand(static_cast<unsigned int>(time(NULL)));

    int i = size;
    for (int x = 0; x < i; x++)                                         //generate random numbers
    {
        dataSet[i] = (rand() % 31) - 15;
        cout << dataSet[i] << " ";
    }


} 

Хотя он все еще компилируется, мне нужно, чтобы он делал это без ошибок, а выделенный ниже раздел вызывает две ошибки, которые я не знаю, как исправить.

int i = size;
    for (int x = 0; x < i; x++)                                         //generate random numbers
    {
        dataSet[i] = (rand() % 31) - 15;
        cout << dataSet[i] << " ";
    }

C6386 Переполнение буфера при записи в «dataSet»: размер записи равен «size * 4», но могут записываться «i».

AND

C6385 Чтение недействительных данных из набора данных: читаемый размер составляет размер * 4 байта, но могут считываться байты i.

Answer 1

В этом l oop индексная переменная x не i.

Так что измените l oop как

int i = size;
for (int x = 0; x < i; x++)                                         //generate random numbers
{
    dataSet[x] = (rand() % 31) - 15;
    cout << dataSet[x] << " ";
}

Фактически переменная i является избыточной и создает код подвержен ошибкам Почему бы не написать

for (int i = 0; i < size; i++)                                         //generate random numbers
{
    dataSet[i] = (rand() % 31) - 15;
    cout << dataSet[i] << " ";
}