Ошибка в программе

@Blekzet · Регистрация: 21.12.2015

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

Написал 2 класса: 1 - описывает дробь с числителем и знаменателем (+ операции), 2 - шаблон, описывающий динамический массив указателей. Проблема в том, что выдает ошибку показывая на деструкторы классов... Что я сделал не так?
Код программы.

Кликните здесь для просмотра всего текста

C++

#include <iostream>
#include <cstring>
#include <cctype>
#include <cstring>
#include <windows.h>
 
using namespace std;
 
class Fractions {
private:
    int *arr;
    char tmp[500];
    void simplifyStr (char str[]){
        int cnt = 0;
        for (int j = 0; j < strlen(str); j++){
            if (str[j] == ' ') {
                cnt++;
                for (int dj = j; dj < strlen(str) - 1; dj++){
                    str[dj] = str[dj+1];
                }
            }
        }
        str[strlen(str) - cnt] = '\0';
    }
public:
    Fractions(): arr(new int[3]) {
        arr[1] = 0;
        arr[2] = 1;
        arr[0] = 0;
    }
    Fractions(char str[]): arr(new int[3]){
        simplifyStr(str);
        char *pch = strtok(str," +/");
        for (int i = 0; i < 3; i++){
            arr[i] = atoi(pch);
            pch = strtok(NULL," +/");
        }
    }
    Fractions(int pArr[]): arr(new int[3]) {
        for (int i = 0; i < 3; i++){
            arr[i] = pArr[i];
        }
    }
    Fractions(const Fractions &obj) {
        arr = obj.arr;
    }
    ~Fractions() {
        delete [] arr;
    }
 
    void getData(){
        cout << "Целое = " << arr[0] << endl;
        cout << "Делимое = "<< arr[1] << endl;
        cout << "Делитель = "<< arr[2] << endl << endl;
    }
 
    void setData(){
        cout << "Введите выражение = ";
        cin.getline(tmp,500);
        simplifyStr(tmp);
        char *pch = strtok(tmp," +/");
        for (int i = 0; i < 3; i++){
            arr[i] = atoi(pch);
            pch = strtok(NULL," +/");
        }
    }
    void RWIndividualComponents(){
        int cnt, num;
        while (cnt != 0) {
            cout << "1 - Изменить целую часть" << endl;
            cout << "2 - Изменить числитель" << endl;
            cout << "3 - Изменить знаменатель" << endl;
            cout << "4 - Получить значение целой части" << endl;
            cout << "5 - Получить значение числителя" << endl;
            cout << "6 - Получить значение знаменателя" << endl;
            cout << "0 - Закончить" << endl;
            cout << "Выберите действие = ";
            cin >> cnt;
            cout << endl;
            switch(cnt){
                case 1: {
                    cout << "Целая часть = ";
                    cin >> num;
                    cout << endl;
                    arr[0] = num;
                    continue;
                }
                case 2: {
                    cout << "Числитель = ";
                    cin >> num;
                    cout << endl;
                    arr[1] = num;
                    continue;
                }
                case 3: {
                    cout << "Знаменатель = ";
                    cin >> num;
                    cout << endl;
                    arr[2] = num;
                    continue;
                }
                case 4: {
                    cout << "Значение целой части = " << arr[0] << endl << endl;
                    continue;
                }
                case 5: {
                    cout << "Значение числителя = " << arr[1] << endl << endl;
                    continue;
                }
                case 6: {
                    cout << "Значение знаменателя = " << arr[2] << endl << endl;
                    continue;
                }
                case 0: {
                    cout << "Завершено!" << endl << endl;
                    break;
                }
            }
        }
    }
    double getNumerator(){
        if (arr[0] != 0){ return (arr[1] + (arr[0] * arr[2])); }
        else { return arr[1]; }
    }
    double getDenominator(){
        return arr[2];
    }
    double getNumerator(int arr2[]){
        if (arr2[0] != 0){ return (arr2[1] + (arr2[0] * arr2[2])); }
        else { return arr2[1]; }
    }
    double getDenominator(int arr2[]){
        return arr2[2];
    }
    Fractions& operator+ (int arr2[]) {
        if (getDenominator() != getDenominator(arr2)) {
            arr[1] = getNumerator() * getDenominator(arr2) + getNumerator(arr2) * getDenominator();
            arr[2] = getDenominator() * getDenominator(arr2);
            arr[0] = 0;
            return *this;
        }
        else {
            arr[1] = getNumerator() + getNumerator(arr2);
            arr[0] = 0;
            return *this;
        }
    }
    Fractions& operator- (int arr2[]) {
        if (getDenominator() != getDenominator(arr2)) {
            arr[1] = getNumerator() * getDenominator(arr2) - getNumerator(arr2) * getDenominator();
            arr[2] = getDenominator() * getDenominator(arr2);
            arr[0] = 0;
            return *this;
        }
        else {
            arr[1] = getNumerator() - getNumerator(arr2);
            arr[0] = 0;
            return *this;
        }
    }
    Fractions& operator* (int arr2[]) {
        arr[1] = getNumerator() * getNumerator(arr2);
        arr[2] = getDenominator() * getDenominator(arr2);
        arr[0] = 0;
        return *this;
    }
    bool operator< (int arr2[]) {
        if ((getNumerator() / getDenominator()) < (getNumerator(arr2) / getDenominator(arr2))){
            return true;
        }
        else {
            return false;
        }
    }
    bool operator> (int arr2[]) {
        if ((getNumerator() / getDenominator()) > (getNumerator(arr2) / getDenominator(arr2))){
            return true;
        }
        else {
            return false;
        }
    }
    bool operator<= (int arr2[]) {
        if ((getNumerator() / getDenominator()) <= (getNumerator(arr2) / getDenominator(arr2))){
            return true;
        }
        else {
            return false;
        }
    }
    bool operator>= (int arr2[]) {
        if ((getNumerator() / getDenominator()) >= (getNumerator(arr2) / getDenominator(arr2))){
            return true;
        }
        else {
            return false;
        }
    }
    bool operator== (int arr2[]) {
        if ((getNumerator() / getDenominator()) == (getNumerator(arr2) / getDenominator(arr2))){
            return true;
        }
        else {
            return false;
        }
    }
    bool operator!= (int arr2[]) {
        if ((getNumerator() / getDenominator()) != (getNumerator(arr2) / getDenominator(arr2))){
            return true;
        }
        else {
            return false;
        }
    }
    Fractions& operator^ (int arr2[]) {
        int cnt;
        while (cnt != 0) {
            cout << "1 - Изменить целую часть" << endl;
            cout << "2 - Изменить числитель" << endl;
            cout << "3 - Изменить знаменатель" << endl;
            cout << "0 - Закончить" << endl;
            cout << "Выберите действие = ";
            cin >> cnt;
            cout << endl;
            switch(cnt){
                case 1: {
                    cout << "Целая часть = ";
                    arr[0] = arr2[0];
                    cout << arr[0] << endl;
                    continue;
                }
                case 2: {
                    cout << "Числитель = ";
                    arr[1] = arr2[1];
                    cout << arr[1] << endl;
                    continue;
                }
                case 3: {
                    cout << "Знаменатель = ";
                    arr[2] = arr2[2];
                    cout << arr[2] << endl;
                    continue;
                }
                case 0: {
                    cout << "Завершено!" << endl << endl;
                    break;
                }
            }
        }
        return *this;
    }
    Fractions operator+ (Fractions &obj) {
        Fractions tmp;
        if (getDenominator() != obj.getDenominator()) {
            tmp.arr[1] = getNumerator() * obj.getDenominator() + obj.getNumerator() * getDenominator();
            tmp.arr[2] = getDenominator() * obj.getDenominator();
            tmp.arr[0] = 0;
            return tmp;
        }
        else {
            tmp.arr[1] = getNumerator() + obj.getNumerator();
            tmp.arr[2] = arr[2];
            tmp.arr[0] = 0;
            return tmp;
        }
    }
    Fractions operator- (Fractions &obj) {
        Fractions tmp;
        if (getDenominator() != obj.getDenominator()) {
            tmp.arr[0] = 0;
            tmp.arr[1] = getNumerator() * obj.getDenominator() - obj.getNumerator() * getDenominator();
            tmp.arr[2] = getDenominator() * obj.getDenominator();
            return tmp;
        }
        else {
            tmp.arr[1] = getNumerator() - obj.getNumerator();
            tmp.arr[2] = arr[2];
            tmp.arr[0] = 0;
            return tmp;
        }
    }
    Fractions operator* (Fractions &obj) {
        Fractions tmp;
        tmp.arr[1] = getNumerator() * obj.getNumerator();
        tmp.arr[2] = getDenominator() * obj.getDenominator();
        tmp.arr[0] = 0;
        return tmp;
    }
    bool operator< (Fractions &obj) {
        if ((getNumerator() / getDenominator()) < (obj.getNumerator() / obj.getDenominator())){
            return true;
        }
        else {
            return false;
        }
    }
    bool operator> (Fractions &obj) {
        if ((getNumerator() / getDenominator()) > (obj.getNumerator() / obj.getDenominator())){
            return true;
        }
        else {
            return false;
        }
    }
    bool operator>= (Fractions &obj) {
        if ((getNumerator() / getDenominator()) >= (obj.getNumerator() / obj.getDenominator())){
            return true;
        }
        else {
            return false;
        }
    }
    bool operator<= (Fractions &obj) {
        if ((getNumerator() / getDenominator()) <= (obj.getNumerator() / obj.getDenominator())){
            return true;
        }
        else {
            return false;
        }
    }
    bool operator== (Fractions &obj) {
        if ((getNumerator() / getDenominator()) == (obj.getNumerator() / obj.getDenominator())){
            return true;
        }
        else {
            return false;
        }
    }
    bool operator!= (Fractions &obj) {
        if ((getNumerator() / getDenominator()) != (obj.getNumerator() / obj.getDenominator())){
            return true;
        }
        else {
            return false;
        }
    }
    Fractions& operator= (Fractions &obj) {
        arr[0] = obj.arr[0];
        arr[1] = obj.arr[1];
        arr[2] = obj.arr[2];
        return *this;
    }
    friend istream& operator>> (istream& is, Fractions &obj);
    friend ostream& operator<< (ostream& os, Fractions &obj);
};
istream& operator>> (istream &is, Fractions &obj) {
    cout << "Введите целую часть = ";
    is >> obj.arr[0];
    cout << "Введите числитель = ";
    is >> obj.arr[1];
    cout << "Введите знаменатель = ";
    is >> obj.arr[2];
    cout << endl;
    return is;
}
ostream& operator<< (ostream &os, Fractions &obj) {
    os << "Целая часть = " << obj.arr[0] << endl;
    os << "Числитель = " << obj.arr[1] << endl;
    os << "Знаменатель = " << obj.arr[2] << endl;
    return os;
}
 
