0

Подскажите, что не так с перегрузкой оператора.

#include<iostream>
using namespace std;
class set 
{   const static int n=10;
    int mass[n];

public:
    int set_construct()
    {
        for(int i = 0; i < n; i++){
            mass[i] = 1 + rand() % 500;};
            return 0;
    }
     void print() 
     {
         for(int i=0;i<n;i++)
             cout<<mass[i]<<"  ";
     }

    set operator+(set rs)
    {
        const int x=2*n;
        set y;
        for(int i=0;i<n;i++)
            y.mass[i]=mass[i];
        for(int i=0;i<x;i++)
            y.mass[i+n]=rs.mass[i];
        return y;
    }
};
int main()
{
    setlocale(LC_ALL,"Russian");
    set ob1;
    ob1.set_construct();
    cout<<"Первое множество\n";
    ob1.print();
    cout<<"\n";
    set ob2;
    ob2.set_construct();
    cout<<"Второе множество\n";
    ob2.print();
    cout<<"\n";
    set set_union;
    set_union=ob1+ob2;
    cout<<"Объединение множеств\n";
    set_union.print();
    system ("pause");
    return 0;
}

Выдает на return y необработанное исключение.

1 ответ 1

2

Вы производите доступ к неаллоцированной памяти в operator+:

for(int i=0;i<x;i++)
    y.mass[i+n]=rs.mass[i];

Здесь x == 20, индекс i меняется от 0 до 19, индекс i+n меняется от 10 до 29.

Поскольку размер mass ровно 10 элементов, ваш код производит доступ за пределами разрешённой памяти, затирая случайные объекты. Вам повезло, что произошёл креш.

«Не так» здесь с дизайном вашего класса: для множества вы не можете рассчитывать на данные фиксированного размера. Поэтому уже объявление

const static int n=10;
int mass[n];

неверно и ведёт ко всем проблемам.

Кроме этого, вы ошиблись с индексами в operator+, но это уже второстепенная проблема.

А почему вы не используете std::set, а пишете велосипед?

(И да, вместо функций типа set_construct умудрённые опытом домохозяйки используют конструктор.)

6
  • Как исправить? Что мне сделать с const static int n=10; int mass[n]; Такое задание - написать класс множество целых чисел. Я совсем недавно начал изучать С++. Так что в число таких домохозяек не вхожу:)
    – abc7
    Commented 8 июн. 2013 в 18:15
  • Вы должны выделять память динамически, при помощи new. И не забудьте потом (в десткукторе, конечно) удалить её, если не планируете утечек памяти.
    – VladD
    Commented 8 июн. 2013 в 18:24
  • Это будет выглядеть как-то так? ..... class set { int n; int *mass; public: set(){n=10;mass=new int[n];} ~set(){delete[] mass;} ..... Или нужно еще в каком месте изменить код?
    – abc7
    Commented 9 июн. 2013 в 13:57
  • Не совсем. Откуда взялась константа 10? Сколько элементов в вашем множестве? (И если больше 0, то какие?) Элементы массива лучше получать снаружи. Затем, в списке элементов массива не должно быть повторений. Вы это проверяете? Нет. Затем, при вычислении объединения множеств снова не должно быть повторений. Вы это учитываете? Нет.
    – VladD
    Commented 9 июн. 2013 в 13:59
  • 10 это количество элементов множества. Почему удобнее получать снаружи?для избежания повторений? С повторениями в объединении я разберусь после того как заставлю работать работать хотя бы так работать.
    – abc7
    Commented 9 июн. 2013 в 16:20

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

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