Сортировка элементов массива объектов в классе

@sv1tlana · Регистрация: 01.10.2015

Здравствуйте! Помогите пожалуйста найти ошибку в коде - я уже 2 дня над ней бьюсь...
Необходимо реализовать программу Телефонный справочник с использованием классов и деструкторов (это первое ДЗ по теме "Введение в ООП").
Программа содержит методы добавления/удаления, печати, поиска и сортировки записей.
Класс Telefon - содержит инфо о полях каждой отдельной записи (абоненте). А класс Tel_kniga содержит массив объектов класса Telefon.
Проблема возникла с функцией сортировки Sort_fio (). Без использования деструктора ~Telefon () - все работает.
А вот с деструктором возникает ошибка: HEAP CORRUPTION DETECTED: after Normal block (#153).
Заранее благодарна за помощь и любые замечания по программе.
Вот код:

C++

// Класс телефонная книга
 
#include <iostream>
#include <windows.h>
#include <string.h>
#include <conio.h>
#include <stdlib.h>
using namespace std;
class Telefon
{   
    char *Fio;
    char t_dom [14];
    char t_rab [14];
    char t_mob [14];
    char *dop_inf;
public:
    
    Telefon ()
    {
        //cout << "\n Констр 0\n";
    }
    Telefon (char *F, char *t_d,  char *t_r, char *t_m, char *dop)
    {
        //cout << "\n Констр с пар-ми\n";
        Fio = new char [strlen(F)];
        strcpy (Fio, F);
 
        strcpy (t_dom, t_d);
        strcpy (t_rab, t_r);
        strcpy (t_mob, t_m);
 
        dop_inf = new char [strlen(dop)];
        strcpy (dop_inf, dop);
    }
    Telefon (const Telefon &obj) // Конструктор копирования
    {
        cout << "\n Конструктор Telefon копирования \n";
        Fio = new char [strlen(obj.Fio)];
        strcpy (Fio, obj.Fio);
        
        strcpy (t_dom, obj.t_dom);
        strcpy (t_rab, obj.t_rab);
        strcpy (t_mob, obj.t_mob);
        
        dop_inf = new char [strlen(obj.dop_inf)];
        strcpy (dop_inf, obj.dop_inf);
    }
    ~Telefon ()
    {
        cout << "\n Деструктор ФИО \n";
        delete [] Fio;
        Fio = NULL;
        cout << "\n Деструктор доп. инф. \n";
        delete [] dop_inf;
        dop_inf = NULL;
    }
    void Vvod ()
    {
        int maxlen = 30;
        char *STRtmp = new char [100]; // Создаем указатель на вспомогат. массив char для ввода текстовых строк
        int len_STRtmp=0; // длина вспомогат. строки
        {
            cout << "\n  Введите ФИО абонента (max " << maxlen << " символов): ";
            gets (STRtmp);
            len_STRtmp = strlen (STRtmp);
            if (len_STRtmp > maxlen)
                cout << "\n   Введенный текст превышает " << maxlen << " символов! Повторите ввод!: ";
        } while (len_STRtmp > maxlen);
        Fio = new char [len_STRtmp+1];
        strcpy (Fio, STRtmp);
        delete [] STRtmp;
        STRtmp = NULL;
 
        cout << "\n  Введите домашний телефон: ";
        gets (t_dom);
        cout << "\n  Введите рабочий телефон: ";
        gets (t_rab);
        cout << "\n  Введите мобильный телефон: ";
        gets (t_mob);
 
        char *STRtmp2 = new char [100] ; // Создаем указатель на вспомогат. массив char для ввода текстовых строк
        int len_STRtmp2=0; // длина вспомогат. строки
        do
        {
            cout << "\n  Введите дополнительную информацию (max " << maxlen << " символов): ";
            gets (STRtmp2);
            len_STRtmp2 = strlen (STRtmp2);
            if (len_STRtmp2 > maxlen)
                cout << "\n   Введенный текст превышает " << maxlen << " символов! Повторите ввод!: ";
        } while (len_STRtmp2 > maxlen);
        dop_inf = new char [len_STRtmp2+1];
        strcpy (dop_inf, STRtmp2);
        delete [] STRtmp2;
        STRtmp2 = NULL;
    }
    void Show ()
    {
        cout << Fio << "\t" << t_dom << "\t" << t_rab << "\t" << t_mob << "\t" << dop_inf << "\n";
    }   
    // Функции доступа к переменным private класса Telefon
    char* GetFio ()
    {   return Fio;
    }
 
    char* Get_t_dom ()
    {   return t_dom;
    }
 
    char* Get_t_rab ()
    {   return t_rab;
    }
 
    char* Get_t_mob ()
    {   return t_mob;
    }
    
    char* Getdop_inf ()
    {   return dop_inf;
    }
    char* Set_Fio (char* STRrnew)
    {
        Fio = new char [strlen(STRrnew)];
        strcpy (Fio, STRrnew);
        return Fio;
    }
    char* Set_dop_inf (char* STRrnew)
    {
        dop_inf = new char [strlen(STRrnew)];
        strcpy (dop_inf, STRrnew);
        return dop_inf;
    }
};
class Tel_kniga
{
    char Nazv [21];
    Telefon *Catalog;
public:
    int n; // кол-во записей в тел. книге
    Tel_kniga (char *name)
    {
        //cout << "\n Констр Tel_kniga с параметром \n";
        strcpy (Nazv, name);
        n=0; // кол-во записей в тел. книге
        Catalog = NULL;
    }
    void Copy_Catalog (Telefon *Catalog_begin, int n0)
    {
        Catalog = new Telefon [n0];
        for (int i = 0; i < n0; i++)
            Catalog[i] = Catalog_begin [i];
        n=n0;
    }
    void Add()
    {
        Telefon *New = new Telefon [n + 1];
        for (int i = 0; i < n; i++)
        {
            New[i] = Catalog [i];
        }
        New[n].Vvod();
        n++;
        Catalog=New;
    }
    void Del(char *f_abon)
    {
        int k = 0;
        for (int i = 0; i < n; i++)
        {
            if (stricmp(Catalog[i].GetFio(), f_abon) == 0)
            {
                k++;
            }
        }
        Telefon *New = new Telefon [n - k];
        for (int i = 0, j = 0; i < n; i++)
        {
            if (stricmp(Catalog[i].GetFio(), f_abon) != 0)
            {
                New[j] = Catalog[i];
                j++;
            }
        }
        if (k == 0)
        {
            cout << "Нет совпадений.\n";
        }
        else
        {
            cout << "Удаление произошло.\n";
        }
        n = n - k;
        Catalog=New;
    }
    void PrintCatalog () //   Показ каталога записей
    {
        cout << "\n\t\t" << Nazv;
        cout << "\n\n№ п/п\t" << "ФИО\t\t" << "Домашний тел.\t" <<  "Рабочий тел.\t" << "Мобильный тел.\t" << "Доп. инфо \n\n";
        for (int i = 0; i < n; i++)
        {
            cout << i+1 <<"\t";
            Catalog[i].Show ();
        }
    }
    void Find_fio (char *f_abon) // Поиск п фамилии абонента
    {
        bool f = 0;
        for (int i = 0; i < n; i++)
        {
            if (stricmp(Catalog[i].GetFio(), f_abon) == 0)
            {
                f = 1;
                Catalog[i].Show ();
            }
        }
        if (f == 0)
        {
            cout << "Нет совпадений.\n";
        }
    }
    void Sort_fio ()
    {
        for (int i = 0; i < n; i++)
        {
            for (int j = 0; j < n - 1; j++)
            {
                if (stricmp(Catalog[j].GetFio(), Catalog[j + 1].GetFio())>0)
                {
                    Telefon tmp = Telefon (Catalog[j]);
                    Catalog[j] = Telefon (Catalog[j + 1]);
 
                    Catalog[j + 1] = Telefon (tmp);
                }
            }
        }
    }
    void Edit (char num)
    {
        char key;
            system("cls");
            cout << "\nМЕНЮ:\n";
            cout << "1-Редактирование фамилии абонента.\n";
            cout << "2-Редактирование домашнего телефона.\n";
            cout << "3-Редактирование рабочего телефона.\n";
            cout << "4-Редактирование мобильного телефона.\n";
            cout << "5-Редактирование доп. информации.\n";
            cout << "0-Выход.\n";
            cout << "\n  Выберите действие: ";
        do
        {
            key = getch();
            switch (key)
            {
            case '1':
            {
                char *s1 = new char [31]; // Создаем вспом. массив char для ввода новой фамилии
                do
                {
                    cout << "\n    Введите новую фамилию абонента (max 30 символов): ";
                    gets (s1);
                    if (strlen (s1)>30)
                        cout << "\n   Введенный текст превышает 30 символов! Повторите ввод!: ";
                } while (strlen (s1)>30);
                Catalog[num].Set_Fio(s1); // Вызов функции для изменения фамилии
                delete [] s1;
                cout << "\n  Фамилия изменена.";
                cout << "\n  Выберите новое поле для редактирования или нажмите '0' для выхода: ";
                Sleep(2000);
                break;
            }
            case '2':
                char s2[14];
                cout << "\n  Введите новый домашний телефон: ";
                gets (s2);
                strcpy(Catalog[num].Get_t_dom(),s2);
                cout << "\n  Телефон изменен.";
                cout << "\n  Выберите новое поле для редактирования или нажмите '0' для выхода: ";
                Sleep(2000);
                break;
            case '3':
                char s3[14];
                cout << "\n  Введите новый рабочий телефон: ";
                gets (s3);
                strcpy(Catalog[num].Get_t_rab(),s3);
                cout << "\n  Телефон изменен.";
                cout << "\n  Выберите новое поле для редактирования или нажмите '0' для выхода: ";
                Sleep(2000);
                break;
            case '4':
                char s4[14];
                cout << "\n  Введите новый мобильный телефон: ";
                gets (s4);
                strcpy(Catalog[num].Get_t_mob(),s4);
                cout << "\n  Телефон изменен.";
                cout << "\n  Выберите новое поле для редактирования или нажмите '0' для выхода: ";
                break;
            case '5':
            {
                char *s5 = new char [31]; // Создаем вспом. массив char для ввода новой фамилии
                do
                {
                    cout << "\n  Введите новую доп. информацию (max 30 символов): ";
                    gets (s5);
                    if (strlen (s5)>30)
                        cout << "\n   Введенный текст превышает 30 символов! Повторите ввод!: ";
                } while (strlen (s5)>30);
                Catalog[num].Set_dop_inf(s5); // Вызов функции для изменения фамилии
                delete [] s5;
                cout << "\n  Доп. информация изменена.";
                cout << "\n  Выберите новое поле для редактирования или нажмите '0' для выхода: ";
                Sleep(2000);
                break;
            }
            default:
                if (key!='0')
                {
                    cout << "\n  Неверный выбор! Повторите ввод:";
                    Sleep(2000);
                }
                break;
            }
        } while (key != '0');
    }
        ~Tel_kniga ()
    {
        cout << "\n Деструктор для удал. массива Catalog\n";
        delete [] Catalog;
        Catalog=NULL;
    }
};
void main ()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
 
    const int n0 = 3;
    Telefon Catalog_begin [n0]= {Telefon ("Иванов Владимир", "056-442-33-33", "056-442-22-22", "096-453-11-11", "В отпуске"), 
                                Telefon ("Петров Василий", "0564-92-33-33", "0564-95-22-22", "097-453-11-11", "Больничный"),
                                Telefon ("Абрамов Алексей", "0564-92-11-11", "0564-95-44-45", "097-453-22-22", "Отгул")};
    char a[21];
    cout << "\n  Введите название телефонной книги (макс. 20 символов): ";
    do
    {
        gets (a);
        if (strlen(a) > 20)
            cout << "\n   Введенный текст превышает 20 символов! Повторите ввод!: ";
    } while (strlen(a) > 20);
 
    Tel_kniga TEL = Tel_kniga (a);
    TEL.Copy_Catalog (Catalog_begin, n0);
    
    char ch;
    char str[31];
    do
    {
        system("cls");
        cout << "\n\nМЕНЮ:\n";
        cout << "1-Добавление записи\n";
        cout << "2-Удаление записи\n";
        cout << "3-Вывод базы записей.\n";
        cout << "4-Поиск абонента по фамилии.\n";
        cout << "5-Сортировка абонентов по фамилии.\n";
        cout << "6-Редактирование записи.\n";
        cout << "Esc-Выход.\n";
        ch = getch();
        switch (ch)
        {
        case '1':system("cls");
            TEL.Add ();
            cout << "Запись добавлена.\n";
            Sleep(2000);
            break;
        case '2':system("cls");
            if (TEL.n != 0)
            {
                cout << "Введите ФИО удаляемого абонента: ";
                gets(str);
                TEL.Del (str);
            }
            else
            {
                cout << "Каталог пуст.\n";
            }
            Sleep(2000);
            break;
        case '3':system("cls");
            if (TEL.n != 0)
            {
                TEL.PrintCatalog ();
                system("pause");
            }
            else
            {
                cout << "Каталог пуст.\n";
                Sleep(2000);
            }
            break;
        case '4':system("cls");
            if (TEL.n != 0)
            {
                cout << "Введите фамилию абонента: ";
                gets(str);
                TEL.Find_fio(str);
            }
            else
            {
                cout << "Каталог пуст.\n";
            }
            system("pause");
            break;
        case '5':system("cls");
            if (TEL.n != 0)
            {
                TEL.Sort_fio ();
                cout << "Сортировка произведена.\n";
            }
            else
            {
                cout << "Каталог пуст.\n";
            }
            Sleep(2000);
            break;
        case '6':system("cls");
            if (TEL.n != 0)
            {
                TEL.PrintCatalog ();
                int num;
                do
                {
                    cout << "\n\n  Выберите номер записи, в которую необходимо внести изменения: ";
                    cin >> num;
                } while (num<1 || num>TEL.n);
                cin.ignore();
                TEL.Edit (num-1);
            }
            else
            {
                cout << "Каталог пуст.\n";
            }
            Sleep(2000);
            break;
        }
    } while (ch != 27);
    cout<<"\n\n\n\t";
}

@nmcf · 31.05.2016, 10:13

Ты не там NULL присваиваешь. На кой чёрт это нужно в деструкторе, если объект уже уничтожается?

C++

    Telefon (): Fio(NULL), dop_inf(NULL)
    {
        //cout << "\n Констр 0\n";
    }

При выделении памяти под строку размер - (strlen() + 1).

@avgoor · 31.05.2016, 13:00

sv1tlana, У вас нарушение правила трех: вы не реализовали оператор присваивания.
В результате в 231 строке в tmp и Catalog[j + 1] есть указатели, которые указывают на одну и ту же память (Fio и dop_inf). При выходе из if в 232 строке для tmp вызывается деструктор, который удаляет все, что выделено для класса и в Catalog[j + 1] остаются "висячие указатели".

@sv1tlana · 31.05.2016, 17:54 **[ТС]**

avgoor, здравствуйте.
Я не могу понять почему происходит использование одной и той же памяти.
Ведь я присваиваю значения, используя конструктор копирования
(строка 35: Telefon (const Telefon &obj) )
Причем вызываю его принудительно...

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

@nmcf · 31.05.2016, 18:36

У тебя и без этого неправильно. Сначала конструктор и длину исправь.

@sv1tlana · 31.05.2016, 18:47 **[ТС]**

nmcf, здравствуйте.
Спасибо, что уделили внимание, но к сожалению, я только больше запуталась.
Не могли бы Вы уточнить какой именно конструктор?
Тот, который по умолчанию (строка 18)? И как его исправить?
И где надо исправить длину?

@nmcf · 31.05.2016, 19:22

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

И как его исправить?

Я же показал как. Просто замени.
Длину везде исправить, где память выделяешь через new.

@sv1tlana · 31.05.2016, 21:18 **[ТС]**

avgoor, я пыталась самостоятельно разобраться - стало "вылетать" чуть позже, но ошибка остается все равно:
Вот что я изменила:

1) Добавила конструктор присваивания копированием, согласно правилу трех:

C++

Telefon & operator = (const Telefon &obj) // Перегрузка операции присваивания копированием
    {
        cout << "\n Перегрузка операции присваивания копированием \n";
        if (this != &obj) // защита от неправильного самоприсваивания
        {
            char *new_Fio = new char[strlen(obj.Fio)];
            strcpy (new_Fio, obj.Fio);
            // delete [] Fio;
             Fio = new_Fio;
 
            strcpy (t_dom, obj.t_dom);
            strcpy (t_rab, obj.t_rab);
            strcpy (t_mob, obj.t_mob);
 
            char *new_dop_inf = new char[strlen(obj.dop_inf)];
            strcpy (new_dop_inf, obj.dop_inf);
            // delete [] dop_inf;
            dop_inf=new_dop_inf;
        }
        // по соглашению всегда возвращаем *this
        return *this;
    }

2) Подкорректировала деструктор чтобы удобнее было анализировать

C++

 ~Telefon ()
    {
    cout << "\n\n  Адрес Fio =" << &Fio;
    cout << "\n Адрес dop_inf =" << &dop_inf;
                         
        cout << "\n Деструктор ФИО \n";
    if (Fio)
    delete [] Fio;
// Fio = NULL;
        cout << "\n Деструктор доп. инф. \n";
        if (dop_inf)
         delete [] dop_inf;
        // dop_inf = NULL;
    }

3) Немного подкорректировала функцию сортировки, (в т.ч. вывожу на экран адреса указателей, чтобы отследить что происходит с памятью)

C++

 // Функция сортировки
    void Sort_fio ()
    {
        for (int i = 0; i < n; i++)
        {
            for (int j = 0; j < n - 1; j++)
            {
                if (stricmp(Catalog[j].GetFio(), Catalog[j + 1].GetFio())>0)
                {
                    Telefon tmp = Catalog[j];
                    Catalog[j] =  Catalog[j + 1];
                    Catalog[j + 1] = tmp;
                    
                    cout << "\n Адрес tmp =" << &tmp;
                    cout << "\n Адрес Catalog[j] =" << &Catalog[j];
                    cout << "\n Адрес Catalog[j+1] =" << &Catalog[j+1];
                }
            }
        }
    }

4) Исправила конструктор без параметров - пыталась учесть замечание от nmcf.

C++

Telefon ()
    {
        Fio=NULL;
        dop_inf=NULL;
       
    }

@nmcf · 31.05.2016, 21:20

Так, как у меня, делай, а не через равно NULL.
И где корректировка длины?

@avgoor · 31.05.2016, 22:12

sv1tlana, 1) В операторе присваивания, если выделяете новую память, старую надо удалать раскомментируйте. 2) Когда выделяете память не забывайте о нуль-терминаторе (new char[strlen(...)+1])

@sv1tlana · 01.06.2016, 00:22 **[ТС]**

nmcf и avgoor, СПАСИБО ВАМ ОГРОМНОЕ! ЗАРАБОТАЛО!
Дело действительно было в том, что я неправильно устанавливала длину строки и не использовала конструктор присваивания копированием. Не верю своему счастью

Не подскажете еще один момент: при сортировке - сортируются только фамилии с большой буквы, хотя используется функция stricmp, которая сравнивает без учета регистра символов.

Вот исправленный код:

C++

// Класс телефонная книга
#include <iostream>
#include <windows.h>
#include <string.h>
#include <conio.h>
#include <stdlib.h>
using namespace std;
class Telefon
{   
    char *Fio;
    char t_dom [14];
    char t_rab [14];
    char t_mob [14];
    char *dop_inf;
public:
    Telefon (); // Конструктор без параметров
    Telefon (char *, char *,  char *, char *, char *); // Конструктор с параметрами
    Telefon (const Telefon &); // Конструктор копирования
    Telefon & operator = (const Telefon &); // Перегрузка операции присваивания копированием
    ~Telefon ();
    void Vvod ();
    void Show ();
    char* GetFio ();
    char* Get_t_dom ();
    char* Get_t_rab ();
    char* Get_t_mob ();
    char* Getdop_inf ();
    char* Set_Fio (char*);
    char* Set_dop_inf (char*);
};
class Tel_kniga
{
    char Nazv [21];
    Telefon *Catalog;
    Telefon tmp;
public:
    int n; // кол-во записей в тел. книге
    Tel_kniga (char *);
    void Copy_Catalog (Telefon *, int);
    void Add();
    void Del(char *);
    void PrintCatalog (); //   Показ каталога записей
    void Find_fio (char *); // Поиск по фамилии абонента
    void Sort_fio ();
    void Edit (char);
    ~Tel_kniga ();
};
void main ()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
 
    const int n0 = 4;
    Telefon Catalog_begin [n0]= {Telefon ("Иванов Владимир", "056-442-33-33", "056-442-22-22", "096-453-11-11", "В отпуске"), 
        Telefon ("Петров Василий", "0564-92-33-33", "0564-95-22-22", "097-453-11-11", "Больничный"),
        Telefon ("грачев Петр", "0564-92-33-33", "0564-95-22-22", "097-453-11-11", "Больничный"),
        Telefon ("Абрамов Алексей", "0564-92-11-11", "0564-95-44-45", "097-453-22-22", "Отгул")};
    char a[21];
    cout << "\n  Введите название телефонной книги (макс. 20 символов): ";
    do
    {
        gets (a);
        if (strlen(a) > 20)
            cout << "\n   Введенный текст превышает 20 символов! Повторите ввод!: ";
    } while (strlen(a) > 20);
 
    Tel_kniga TEL = Tel_kniga (a);
    TEL.Copy_Catalog (Catalog_begin, n0);
 
    char ch;
    char str[31];
    do
    {
        system("cls");
        cout << "\n\nМЕНЮ:\n";
        cout << "1-Добавление записи\n";
        cout << "2-Удаление записи\n";
        cout << "3-Вывод базы записей.\n";
        cout << "4-Поиск абонента по фамилии.\n";
        cout << "5-Сортировка абонентов по фамилии.\n";
        cout << "6-Редактирование записи.\n";
        cout << "Esc-Выход.\n";
        ch = getch();
        switch (ch)
        {
        case '1':system("cls");
            TEL.Add ();
            cout << "Запись добавлена.\n";
            Sleep(2000);
            break;
        case '2':system("cls");
            if (TEL.n != 0)
            {
                cout << "Введите ФИО удаляемого абонента: ";
                gets(str);
                TEL.Del (str);
            }
            else
            {
                cout << "Каталог пуст.\n";
            }
            Sleep(2000);
            break;
        case '3':system("cls");
            if (TEL.n != 0)
            {
                TEL.PrintCatalog ();
                system("pause");
            }
            else
            {
                cout << "Каталог пуст.\n";
                Sleep(2000);
            }
            break;
        case '4':system("cls");
            if (TEL.n != 0)
            {
                cout << "Введите фамилию абонента: ";
                gets(str);
                TEL.Find_fio(str);
            }
            else
            {
                cout << "Каталог пуст.\n";
            }
            system("pause");
            break;
        case '5':system("cls");
            if (TEL.n != 0)
            {
                TEL.Sort_fio ();
                cout << "Сортировка произведена.\n";
            }
            else
            {
                cout << "Каталог пуст.\n";
            }
            Sleep(2000);
            break;
        case '6':system("cls");
            if (TEL.n != 0)
            {
                TEL.PrintCatalog ();
                int num;
                do
                {
                    cout << "\n\n  Выберите номер записи, в которую необходимо внести изменения: ";
                    cin >> num;
                } while (num<1 || num>TEL.n);
                cin.ignore();
                TEL.Edit (num-1);
            }
            else
            {
                cout << "Каталог пуст.\n";
            }
            Sleep(2000);
            break;
        }
    } while (ch != 27);
    cout<<"\n\n\n\t";
}
Telefon::Telefon () // Конструктор без параметров
{
    Fio=NULL;
    dop_inf=NULL;
}
Telefon::Telefon (char *F, char *t_d,  char *t_r, char *t_m, char *dop) // Конструктор с параметрами
{
    Fio = new char [strlen(F)+1];
    strcpy (Fio, F);
 
    strcpy (t_dom, t_d);
    strcpy (t_rab, t_r);
    strcpy (t_mob, t_m);
 
    dop_inf = new char [strlen(dop)+1];
    strcpy (dop_inf, dop);
}
Telefon::Telefon (const Telefon &obj) // Конструктор копирования
{
    Fio = new char [strlen(obj.Fio)+1];
    strcpy (Fio, obj.Fio);
 
    strcpy (t_dom, obj.t_dom);
    strcpy (t_rab, obj.t_rab);
    strcpy (t_mob, obj.t_mob);
 
    dop_inf = new char [strlen(obj.dop_inf)+1];
    strcpy (dop_inf, obj.dop_inf);
}
Telefon&Telefon:: operator = (const Telefon &obj) // Перегрузка операции присваивания копированием
{
    if (this != &obj) // защита от неправильного самоприсваивания
    {
        char *new_Fio = new char[strlen(obj.Fio)+1];
        strcpy (new_Fio, obj.Fio);
        delete [] Fio;
        Fio = new_Fio;
 
        strcpy (t_dom, obj.t_dom);
        strcpy (t_rab, obj.t_rab);
        strcpy (t_mob, obj.t_mob);
 
        char *new_dop_inf = new char[strlen(obj.dop_inf)+1];
        strcpy (new_dop_inf, obj.dop_inf);
        delete [] dop_inf;
        dop_inf=new_dop_inf;
    }
    return *this;
}
Telefon:: ~Telefon ()
{
    if (Fio)
    {
        delete [] Fio;
        Fio = NULL;
    }
    if (dop_inf)
    {
        delete [] dop_inf;
        dop_inf = NULL;
    }
}
void Telefon:: Vvod ()
{
    int maxlen = 30;
    char *STRtmp = new char [100]; // Создаем указатель на вспомогат. массив char для ввода текстовых строк
    int len_STRtmp=0; // длина вспомогат. строки
    {
        cout << "\n  Введите ФИО абонента (max " << maxlen << " символов): ";
        gets (STRtmp);
        len_STRtmp = strlen (STRtmp);
        if (len_STRtmp > maxlen)
            cout << "\n   Введенный текст превышает " << maxlen << " символов! Повторите ввод!: ";
    } while (len_STRtmp > maxlen);
    Fio = new char [len_STRtmp+1];
    strcpy (Fio, STRtmp);
    delete [] STRtmp;
    STRtmp = NULL;
 
    cout << "\n  Введите домашний телефон: ";
    gets (t_dom);
    cout << "\n  Введите рабочий телефон: ";
    gets (t_rab);
    cout << "\n  Введите мобильный телефон: ";
    gets (t_mob);
 
    char *STRtmp2 = new char [100] ; // Создаем указатель на вспомогат. массив char для ввода текстовых строк
    int len_STRtmp2=0; // длина вспомогат. строки
    do
    {
        cout << "\n  Введите дополнительную информацию (max " << maxlen << " символов): ";
        gets (STRtmp2);
        len_STRtmp2 = strlen (STRtmp2);
        if (len_STRtmp2 > maxlen)
            cout << "\n   Введенный текст превышает " << maxlen << " символов! Повторите ввод!: ";
    } while (len_STRtmp2 > maxlen);
    dop_inf = new char [len_STRtmp2+1];
    strcpy (dop_inf, STRtmp2);
    delete [] STRtmp2;
    STRtmp2 = NULL;
}
void Telefon:: Show ()
{
    cout << Fio << "\t" << t_dom << "\t" << t_rab << "\t" << t_mob << "\t" << dop_inf << "\n";
}   
// Функции доступа к переменным private класса Telefon
char* Telefon:: GetFio ()
{   return Fio;
}
char* Telefon:: Get_t_dom ()
{   return t_dom;
}
char* Telefon:: Get_t_rab ()
{   return t_rab;
}
char* Telefon:: Get_t_mob ()
{   return t_mob;
}
char* Telefon:: Getdop_inf ()
{   return dop_inf;
}
char* Telefon:: Set_Fio (char* STRrnew)
{
    Fio = new char [strlen(STRrnew)+1];
    strcpy (Fio, STRrnew);
    return Fio;
}
char*  Telefon:: Set_dop_inf (char* STRrnew)
{
    dop_inf = new char [strlen(STRrnew)+1];
    strcpy (dop_inf, STRrnew);
    return dop_inf;
}
Tel_kniga:: Tel_kniga (char *name)
{
    //cout << "\n Констр Tel_kniga с параметром \n";
    strcpy (Nazv, name);
    n=0; // кол-во записей в тел. книге
    Catalog = NULL;
}
void Tel_kniga:: Copy_Catalog (Telefon *Catalog_begin, int n0)
{
    Catalog = new Telefon [n0];
    for (int i = 0; i < n0; i++)
        Catalog[i] = Catalog_begin [i];
    n=n0;
}
void Tel_kniga:: Add()
{
    Telefon *New = new Telefon [n + 1];
    for (int i = 0; i < n; i++)
    {
        New[i] = Catalog [i];
    }
    New[n].Vvod();
    n++;
    Catalog=New;
}
void Tel_kniga:: Del(char *f_abon)
{
    int k = 0;
    for (int i = 0; i < n; i++)
    {
        if (stricmp(Catalog[i].GetFio(), f_abon) == 0)
        {
            k++;
        }
    }
    Telefon *New = new Telefon [n - k];
    for (int i = 0, j = 0; i < n; i++)
    {
        if (stricmp(Catalog[i].GetFio(), f_abon) != 0)
        {
            New[j] = Catalog[i];
            j++;
        }
    }
    if (k == 0)
    {
        cout << "Нет совпадений.\n";
    }
    else
    {
        cout << "Удаление произошло.\n";
    }
    n = n - k;
    Catalog=New;
}
void Tel_kniga:: PrintCatalog () //   Показ каталога записей
{
    cout << "\n\t\t Телефонная книга "" << Nazv;
    cout << ""\n\n№ п/п\t" << "ФИО\t\t" << "Домашний тел.\t" <<  "Рабочий тел.\t" << "Мобильный тел.\t" << "Доп. инфо \n\n";
    for (int i = 0; i < n; i++)
    {
        cout << i+1 <<"\t";
        Catalog[i].Show ();
    }
}
void Tel_kniga:: Find_fio (char *f_abon) // Поиск по фамилии абонента
{
    bool f = 0;
    for (int i = 0; i < n; i++)
    {
        if (stricmp(Catalog[i].GetFio(), f_abon) == 0)
        {
            f = 1;
            Catalog[i].Show ();
        }
    }
    if (f == 0)
    {
        cout << "Нет совпадений.\n";
    }
}
void Tel_kniga:: Sort_fio ()
{
    for (int i = 0; i < n; i++)
    {
        for (int j = 0; j < n - 1; j++)
        {
            if (stricmp(Catalog[j].GetFio(), Catalog[j + 1].GetFio())>0)
            {
                Telefon tmp = Catalog[j];
                Catalog[j] =  Catalog[j + 1];
                Catalog[j + 1] = tmp;
            }
        }
    }
}
void Tel_kniga:: Edit (char num)
{
    char key;
    system("cls");
    cout << "\nМЕНЮ:\n";
    cout << "1-Редактирование фамилии абонента.\n";
    cout << "2-Редактирование домашнего телефона.\n";
    cout << "3-Редактирование рабочего телефона.\n";
    cout << "4-Редактирование мобильного телефона.\n";
    cout << "5-Редактирование доп. информации.\n";
    cout << "0-Выход.\n";
    cout << "\n  Выберите действие: ";
    do
    {
        key = getch();
        switch (key)
        {
        case '1':
            {
                char *s1 = new char [31]; // Создаем вспом. массив char для ввода новой фамилии
                do
                {
                    cout << "\n    Введите новую фамилию абонента (max 30 символов): ";
                    gets (s1);
                    if (strlen (s1)>30)
                        cout << "\n   Введенный текст превышает 30 символов! Повторите ввод!: ";
                } while (strlen (s1)>30);
                Catalog[num].Set_Fio(s1); // Вызов функции для изменения фамилии
                delete [] s1;
                cout << "\n  Фамилия изменена.";
                cout << "\n  Выберите новое поле для редактирования или нажмите '0' для выхода: ";
                Sleep(2000);
                break;
            }
        case '2':
            char s2[14];
            cout << "\n  Введите новый домашний телефон: ";
            gets (s2);
            strcpy(Catalog[num].Get_t_dom(),s2);
            cout << "\n  Телефон изменен.";
            cout << "\n  Выберите новое поле для редактирования или нажмите '0' для выхода: ";
            Sleep(2000);
            break;
        case '3':
            char s3[14];
            cout << "\n  Введите новый рабочий телефон: ";
            gets (s3);
            strcpy(Catalog[num].Get_t_rab(),s3);
            cout << "\n  Телефон изменен.";
            cout << "\n  Выберите новое поле для редактирования или нажмите '0' для выхода: ";
            Sleep(2000);
            break;
        case '4':
            char s4[14];
            cout << "\n  Введите новый мобильный телефон: ";
            gets (s4);
            strcpy(Catalog[num].Get_t_mob(),s4);
            cout << "\n  Телефон изменен.";
            cout << "\n  Выберите новое поле для редактирования или нажмите '0' для выхода: ";
            break;
        case '5':
            {
                char *s5 = new char [31]; // Создаем вспом. массив char для ввода новой фамилии
                do
                {
                    cout << "\n  Введите новую доп. информацию (max 30 символов): ";
                    gets (s5);
                    if (strlen (s5)>30)
                        cout << "\n   Введенный текст превышает 30 символов! Повторите ввод!: ";
                } while (strlen (s5)>30);
                Catalog[num].Set_dop_inf(s5); // Вызов функции для изменения фамилии
                delete [] s5;
                cout << "\n  Доп. информация изменена.";
                cout << "\n  Выберите новое поле для редактирования или нажмите '0' для выхода: ";
                Sleep(2000);
                break;
            }
        default:
            if (key!='0')
            {
                cout << "\n  Неверный выбор! Повторите ввод:";
                Sleep(2000);
            }
            break;
        }
    } while (key != '0');
}
Tel_kniga:: ~Tel_kniga ()
{
    cout << "\n Деструктор для удал. массива Catalog\n";
    delete [] Catalog;
    Catalog=NULL;
}

@nmcf · 01.06.2016, 09:41

Функция локалезависима. Надо включать русскую локаль на время сравнения.

C++

void Tel_kniga:: Del(char *f_abon)
{
    setlocale(LC_CTYPE, ""); // русская
 
    int k = 0;
    for (int i = 0; i < n; i++)
    {
        if (stricmp(Catalog[i].GetFio(), f_abon) == 0)
        {
            k++;
        }
    }
    Telefon *New = new Telefon [n - k];
    for (int i = 0, j = 0; i < n; i++)
    {
        if (stricmp(Catalog[i].GetFio(), f_abon) != 0)
        {
            New[j] = Catalog[i];
            j++;
        }
    }
 
    setlocale(LC_CTYPE, "C"); // стандартная
 
    if (k == 0)
    {
        cout << "Нет совпадений.\n";
    }
    else
    {
        cout << "Удаление произошло.\n";
    }
    n = n - k;
    Catalog=New;
}

@sv1tlana · 01.06.2016, 16:15 **[ТС]**

nmcf , еще раз большое спасибо ! Подключила, теперь сортирует все как положено.

@avgoor 1550 / 875 / 179 Регистрация: 05.12.2015 Сообщений: 2,555
	31.05.2016, 13:00	3
	sv1tlana, У вас нарушение правила трех: вы не реализовали оператор присваивания. В результате в 231 строке в `tmp` и `Catalog[j + 1]` есть указатели, которые указывают на одну и ту же память (`Fio` и `dop_inf`). При выходе из `if` в 232 строке для `tmp` вызывается деструктор, который удаляет все, что выделено для класса и в `Catalog[j + 1]` остаются "висячие указатели". 0

