1

В результате исполнения этого кода - у программы возникает ошибка сегментации. Подскажите какова ее природа?

Порядок вызова методов

trainData obj;
obj.getSize(size);
obj.setData(inputArray);
obj.getData();

Исполнительный файл

void trainData::setData(int inputArray[])
{
    int *incomeData = new int [mySize];
    for (int i=0;i<mySize;i++)
    {
        incomeData[i] = inputArray[i];
    }
}
/**
    Функция которая будет получать размер входящих данных.
*/
void trainData::getSize(int size)
{
    mySize = size;
}
/**
    Вывод данных
*/
void trainData::getData()
{
    for (int i = 0; i < mySize; i++)
    {

        cout << incomeData[i] << " "; <- тут возникает ошибка сегментации
    }
}

Файл с заголовками

class trainData
{
public:
    trainData();
    ~trainData();
    void setData(int*);
    void getData();
    void getSize(int);

private:
    int *incomeData[];
    int mySize;
};
5
  • Природа не ясна, поскольку что есть myData вы не указали, порядок вызова методов тоже. 2 ноя 2015 в 11:46
  • Кусочек ненужного кода забыл удалить. так правильно.
    – Ximerik
    2 ноя 2015 в 11:51
  • Нигде в Вашем коде Вы не присваиваете полю incomeData какого-либо значения.
    – user181245
    2 ноя 2015 в 11:52
  • Я ему присваиваю значения 'inputArray[]'
    – Ximerik
    2 ноя 2015 в 11:54
  • Не ему вы присываиваете. Не ему. 2 ноя 2015 в 11:55

1 ответ 1

5

int *incomeData = new int [mySize]; - локальное объявление переменной, а cout << incomeData[i] - обращение к полю класса, которое не инициализировано. Вот и природа. Осваивайте отладчик, пришло время.

3
  • Абсолютно точно. И лучше бы это все сделать в конструкторе.
    – andy.37
    2 ноя 2015 в 12:01
  • @Ximerik, разобрались, что у Вас локальная переменная int *incomeData в функции trainData::setData перекрывает одноимённое поле класса trainData? Если напишете так: incomeData = new int [mySize]; (т.е. без int *) должно сработать.
    – aleks.andr
    2 ноя 2015 в 14:14
  • Разобрался. Реализовал через конструктор.
    – Ximerik
    11 ноя 2015 в 13:51

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.