Форум программистов, компьютерный форум CyberForum.ru

Простая программка (найти ошибку в коде) - C++

Восстановить пароль Регистрация
 
hil400k
0 / 0 / 0
Регистрация: 06.01.2013
Сообщений: 68
08.06.2013, 18:14     Простая программка (найти ошибку в коде) #1
Не могу понять в чем проблема

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
#include<iostream>
#include<conio.h>
using namespace std;
 
 
class ARR
{
    public:
    int i;
    int *data;
 
    ARR(){i=0;};
    ~ARR(){};
    void add(int r){
        data[i]=r;
        i++;
    }
};
void main()
{
    int y=9;
    ARR One;
    One.add(y);
    cout<<One.data;
    _getch();
}
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Гром
 Аватар для Гром
199 / 118 / 10
Регистрация: 20.03.2009
Сообщений: 1,075
Записей в блоге: 15
08.06.2013, 18:30     Простая программка (найти ошибку в коде) #2
У вас не выделяется память под data. Должно быть что-то вроде
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
class ARR
{
    public:
    int i;
    int *data;
 
    ARR(): i(0), data(0) {}   //Используем список инициализации вместо присваивания в теле функции
    APR(int sz):
       i(sz), data(new int[sz])   //Другой конструктор, в котором создаем массив размера sz
       {
       for (int k = 0; k < sz; ++k)   //В теле конструктора инициализируем элементы нулями
              data[k] = 0;
       }
    ~ARR() { delete[] data; }   //Освобождаем память, если применяем delete к нулевому указателю, ничего не происходит
    void add(int r)
    {
       int* temp = new int[i + 1];   //Создаем новый массив размера на единицу больше, копируем в него старые значения
       for (int k = 0; k < i; k++)
              temp[k] = data[k];
       temp[i]=r;   //Добавляем в последний элемент новое значение
       delete[] data;   //Удаляем старый массив
       data = temp;   //Присваиваем указателю адрес нового массива
        i++;
    }
};
Зря вы только размер массива назвали i, оно обычно используется как счетчик циклов. Используйте лучше что-то вроде size, sz, arr_sz, N. И выводить нужно не массив (указатель), а элемент массива:
C++
1
cour << One.data[0];
BumerangSP
 Аватар для BumerangSP
4283 / 1405 / 121
Регистрация: 16.12.2010
Сообщений: 2,941
Записей в блоге: 3
08.06.2013, 18:35     Простая программка (найти ошибку в коде) #3
data - указатель, не выделив память, Вы пытаетесь записать какие-то данные по нужному адресу. Оттого и ошибка.
Так будет правильней:
C++
1
2
3
4
ARR(): data(new int[10]), i() {}
    ~ARR(){delete[] data;};
...
One.data[0] // подозреваю, что data - это динамический массив, тогда вызов такой
Добавлено через 26 секунд

Не по теме:

опоздал)

hil400k
0 / 0 / 0
Регистрация: 06.01.2013
Сообщений: 68
08.06.2013, 18:57  [ТС]     Простая программка (найти ошибку в коде) #4
спасибо, ребята
Yandex
Объявления
08.06.2013, 18:57     Простая программка (найти ошибку в коде)
Ответ Создать тему
Опции темы

Текущее время: 08:55. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru