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

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
hil400k
0 / 0 / 0
Регистрация: 06.01.2013
Сообщений: 68
#1

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

08.06.2013, 18:14. Просмотров 203. Ответов 3
Метки нет (Все метки)

Не могу понять в чем проблема

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();
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
08.06.2013, 18:14     Простая программка (найти ошибку в коде)
Посмотрите здесь:

C++ Простая задачка, но туплю, помогите найти ошибку с new.
C++ Простая программа. Высчитать формулу. Нужно найти ошибку
Найти ошибку в коде C++
C++ простая программка с if и switch, пожалуйста!!
найти ошибку в коде C++
Найти ошибку в коде C++
C++ Найти ошибку в коде
Найти ошибку в коде C++
Найти ошибку в коде C++
C++ Найти ошибку в коде
C++ найти ошибку в коде
Найти ошибку в коде C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Гром
 Аватар для Гром
200 / 119 / 11
Регистрация: 20.03.2009
Сообщений: 1,083
Записей в блоге: 16
Завершенные тесты: 1
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     Простая программка (найти ошибку в коде)
Ответ Создать тему
Опции темы

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