template <typename T>
class dynArrPoint {
    private:
        T **dap;
        int size;
    public:
        dynArrPoint(int s): dap(new T*[size]) {
            if (s > 0) {size = s;}
            else {size = 10;}
            for (int i = 0; i < 20; i++){
                dap[i] = 0;
            }
        }
        ~dynArrPoint() {delete []dap;}
 
        void idAdd (int i, T *elem) {
            if (dap[i] == NULL){
                dap[i] = elem;
            }
        }
        void idDelete (int i) {
            dap[i] = NULL;
        }
        void getData() {
            for (int i = 0; i < size; i++) {
                if (dap[i] != NULL){
                    cout << *dap[i] << endl;
                }
                else {
                    cout << i << " - " << dap[i] << endl;
                }
            }
        }
        void maxSearch() {
            T max;
            max = *dap[0];
            for (int i = 1; i < size; i++) {
                if (dap[i] == NULL){continue;}
                else if (max > *dap[i]) {
                    max = *dap[i];
                }
            }
            cout << endl << "-----Максимум-----" << endl << max;
            cout << "------------------" << endl << endl;
        }
        void minSearch() {
            T min;
            min = *dap[0];
            for (int i = 1; i < size; i++) {
                if (dap[i] == NULL){continue;}
                else if (min > *dap[i]) {
                    min = *dap[i];
                }
            }
            cout << endl << "-----Минимум-----" << endl << min;
            cout << "-----------------" << endl << endl;
        }
        void selectionSort(){
            cout << "-----Сортировка-----" <<endl;
            for (int i = 0; i < size - 1; i++) {
                for (int j = i+1; j < size; j++){
                    if (dap[i] > dap[j]) {
                        T *minElem = dap[i];
                        dap[i] = dap[j];
                        dap[j] = minElem;
                    }
                }
            }
            for (int i = 1; i < size - 1; i++) {
                for (int j = i+1; j < size; j++){
                    if (*dap[i] > *dap[j]) {
                        T *minElem = dap[i];
                        dap[i] = dap[j];
                        dap[j] = minElem;
                    }
                }
            }
        }
 
};
 
int main() {
    setlocale(LC_ALL, "Rus");
    int arr[3] = {0, 1, 2};
    int arr2[3] = {0, 2, 2};
    int arr3[3] = {0, 1, 7};
    int arr5[3] = {1, 20, 10};
 
    Fractions *objPointer1;
    Fractions obj1(arr);
    objPointer1 = &obj1;
 
    Fractions *objPointer2;
    Fractions obj2(arr2);
    objPointer2 = &obj2;
 
    Fractions *objPointer3;
    Fractions obj3(arr3);
    objPointer3 = &obj3;
 
    dynArrPoint <Fractions> arrObj(4);
    arrObj.idAdd(0, objPointer1);
    arrObj.idAdd(1, objPointer2);
    arrObj.idAdd(2, objPointer3);
 
    Fractions obj5(arr5);
    Fractions *objPtr;
    objPtr = &obj5;
 
    arrObj.getData();
    arrObj.maxSearch();
    arrObj.getData();
    arrObj.minSearch();
    arrObj.getData();
    arrObj.selectionSort();
    arrObj.getData();
 
    system("pause");
    return 0;
}

