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

Конструктор копирования, найти недочеты в программном коде - C++

Восстановить пароль Регистрация
 
Notty
0 / 0 / 0
Регистрация: 11.10.2011
Сообщений: 30
02.03.2012, 21:02     Конструктор копирования, найти недочеты в программном коде #1
Здравствуйте, посмотрите пожалуйста, тут у меня конструктор копирования. Я нашел какой-то пример, попробовал все переделать под свою программу. Если тут бред, очень не смейтесь) препод как-то объяснил быстро и ничего не понятно, надеюсь вы поможете.

Добавлено через 2 минуты
adress.h
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
27
28
29
30
31
32
class adress
{
public:
 adress (char*, char*, char*, char*, int, int);
 adress ();
 ~adress();
 friend void porivn (adress x);
 
 char name[64];
 char country[64];
 char city[64];
 char street[64];
 int house;
 int flat;
 void show_adress();
 void get(char* fname); 
 void set(char* fname);
};
 
class copy_adress
{
public:
    adress *c_name;
    adress *c_country;
    adress *c_city;
    adress *c_street;
    adress *c_house;
    adress *c_flat;
 
    copy_adress();
    copy_adress(const copy_adress & w);
};
adress.cpp
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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
#include "adress.h"
#include <iostream>
#include <cstring>
#include <fstream>
using namespace std;
 
//конструктор з параметрами
adress::adress(char *name, char *country, char *city, char *street, int house, int flat)
{
    strcpy_s(adress::name, name);
    strcpy_s(adress::country, country);
    strcpy_s(adress::city, city);
    strcpy_s(adress::street, street);
    adress::house = house;
    adress::flat = flat;
}
 
//деструктор
adress::~adress()
{
    cout<<"Знищення об єкта з наступним именем:  "<<endl;
    cout<<"Им*я: "<<name<<endl;
    cout<<endl;
}
 
//дружня функція
void porivn(adress x)
{
    if (x.house>x.flat) 
    {
        cout<<"Номер дому "<<x.name<<"="<<x.house<<" бильший за номер квартири "<<x.flat<<endl;
        cout<<endl;
    }
    else 
    {
        cout<<"Номер квартири "<<x.name<<"="<<x.flat<<" бильший за номер дому "<<x.house<<endl;
        cout<<endl;
    }
}
 
//Конструктор без параметрів
adress::adress()
{
 cout<< "Им*я: " <<name<<endl;
 cout<< "Країна: " <<country<<endl;
 cout<< "Мисто: " <<city<<endl;
 cout<< "Вулиця: " <<street<<endl;
 cout<< "Будинок: " <<house<<endl;
 cout<< "Квартира: " <<flat<<endl;
 cout<<endl;
}
 
 
void adress::set(char *fname)
 {
  ofstream file(fname);
   file<<name<<endl <<country<<endl <<city<<endl <<street<<endl <<house<<endl <<flat<<endl;
 }
 
void adress::get(char *fname)
 {
  ifstream file(fname);
   file.getline(name,64);
   file.getline(country,64);
   file.getline(city,64);
   file.getline(street,64);
   file>>house;
   file>>flat;
 }
 
copy_adress::copy_adress() 
{
    c_house=new adress;
}
 
 
copy_adress::copy_adress(const copy_adress & w)
{
    
        this->c_name = new adress;
        strcpy_s(this->c_name->name,w.c_name->name);
 
        
        this->c_country = new adress;
        strcpy_s(this->c_country->country,w.c_country->country);
        
        
        this->c_city = new adress;
        strcpy_s(this->c_city->city,w.c_city->city);
        
        this->c_street = new adress;
        strcpy_s(this->c_street->street,w.c_street->street);
        
        this->c_house = new adress;
        this->c_house->house = w.c_house->house;
 
    
        this->c_flat = new adress;
        this->c_flat->flat = w.c_flat->flat;
}
lab.cpp

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
#include <iostream>
#include <cstring>
#include <fstream>
#include "adress.h"
using namespace std;
 
void main()
{
 setlocale(LC_ALL, "Ukrainian");
 
 adress first_human("Джон Смит","Англия","Лондон","Бромптон Роуд",79,15);
 adress second_human("Артем Милевський","Україна","Київ","Институтська",23,137);
 
 
 
 first_human.set("1.txt");
 second_human.set("2.txt");
 
 
 first_human.get("1.txt");
 second_human.get("2.txt");
 
 
 first_human.adress::adress();
 second_human.adress::adress();
 
porivn(first_human);
porivn(second_human);
 
 //copy_adress() 
// copy_adress(const copy_adress & w)
 
 copy_adress q;
 copy_adress q1 = q ;
 
 
 
 cout<<q1.c_name->name;
 cout<<q1.c_country->country;
 cout<<q1.c_city->city;
 cout<<q1.c_street->street;
 cout<<q1.c_house->house;
 cout<<q1.c_flat->flat;
 
 cin.get();
 
 
}



что я тут такое наделал, что оно не хочет нормально копировать?))
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
02.03.2012, 21:02     Конструктор копирования, найти недочеты в программном коде
Посмотрите здесь:

C++ Помогите, пожалуйста, найти ошибки в программном коде!!!
C++ Конструктор копирования
Ошибка в программном коде C++
C++ Открытие файла, найти недочеты в коде
C++ Замена слов в цикле. Найти недочеты в коде
Программа из учебника Прата. Структуры в многофайловом приложении. Исправить недочеты в коде C++
Разность множеств: найти недочеты в коде C++
C++ Конструктор со списком инициализации: найти ошибку в коде

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

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
OstapBender
 Аватар для OstapBender
581 / 519 / 35
Регистрация: 22.03.2011
Сообщений: 1,585
03.03.2012, 01:09     Конструктор копирования, найти недочеты в программном коде #2
Цитата Сообщение от Notty Посмотреть сообщение
C++
1
2
3
4
5
6
7
8
copy_adress q;
copy_adress q1 = q ;
cout<<q1.c_name->name;
cout<<q1.c_country->country;
cout<<q1.c_city->city;
cout<<q1.c_street->street;
cout<<q1.c_house->house;
cout<<q1.c_flat->flat;
а что вы ожидаете получить от пустого объекта?

не падает (в КК) програмка-то? поля не инициализированы.

Цитата Сообщение от Notty Посмотреть сообщение
first_human.adress::adress();
*second_human.adress::adress();
убрать
Deviaphan
Делаю внезапно и красиво
Эксперт C++
 Аватар для Deviaphan
1283 / 1217 / 50
Регистрация: 22.03.2011
Сообщений: 3,744
03.03.2012, 18:51     Конструктор копирования, найти недочеты в программном коде #3
Цитата Сообщение от Notty Посмотреть сообщение
adress::name
избыточно и некрасиво. Можно просто писать name и аналогично остальное.

Добавлено через 36 секунд
А чтобы компилятор не ругался, аргументы назвать по другому.

Добавлено через 1 минуту
В функцию porivn адрес передавать по константной ссылке.

Добавлено через 1 минуту
Над бредом смеяться не буду, но в copy_adress ты создаёшь отдельную структуру для каждого поля из адреса, хотя достаточно одного объекта адреса. Но я не смеюсь, нет.
Yandex
Объявления
03.03.2012, 18:51     Конструктор копирования, найти недочеты в программном коде
Ответ Создать тему
Опции темы

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