@sv1tlana 0 / 0 / 0 Регистрация: 01.10.2015 Сообщений: 10
	31.05.2016, 17:54 [ТС]	4
	avgoor, здравствуйте. Я не могу понять почему происходит использование одной и той же памяти. Ведь я присваиваю значения, используя конструктор копирования (строка 35: Telefon (const Telefon &obj) ) Причем вызываю его принудительно... Вы не могли бы написать фрагмент кода, показывающий как правильно сделать обмен между двумя объектами. 0

@nmcf 7275 / 6220 / 2833 Регистрация: 14.04.2014 Сообщений: 26,871
	31.05.2016, 18:36	5
	У тебя и без этого неправильно. Сначала конструктор и длину исправь. 0

@sv1tlana 0 / 0 / 0 Регистрация: 01.10.2015 Сообщений: 10
	31.05.2016, 18:47 [ТС]	6
	nmcf, здравствуйте. Спасибо, что уделили внимание, но к сожалению, я только больше запуталась. Не могли бы Вы уточнить какой именно конструктор? Тот, который по умолчанию (строка 18)? И как его исправить? И где надо исправить длину? 0

@nmcf 7275 / 6220 / 2833 Регистрация: 14.04.2014 Сообщений: 26,871
	31.05.2016, 19:22	7
	Сообщение от sv1tlana И как его исправить? Я же показал как. Просто замени. Длину везде исправить, где память выделяешь через new. 0

@nmcf 7275 / 6220 / 2833 Регистрация: 14.04.2014 Сообщений: 26,871
	31.05.2016, 21:20	9
	Так, как у меня, делай, а не через равно NULL. И где корректировка длины? 0

@avgoor 1550 / 875 / 179 Регистрация: 05.12.2015 Сообщений: 2,555
	31.05.2016, 22:12	10
	sv1tlana, 1) В операторе присваивания, если выделяете новую память, старую надо удалать раскомментируйте. 2) Когда выделяете память не забывайте о нуль-терминаторе (new char[strlen(...)+1]) 1

@sv1tlana 0 / 0 / 0 Регистрация: 01.10.2015 Сообщений: 10
	01.06.2016, 16:15 [ТС]	13
	nmcf , еще раз большое спасибо ! Подключила, теперь сортирует все как положено. 0