nd2 · 15.12.2016, 04:27

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

это уже успел увидеть.

И сортировка у тебя кривая.

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

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

"Как загрузить из файла объект в динамический массив указателей???"

Сохраняешь в файле данные объекта. Создаёшь объект, инициализируешь его данным из файла, адрес объекта помещаешь в массив указателей.

@Blekzet · 15.12.2016, 04:33 **[ТС]**

nd2, понятно, что сортировка кривая... просто этот NULLменя доконал и я так сделал.
А про создание объекта, нужно сделать отдельный метод ВНУТРИ класса, что меня и сбивает... Объект то как бы после завершения метода пуф и удаляется...

nd2 · 15.12.2016, 04:45

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

Объект то как бы после завершения метода пуф и удаляется...

Если в динамической памяти создаётся (или в статической), то не удаляется.

Добавлено через 3 минуты
Про объекты Fractions речь ведёшь?

@Blekzet · 15.12.2016, 04:48 **[ТС]**

nd2, то есть внутри метода нужно создать динамический объект, в него загрузить данные и по ссылке передать в массив указателей?

Добавлено через 2 минуты
nd2, да, про Fractions.

nd2 · 15.12.2016, 05:12

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

внутри метода

Речь о методе твоего шаблонного класса?

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

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

Ссылка тут не к чему (или о чём ты?). Просто поместить адрес в массив указателей.

@Blekzet · 15.12.2016, 05:13 **[ТС]**

nd2,

Речь о методе твоего шаблонного класса?

Именно

nd2 · 15.12.2016, 05:14

Если предполагается делать такой метод в dynArrPoint, то нужно там будет деструктор изменить: освобождать память не только под указателем на указатель, но и (в цикле) под указателями в массиве указателей. И тогда не делать так, как у тебя в main(), т.е., не создавать объекты Fractions в автомитической памяти, только в динамической.

@Blekzet · 15.12.2016, 05:24 **[ТС]**

nd2, вот так думаю будет проще:

Кликните здесь для просмотра всего текста

Разработать шаблоны структуры данных в памяти. Тип хранимого элемента – параметр шаблона. Шаблон может содержать указатель на объект, либо сам объект (в зависимости от варианта). Реализовать операции включения/удаления по заданному логическому номеру, поиск минимального/максимального, включение с сохранением порядка, сортировки выбором или вставками, загрузки хранимых элементов в структуру данных из последовательного потока. Проверить работу шаблона на разработанном классе, используя его как параметр шаблона. Структуры данных:

Динамический массив указателей на объекты;

если прям конкретно по вопросам, то:
1)Что значит логический номер
2)Что от меня требует вот это "загрузки хранимых элементов в структуру данных из последовательного потока" (это файл или консоль?)

Добавлено через 6 минут
Про логический номер, я думаю, что это тот же индекс как и в обычном массиве, просто если в обычном массиве мы заносим переменную в ячейку памяти массива, то тут просто будет указатель на ячейку памяти которую мы до этого заполнили созданием элемента (допустим int a = 10).

nd2 · 15.12.2016, 05:29

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

вот так думаю будет проще:

Нет, не проще.

@Blekzet · 15.12.2016, 05:32 **[ТС]**

nd2, согласен, задание составлено с помощью головы, клавиатуры и гравитации, но все же...

nd2 · 15.12.2016, 05:38

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

1)Что значит логический номер

Предположу, что не порядковый: в массивах с++ индексы не совпадают с порядковыми номерами (хотя, причём здесь "логический"?)

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

2)Что от меня требует вот это "загрузки хранимых элементов в структуру данных из последовательного потока" (это файл или консоль?)

И то, и то подходит (на выбор).

@Blekzet · 15.12.2016, 05:47 **[ТС]**

nd2,
Вот такой деструктор?

C++

~dynArrPoint() {
            for (int i=0; i<size; i++) {
                delete dap[i];
            }
            delete [] dap;
        }

А вот создание и добавление переменной.

C++

void idAdd (int i) {
            T *elem = new T;
 
            if (dap[i] == NULL){
                dap[i] = elem;
                nullCnt--;
            }
        }

nd2 · 15.12.2016, 05:52

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

Вот такой деструктор?

Да.

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

А вот создание, допустим переменной.

