Std::bad_alloc при суммировании матрицы

@Rickk · Регистрация: 17.12.2013

Студворк — интернет-сервис помощи студентам

Есть некий класс матрицы.
Конструктор, деструктор

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
Matrix (size_t a, size_t b, bool is_rand = true, bool writte = true)
            : date(new Type *[a]), a_size(a), b_size(b)
    {
        for (size_t i = 0; i != a; ++i) {
            date[i] = new Type [b];
        }
        if(is_rand==true) {
            this->addRandElement();
        } else {
            for (int j = 0; j < a; ++j) {
                for (int i = 0; i < b; ++i) {
                    date[j][i]=0;
                }
            }
        }
        if(writte== true) {
            this->writte();
        }
    }
    ~Matrix() {
        for (int i = 0; i != a_size; ++i) {
            delete  []date[i];
        }
        delete []date;
    }

Оператор=

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
Matrix &operator= (Matrix const &a) {
        if(this!=&a) {
            this->~Matrix();
            a_size = a.a_size;
            b_size  = a.b_size;
            date = new Type *[a_size];
            for (int i = 0; i != a_size; ++i) {
                date[i] = new Type [b_size];
            }
            for (int j = 0; j != a_size; ++j) {
                for (int i = 0; i != b_size; ++i) {
                    date[j][i] = a.date[j][i];
                }
            }
        }
        return  * this;
 
    }

И само суммирование

C++
1
2
3
4
5
6
7
Matrix operator+ (Matrix const &T) {
        for (int i = 0; i != a_size; ++i) {
            for (int j = 0; j != b_size; ++j) {
                this->date[i][j] = date[i][j]+T.date[i][j];
            }
        }
    }

Делаю

C++
1
2
3
Matrix <int> k(5,5);
    Matrix <int> l(5,5);
   l = k+l;

Ошибок компиляции нету.
Но получаю This application has requested the Runtime to terminate it in an unusual way.
Please contact the application's support team for more information.
terminate called after throwing an instance of 'std::bad_alloc'
what(): std::bad_alloc

Что-то я намудрил во всем этом, но что понять не могу.

@Melg · 03.01.2015, 00:12

Во-первых, не могли бы вы либо предоставить полностью листинги класса и вызова, либо - предоставить единым файлом листинг, минимально необходимый, для воспроизведения проблемы. Во-вторых, кидает эксепшн он на операции k + l; или на операции присваивания? В третьих, из того что я вижу сразу -

C++
1
2
       if(this!=&a) {
            this->~Matrix();

для чего явный вызов деструктора?

C++
1
2
3
4
5
6
 ~Matrix() {
        for (int i = 0; i != a_size; ++i) {
            delete  []date[i];
        }
        delete []date;
    }

нет зануления указателей на освобождаемую память - особенно актуально, когда вы используете деструктор явным образом

Скорее всего косяк где-то тут:

C++
1
2
3
4
5
b_size  = a.b_size;
            date = new Type *[a_size];
            for (int i = 0; i != a_size; ++i) {
                date[i] = new Type [b_size];
            }

а именно, если а.b_sizе - окажется не инициализирован (там будет мусор условно говоря) то new попытается выделить ну скажем много-много байт памяти, и выкенет эксепшн, из-за того что столько нету.
Еще 1 вариант - все ваши операции, которые выделяют память ( сначала при помощи new - потом при помощи контейнеров - если таковые используются) проверить дебагом на то, какие куски памяти у них запрашиваются. Проверить, все ли элементы получают корректные значения.

Добавлено через 2 минуты
На всякий случай, чтоб вы понимали куда рыть http://www.cplusplus.com/reference/new/bad_alloc/

Добавлено через 6 минут

C++
1
2
3
4
5
6
7
Matrix operator+ (Matrix const &T) {
        for (int i = 0; i != a_size; ++i) {
            for (int j = 0; j != b_size; ++j) {
                this->date[i][j] = date[i][j]+T.date[i][j];
            }
        }
    }

Вы ничего не возвращаете из метода, который должен вернуть матрицу? там шалапуша вместо матрицы забитая черти чем?

Добавлено через 14 минут
Можно почитать на счет перегрузки операторов в канонической форме:
http://stackoverflow.com/quest... verloading
http://en.cppreference.com/w/c... /operators

@castaway · 03.01.2015, 22:17

Rickk, в предоставленном коде я проблем не вижу. Где-нибудь еще память выделяется по средствам оператора new?

Сообщение от Melg

а именно, если а.b_sizе - окажется не инициализирован

Он в конструкторе явно инициализируется.

@Rickk · 03.01.2015, 22:36 **[ТС]**

Сообщение от castaway

Rickk, в предоставленном коде я проблем не вижу. Где-нибудь еще память выделяется по средствам оператора new?

Да

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
Matrix (Matrix &a)
            : a_size(a.a_size), b_size(a.b_size)
    {
        date = new Type *[a_size];
        for (size_t i = 0; i != a_size; ++i) {
            date[i] = new Type [b_size];
        }
        for (int i = 0; i != a_size; ++i) {
            for (int j = 0; j != b_size; ++j) {
                date[i][j] = a.date[i][j];
            }
        }
    }

@castaway · 03.01.2015, 23:12

Увы, но тут я тоже не вижу проблем.
Rickk, можно код целиком?

@Melg · 03.01.2015, 23:35

Еще раз, по-вашему этот код сложения матриц:

C++
1
2
3
4
5
6
7
Matrix operator+ (Matrix const &T) {
        for (int i = 0; i != a_size; ++i) {
            for (int j = 0; j != b_size; ++j) {
                this->date[i][j] = date[i][j]+T.date[i][j];
            }
        }
    }

является корректным? Я не вижу тут return-а. И настаиваю на том, что его наличие необходимо. Насколько я себе представляю - после сложения возвращается некоторый объект, который заполнен некорректными значениями, и благодаря которому, при выполнении оператора присваивания происходит попытка выделить блок памяти недопустимого размера.

Новые блоги и статьи Все статьи Все блоги /
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Рецензия / Мнение Это мой обзор планшета X220 с точки зрения школьника. Недавно я решила попытаться уменьшить свой. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,170 Записей в блоге: 10
	03.01.2015, 23:12
	Увы, но тут я тоже не вижу проблем. Rickk, можно код целиком? 0