Да.

@Blekzet 4 / 4 / 0 Регистрация: 21.12.2015 Сообщений: 195
	15.12.2016, 04:33 [ТС]	22
	nd2, понятно, что сортировка кривая... просто этот NULLменя доконал и я так сделал. А про создание объекта, нужно сделать отдельный метод ВНУТРИ класса, что меня и сбивает... Объект то как бы после завершения метода пуф и удаляется... 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	15.12.2016, 04:45	23
	Сообщение от Blekzet Объект то как бы после завершения метода пуф и удаляется... Если в динамической памяти создаётся (или в статической), то не удаляется. Добавлено через 3 минуты Про объекты Fractions речь ведёшь? 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	15.12.2016, 05:12	25
	Сообщение от Blekzet внутри метода Речь о методе твоего шаблонного класса? Сообщение от Blekzet внутри метода нужно создать динамический объект, в него загрузить данные и по ссылке передать в массив указателей? Ссылка тут не к чему (или о чём ты?). Просто поместить адрес в массив указателей. 0

@Blekzet 4 / 4 / 0 Регистрация: 21.12.2015 Сообщений: 195
	15.12.2016, 05:13 [ТС]	26
	nd2, Речь о методе твоего шаблонного класса? Именно 0

@Blekzet 4 / 4 / 0 Регистрация: 21.12.2015 Сообщений: 195
	15.12.2016, 05:24 [ТС]	28
	nd2, вот так думаю будет проще: Кликните здесь для просмотра всего текста Разработать шаблоны структуры данных в памяти. Тип хранимого элемента – параметр шаблона. Шаблон может содержать указатель на объект, либо сам объект (в зависимости от варианта). Реализовать операции включения/удаления по заданному логическому номеру, поиск минимального/максимального, включение с сохранением порядка, сортировки выбором или вставками, загрузки хранимых элементов в структуру данных из последовательного потока. Проверить работу шаблона на разработанном классе, используя его как параметр шаблона. Структуры данных: Динамический массив указателей на объекты; если прям конкретно по вопросам, то: 1)Что значит логический номер 2)Что от меня требует вот это "загрузки хранимых элементов в структуру данных из последовательного потока" (это файл или консоль?) Добавлено через 6 минут Про логический номер, я думаю, что это тот же индекс как и в обычном массиве, просто если в обычном массиве мы заносим переменную в ячейку памяти массива, то тут просто будет указатель на ячейку памяти которую мы до этого заполнили созданием элемента (допустим int a = 10). 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	15.12.2016, 04:27	21
	Сообщение от Blekzet это уже успел увидеть. И сортировка у тебя кривая. Добавлено через 3 минуты Сообщение от Blekzet "Как загрузить из файла объект в динамический массив указателей???" Сохраняешь в файле данные объекта. Создаёшь объект, инициализируешь его данным из файла, адрес объекта помещаешь в массив указателей. 0

@Blekzet 4 / 4 / 0 Регистрация: 21.12.2015 Сообщений: 195
	15.12.2016, 04:48 [ТС]	24
	nd2, то есть внутри метода нужно создать динамический объект, в него загрузить данные и по ссылке передать в массив указателей? Добавлено через 2 минуты nd2, да, про Fractions. 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	15.12.2016, 05:14	27
	Если предполагается делать такой метод в dynArrPoint, то нужно там будет деструктор изменить: освобождать память не только под указателем на указатель, но и (в цикле) под указателями в массиве указателей. И тогда не делать так, как у тебя в main(), т.е., не создавать объекты Fractions в автомитической памяти, только в динамической. 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	15.12.2016, 05:29	29
	Сообщение от Blekzet вот так думаю будет проще: Нет, не проще. 0

@Blekzet 4 / 4 / 0 Регистрация: 21.12.2015 Сообщений: 195
	15.12.2016, 05:32 [ТС]	30
	nd2, согласен, задание составлено с помощью головы, клавиатуры и гравитации, но все же... 1

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	15.12.2016, 05:38	31
	Сообщение от Blekzet 1)Что значит логический номер Предположу, что не порядковый: в массивах с++ индексы не совпадают с порядковыми номерами (хотя, причём здесь "логический"?) Сообщение от Blekzet 2)Что от меня требует вот это "загрузки хранимых элементов в структуру данных из последовательного потока" (это файл или консоль?) И то, и то подходит (на выбор). 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	15.12.2016, 05:52	33
	Сообщение было отмечено Blekzet как решение Решение Сообщение от Blekzet Вот такой деструктор? Да. Сообщение от Blekzet А вот создание, допустим переменной. Да. 0

Ошибка в программе

Решение