Прошу оценить стиль программы

@Руслан92 · Регистрация: 10.04.2016

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

Всем мира!
Ребят, в общем наваял я тут чудо-чудное, диво-дивное (мне так кажется

)
Прошу, чтоб посмотрели код, посоветовали, что можно исправить, что добавить, как короче можно написать, как логичней ну и т.д. В общем взглянуть на программу сторонними разбирающимися глазами

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

Я не студент, это не моя курсовая, экзамен и т.д. Я не оказываю кому-либо медвежью или финансовую услугу. Я самоучка стремящийся к Зеленому Бегущему Свету

Сначала я кину .h, затем .cpp

Если нужно кину на почту .zip с проектом (среда VS 2017)!

Спасибо!

pch.h - основной заголовок.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
#ifndef PCH_H
#define PCH_H
#include <iostream>
#include <fstream>
#include <ios>
#include <vector>
#include <iomanip>
#include <cstring>
#include <map>
#include <conio.h>
#include <windows.h>
int menu();
#endif

Class.h - заголовок класса petshopБ в котором реализуются все звери

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
#pragma once
#include <fstream>
 
class petshop {
    char animal[20];
    char name[20];
    char gender[7];
    int cost;
    int count;
public:
    petshop();
    petshop(const petshop &ob);
    void set_name();
    void set_animal();
    void set_gender();
    void set_cost();
    void set_count();
    void show_sample();
    char *get_animal();
    char *get_name();
    char *get_gender();
    int *get_cost();
    int *get_count();
    petshop operator=(petshop ob);
    friend std::ostream &operator<<(std::ostream &stream, const petshop ob);
    friend std::istream &operator>>(std::istream &stream, petshop &ob);
    ~petshop();
};

draw.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
#pragma once
#include "Class.h"
enum ConsoleColor
{
    Black = 0,
    Blue = 1,
    Green = 2,
    Cyan = 3,
    Red = 4,
    Magenta = 5,
    Brown = 6,
    LightGray = 7,
    DarkGray = 8,
    LightBlue = 9,
    LightGreen = 10,
    LightCyan = 11,
    LightRed = 12,
    LightMagenta = 13,
    Yellow = 14,
    White = 15
};
 
void draw(const char str[15]);
void draw(petshop ob);

update.h - заголовок функционала редактирования зверей в магазине

C++
1
2
3
4
5
6
7
8
9
#include "Class.h"
#include <vector>
 
#ifndef UPDATE
#define UPDATE
int menu_update();
int choose_animal(std::vector<petshop> &v);
void update_animal(std::vector <petshop> &v);
#endif

remove.h - заголовок функционала удаления зверей

C++
1
2
3
4
5
6
7
8
#include "Class.h"
#include <vector>
 
#ifndef REMOVE
#define REMOVE
void remove_animal(std::vector<petshop> &v);
int choose_animal2(std::vector<petshop> &v);
#endif

write_file.h - заголовок функции записи БД в файл

C++
1
2
3
4
#pragma once
#include "Class.h"
#include <vector>
void write_to_file (std::vector<petshop> &v);

read_file.h - заголовок функции чтения данных из файла и переноса данных в БД

C++
1
2
3
4
#pragma once
#include "Class.h"
#include <vector>
void read_from_file(std::vector<petshop> &v);

main.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
101
102
#include "pch.h"
#include "draw.h"
#include "Class.h"
#include "update.h"
#include "remove.h"
#include "write_file.h"
#include "read_file.h"
#include <iostream>
#include <conio.h>
#include <vector>
using namespace std;
 
int main()
{
    int answer;
    char r1[20];
    vector <petshop> v;
    v.reserve(1000);
    petshop *s;
    int i;
 
    for (;;) {
        answer = menu();
        switch (answer)
        {
        case 0: {
            s = new petshop;
            s->set_animal();
            s->set_name();
            s->set_gender();
            s->set_cost();
            s->set_count();
            v.emplace_back(*s);
            delete s;
            break;
        }
        case 1: {
            for (i = 0; i < v.size(); i++) {
                v[i].show_sample();
            }
            cin.get();
            break;
        }
        case 2: {
            update_animal(v);
            break;
        }
        case 3: {
            remove_animal(v);
            break;
        }
        case 4: {
            write_to_file(v);
            break;
        }
        case 5: {
            read_from_file(v);
            break;
        }
        case 6: {
            return 0;
        }
        }
    }
}
    
int menu()
    {
    static int key = 0;
    int code;
    do {
        system("cls");
        if (key == 0) draw("Add animal");
        else cout << "Add animal\n";
        if (key == 1) draw("Display");
        else cout << "Display\n";
        if (key == 2) draw("Update animal");
        else cout << "Update animal\n";
        if (key == 3) draw("Remove animal");
        else cout << "Remove animal\n";
        if (key == 4) draw("Write to file");
        else cout << "Write to file\n";
        if (key == 5) draw("Read file");
        else cout << "Read file\n";
        if (key == 6) draw("Close program");
        else cout << "Close program\n";
        cout << endl;
            if (key == key)
        {
            code = getch();
            if (code == 80) {
                key++;
                if (key == 7) key = 0;
            }
            if (code == 72) {
                key--;
                if (key == -1) key = 6;
            }
        }
    } while (code != 13);
    return key;
}

class.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
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
#include "pch.h"
#include "Class.h"
#include <iostream>
#include <cstring>
#include <fstream>
using namespace std;
 
petshop::petshop()
{
    strcpy(animal, "");
    strcpy(name, "");
    strcpy(gender, "");
    cost = 0;
    count = 0;
}
 
petshop::petshop(const petshop &ob)
{
    strcpy(this->animal, ob.animal);
    strcpy(this->name, ob.name);
    strcpy(this->gender, ob.gender);
    cost = ob.cost;
    count = ob.count;
}
 
void petshop::set_animal() {
    char r1[20];
    cout << "What kind of animal you want to write?\n";
    cin >> r1;
    strcpy(animal, r1);
}
 
void petshop::set_name() {
    char r1[20];
    cout << "What name you want to write?\n";
    cin >> r1;
    strcpy(name, r1);
}
 
void petshop::set_gender() {
    char r1[20];
    cout << "What gender do the animal have?\n";
    cin >> r1;
    strcpy(gender, r1);
}
 
void petshop::set_cost() {
    int a;
    cout << "What is the cost of the animal?\n";
    cin >> a;
    cost = a;
}
 
void petshop::set_count() {
    int a;
    cout << "How many animals of that type?";
    cin >> a;
    count = a;
}
 
void petshop::show_sample()
{
    cout << "Animal: " << this->animal << '\n';
    cout << "Name: " << this->name << '\n';
    cout << "Gender: " << this->gender << '\n';
    cout << "Cost: " << this->cost << '$' << '\n';
    cout << "Count: " << this->count << '\n';
}
 
char *petshop::get_animal() 
{
    return animal; 
}
 
char *petshop::get_name()
{
    return name;
}
 
char *petshop::get_gender()
{
    return gender;
}
 
int *petshop::get_cost()
{
    return &cost;
}
 
int *petshop::get_count()
{
    return &count;
}
 
petshop::~petshop()
{
}
 
 
petshop petshop::operator=(petshop ob)
{
    strcpy(this->animal, ob.animal);
    strcpy(this->name, ob.name);
    strcpy(this->gender, ob.gender);
    cost = ob.cost;
    count = ob.count;
    return *this;
}
 
ostream &operator<<(ostream &stream, petshop obj) 
{
    stream << obj.animal << '\n';
    stream << obj.name << '\n';
    stream << obj.gender << '\n';
    stream << obj.cost << '\n';
    stream << obj.count << "\n\n";
 
    return stream;
}
 
istream &operator>>(istream &stream, petshop &obj)
{
    stream >> obj.animal;
    stream >> obj.name;
    stream >> obj.gender;
    stream >> obj.cost;
    stream >> obj.count;
 
    return stream;
}

draw.cpp - описание раскраски активной строки меню

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
#include "pch.h"
#include "draw.h"
#include "Class.h"
#include <windows.h>
#include <iostream>
using namespace std;
 
void draw(const char str[15])
{
    HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
    SetConsoleTextAttribute(hStdOut, (WORD)((Black << 4) | LightMagenta));
    cout << str << '\n';
    SetConsoleTextAttribute(hStdOut, (WORD)((Black << 4) | LightGray));
}
 
void draw(petshop ob)
{
    HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
    SetConsoleTextAttribute(hStdOut, (WORD)((Black << 4) | LightMagenta));
    ob.show_sample();
    SetConsoleTextAttribute(hStdOut, (WORD)((Black << 4) | LightGray));
}

update.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
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
#include "pch.h"
#include "update.h"
#include "draw.h"
#include "Class.h"
#include <iostream>
#include <vector>
#include <conio.h>
using namespace std;
 
void update_animal(vector<petshop> &v)
{
    int i;
    if (v.empty()) {
        cout << "There is no animals in database...";
        cin.get();
        return;
    }
    int choose;
    choose = choose_animal(v);
    if (choose == v.size()) return;
    int answer2;
    for (;;) {
        answer2 = menu_update();
        switch(answer2)
        {
        case 0: 
        {
            v[choose].set_animal();
            cout << "Type successfully changed!";
            break;
        }
        case 1: 
        {
            v[choose].set_name();
            cout << "Name successfully changed!";
            break;
        }
        case 2:
        {
            v[choose].set_gender();
            cout << "Gender successfully changed!";
            break;
        }
        case 3:
        {
            v[choose].set_cost();
            cout << "Type successfully changed!";
            break;
        }
        case 4:
        {
            v[choose].set_count();
            cout << "Count successfully changed!";
            break;
        }
        case 5:
        {
            return;
        }
        }
    }
 
}
 
int menu_update()
{
    static int key2 = 0;
    int code2;
    do {
        system("cls");
        if (key2 == 0) draw("Animal type");
        else cout << "Animal type\n";
        if (key2 == 1) draw("Animal name");
        else cout << "Animal name\n";
        if (key2 == 2) draw("Animal gender");
        else cout << "Animal gender\n";
        if (key2 == 3) draw("Animal cost");
        else cout << "Animal cost\n";
        if (key2 == 4) draw("Animal count");
        else cout << "Animal count\n";
        if (key2 == 5) draw("Back to Main menu");
        else cout << "Back to Main menu\n";
        cout << endl;
 
        if (key2 == key2)
        {
            code2 = getch();
            if (code2 == 80) {
                key2++;
                if (key2 == 6) key2 = 0;
            }
            if (code2 == 72) {
                key2--;
                if (key2 == -1) key2 = 5;
            }
        }
    } while (code2 != 13);
    return key2;
}
 
int choose_animal(vector<petshop> &v)
{
    int i;
    static int key3 = 0;
    int code3;
    do {
        system("cls");
        for (i = 0; i < v.size(); i++) {
            if (key3 == i) draw(v[i]);
            else v[i].show_sample();
        }
        if (key3 == v.size()) draw("Back to Main menu");
        else cout << "Back to Main menu\n";
        cout << endl;
 
        if (key3 == key3)
        {
            code3 = getch();
            if (code3 == 80) {
                key3++;
                if (key3 == v.size()+1) key3 = 0;
            }
            if (code3 == 72) {
                key3--;
                if (key3 == -1) key3 = v.size();
            }
        }
    } while (code3 != 13);
    return key3;
}

remove.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
#include "pch.h"
#include "draw.h"
#include "Class.h"
#include "remove.h"
#include <iostream>
#include <vector>
#include <conio.h>
using namespace std;
 
void remove_animal(vector<petshop> &v)
{
    int i;
    if (v.empty()) {
        cout << "There is no animals in database...";
        cin.get();
        return;
    }
    int choose2;
    choose2 = choose_animal2(v);
    if (choose2 == v.size()) return;
    v.erase(v.begin()+choose2);
}
 
int choose_animal2(vector<petshop> &v)
{
    int i;
    static int key3 = 0;
    int code3;
    do {
        system("cls");
        for (i = 0; i < v.size(); i++) {
            if (key3 == i) draw(v[i]);
            else v[i].show_sample();
        }
        if (key3 == v.size()) draw("Back to Main menu");
        else cout << "Back to Main menu\n";
        cout << endl;
 
        if (key3 == key3)
        {
            code3 = getch();
            if (code3 == 80) {
                key3++;
                if (key3 == v.size() + 1) key3 = 0;
            }
            if (code3 == 72) {
                key3--;
                if (key3 == -1) key3 = v.size();
            }
        }
    } while (code3 != 13);
    return key3;
}

write_file.cpp - описание записи БД в файл

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
#include "pch.h"
#include "write_file.h"
#include <fstream>
#include <iostream>
 
using namespace std;
 
void write_to_file(std::vector<petshop> &v)
{
    ofstream tofile("animals.bin", ios::trunc | ios::binary);
    int i;
    if (!tofile.is_open()) {
        cout << "File is not found!";
        return;
    }
 
    for (i = 0; i < v.size(); i++) {
        tofile << v[i];
    }
 
    tofile.close();
}

read_file.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
#include "pch.h"
#include "read_file.h"
#include <fstream>
#include <iostream>
 
using namespace std;
 
void read_from_file(std::vector<petshop> &v)
{
    ifstream fromfile("animals.bin", ios::binary | ios::out);
    int i;
    if (!fromfile.is_open()) {
        cout << "File is not found!";
        return;
    }
    petshop *s = new petshop;
 
    while (!fromfile.eof()) {
        fromfile >> s->get_animal();
        fromfile >> s->get_name();
        fromfile >> s->get_gender();
        fromfile >> *s->get_cost();
        fromfile >> *s->get_count();
        v.emplace_back(*s);
    }
    fromfile.close();
}

Такая вот программка. Жду отзывов и критики!

@Руслан92 · 15.02.2019, 12:50 **[ТС]**

Azazel-San

Я скорее о целях этого проекта вы хотели пощупать ООП?

Ну да. Смысл заниматься и учиться с++, если не реализовывать его главную догму. Хочется на самом деле себя в реальном проекте попробовать. Правда я даже не знаю, какой проект можно вообще выполнить на с++ )))

@Azazel-San · 15.02.2019, 12:51

Не по теме:

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

Может, все же, венгерская нотация? :D

hungarian notation :D
Извиняюсь, русский не мой родной язык)

@_stanislav · 15.02.2019, 12:52

Сообщение от Azazel-San

это тема холиваров

чувствуется XLAT, хочет поспорить, накипело что то где то видать.

@Azazel-San · 15.02.2019, 12:54

Сообщение от Руслан92

Ну да.

Тогда в вашем проекте вы почти ничего, если смотреть со стороны концептов ООП, такого не сделали.

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

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

чувствуется XLAT, хочет поспорить, накипело что то где то видать.

Если люди что пишут так:

C++
1
2
{ int a;
}

Я еле сдерживаюсь, но кто я такой что бы судить..

@Руслан92 · 15.02.2019, 12:54 **[ТС]**

Azazel-San

Тогда в вашем проекте вы почти ничего, если смотреть со стороны концептов ООП, такого не сделали.

А как это в общем должно выглядеть? Реализация деталей - дело второе. Как концептуально эту программу можно исправить (перенести на ООП)?

@_stanislav · 15.02.2019, 12:56

нужно сваливать с темы, нервы дороже

@Azazel-San · 15.02.2019, 12:57

Сообщение от Руслан92

Как концептуально эту программу можно исправить (перенести на ООП)?

Исходя из задачи, ООП - не панацея, использовать его надо как и любой инструмент - по уму, раз он у нас уже есть)
Если хотите практики по ООП, ищите задачу где это можно красиво и просто применить, где нужны наследование, полиморфизм, абстракции, еще есть 5 принципов SOLID - это уже более к проектированию, воот.

@Руслан92 · 15.02.2019, 13:00 **[ТС]**

_stanislav , думаешь? Может ты и прав

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

По сути только для этого я сюда программу то и выложил)

@_stanislav · 15.02.2019, 13:05

Сообщение от Руслан92

Не хочу тратить время на, бессмысленные с точки зрения грамотного продакшна и профессионального кода, программки.

читай про grasp и gof

Добавлено через 2 минуты
и вообще как мне думается, инженерные принципы везде одни, в любой сфере.

@DrOffset · 15.02.2019, 13:08

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

нормальный сетер

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

@Avaddon74 · 15.02.2019, 13:11

Руслан92, Я не буду советовать, просто опишу как вижу это я

1. Разделение на файлы нужно, я для каждого класса выделяю, обычно, отдельные файлы реализации и заголовка. Сегодня у вас там 3 строчки, а через неделю будет 300

2. Я использую один заголовочный файл для всех библиотек, которые я буду использовать практически везде, а потом этот один файл инклудю где нужно
3. Конечно можно, просто в глобальной области не используйте
4. В вектор помещается копия, т.к. вы его так создали, если бы хранили указатели, то тогда в векторе были бы указатели

А оригинал хранится в памяти, где вы выделили на него место методом new, я по всему коду не пробегал, но этот кусок сразу бросился в глаза, что у вас возможна утечка памяти.
7. ifstream по умолчанию открывает на чтение, указывать ios::in не обязательно и т.д. Когда указывают binary, обычно это обозначает что будут работать с бинарными данными, а не текстом, допустим, если написать так:

C++
1
2
3
4
5
6
7
ofstream f("test1.txt", ios::binary);
f << "123\n";
f.close();
 
ofstream f1("test2.txt");
f1 << "123\n";
f1.close();

И потом посмотреть содержимое файла, то они будут разные, т.к. при работе с бинарным типом, символ переноса строки (и не только) будет интерпретироваться по разному

@_stanislav · 15.02.2019, 13:21

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

Сеттер плохой, как раз с точки зрения ООП

сорян, я про этот говорил, думал что он

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

void PetShop::set_cost(int cost)
{
this->cost = cost;
};

@DrOffset · 15.02.2019, 13:23

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

я про этот говорил

Этот нормальный

@XLAT · 15.02.2019, 13:25

Сообщение от Azazel-San

я бы не стал насаживать свой стиль кому-то, это тема холиваров

все верно, тут не о чем спорить.

1. один класс --- один файл.
2. название хедера должно соответствовать названию класса.
3. название класса должно соответствовать сущности, которую класс описывает.

это не я придумал, а все те же люди из того же продакшена.

но если ваш стиль через "одно" место, но если вы его придерживаетесь, вы привыкаете и вас это может вполне устраивать,
пока вы в проекте один.

@_stanislav · 15.02.2019, 13:32

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

пока вы в проекте один.

в своем проекте я царь и бог

Добавлено через 1 минуту
у меня объявления функций в .cpp а определения в .h, мне так удобно, но я ко всему быстро привыкаю.

@Azazel-San · 15.02.2019, 13:37

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

но если ваш стиль через "одно" место, но если вы его придерживаетесь, вы привыкаете и вас это может вполне устраивать,
пока вы в проекте один.

Это ваш девиз?

@XLAT · 15.02.2019, 13:49

Не по теме:

Сообщение от Azazel-San

Это ваш девиз?

троллфейсный офтоп?
а не хотите ли вы знать: пью я колу или нет???

а ваще то это не девиз, а наблюдение за вами)

@Azazel-San · 15.02.2019, 14:00

Не по теме:

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

а наблюдение за вами

Ну, это не я пишу стилем "через одно место" и не у меня подгорает из-за правды..

@Руслан92 · 15.02.2019, 14:05 **[ТС]**

Avaddon74

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

В вектор помещается копия, т.к. вы его так создали, если бы хранили указатели, то тогда в векторе были бы указатели А оригинал хранится в памяти, где вы выделили на него место методом new, я по всему коду не пробегал, но этот кусок сразу бросился в глаза, что у вас возможна утечка памяти.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
vector <petshop> v;
    v.reserve(1000);
    petshop *s;
    int i;
 
    for (;;) {
        answer = menu();
        switch (answer)
        {
        case 0: {
            s = new petshop;
            s->set_animal();
            s->set_name();
            s->set_gender();
            s->set_cost();
            s->set_count();
            v.emplace_back(*s);
            delete s;
            break;
        }

Я же здесь через указатели делаю. new есть, delete есть...где утечка?

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

DrOffset

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

Сеттер плохой, как раз с точки зрения ООП, так как занимается смешением ролей: ввод находится внутри, да еще и фиксирован на конкретный способ.

Приведите пример хорошего сеттера (желательно какой-нибудь сеттер в моем классе исправить на тот, который будет хорошим)

@DrOffset · 15.02.2019, 14:35

Сообщение от Руслан92

fromfile >> *s->get_cost();

Вот, кстати, пример того, что сеттер негодный, в месте, где вам был необходим ввод из файла, пришлось городить такие странные конструкции.

Сообщение от Руслан92

Приведите пример хорошего сеттера

Хороший сеттер - это тот, который занимается своей прямой обязанностью - устаналивает состояние класса.

C++
1
2
3
void petshop::set_cost(int a) {
    cost = a;
}

А вывод сообщения пользователю и ввод данных - это дело того, кто этим сеттером пользуется. Каждая сущность должна заниматься своим делом. А ее нотация должна отражать суть этого дела. Если у вас написано set_cost, то совершенно очевидно ожидать от этой функции установку состояния "цена" у класса, но совершенно не очевидно, что оно будет выводить приглашение для ввода цены в консоль и точно не очевидно, что оно будет блокировать управление, ожидая ввода из консоли. И уж точно никто не увидит этого, если вы попробуете перенести класс в приложение GUI - ведь там нет консоли. Стоит также отличать эту ситуацию от вывода в лог. Ваш текст в консоль - это часть логики, от которой зависит то, как пользователь будет работать с приложением, а вывод в лог обычно пользователя не касается никак.

Тоже самое касается геттеров. Учитывая пример выше, ваш геттер - это тоже некоторым образом сеттер, т.к. предоставляет прямой доступ к внутреннему состоянию класса. Это является и нарушением инкапсуляции (любой неконтролируемым образом сможет измненить это состояние через указатель на данные, которые вернул геттер), так и примером смешения ролей, которое вы продемонстрировали в процитированном в начале сообщения коде.

C++
1
2
3
int petshop::get_cost() const {
    return cost;
}

Вы должны понять, что все эти действия - это не просто ритуалы, это средства понижения сложности разработки и сопровождения кода. Гораздо лучше, как пример, когда у вас есть один контролируемый способ поменять состояние класса, чем множество неконтролируемых.

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

Сообщение от Руслан92

где утечка?

Она здесь потенциально может быть, если emplace_back (или один из сеттеров) бросит исключение.

Новые блоги и статьи Все статьи Все блоги /
Программный домашний кинотеатр russiannick 27.09.2026 Сподобился на программный домашний кинотеатр. В качестве ЯВУ по традиции выбрал js. В помощники взял Яндекс-Алису. Было создано три зала на разные интересы. исторические и ретро сериал Хичкок. . .	Беседа с ИИ о программистах, недопускающих к созданию и правке кода генеративные ИИ и причины этого zorxor 21.09.2026 Раньше я радовался или получал некоторые эмоции, пусть небольшие, но всё же, от самого процесса написания кода, рекомпиляции и запуска, видя постепенное развитие программы и прочее. А теперь лень. . .	Мобильное приложение ColorStep pavlinmavlin 17.09.2026 Реализовал приложение Красный, Зеленый, Синий в Unity3d + c#. Название изменил на ColorStep. Приложение прошло модерацию и теперь доступно для скачивания. Делал его сам, шаг за шагом — и вот,. . .	Запрет дублирования строк в табличной части Maks 13.09.2026 Реализация из решения ниже выполнена на нетиповом справочнике "Нормы ТО" с табличной часть "Виды ТО", разработанного в КА2, со следующими реквизитами: - ВидТО (СправочникСсылка. ВидыТО); - ВидГСМ. . .
Скрипты Tampermonkey для CyberForum, ChatGPT, Claude и пр. Jin X 06.09.2026 Скрипты Tampermonkey для CyberForum, ChatGPT, Claude и пр. Работая с форумом и нейросетями в браузере часто хочется что-то подкорректировать или добавить какого-то функционала. Ниже прикреплён. . .	Программа опроса у.з. расходомера SLS-720F Argus19 02.09.2026 Программа опроса у. з. расходомера SLS-720F Программа опрашивает один раз в минуту три ультразвуковых расходомера SLS-720F через интерфейс RS-485 по протоколу Modbus RTU. Опрашиваются регистры. . .	Hyper-V: Компьютер должен поддерживать доверенный платформенный модуль 2.0. Maks 31.08.2026 При установке Windows 11 на виртуальную машину Hyper-V 2-го поколения вылезла такая ошибка: Решение: в параметрах виртуальной машины, в разделе "Безопасность" (Security) активировать флаг. . .	Архитектура биовида Стива в Майнкрафте: Зачем бонобо кубический каннибализм anaschu 30.08.2026 Кубический Вагинокапитализм в Minecraft: Математический инвариант ОДУ и рок Стивов-бонобо Главная задача разработанной «Модели Всего» — наглядно продемонстрировать наличие системной «судьбы». . .

@_stanislav "C with Classes" 2022 / 1404 / 523 Регистрация: 16.08.2014 Сообщений: 5,885 Записей в блоге: 1
	15.02.2019, 12:56
	нужно сваливать с темы, нервы дороже 1

@Руслан92 45 / 0 / 0 Регистрация: 10.04.2016 Сообщений: 114
	15.02.2019, 13:00 [ТС]
	_stanislav , думаешь? Может ты и прав Я просто хочу узнать концепцию и можно тему закрывать. Не хочу тратить время на, бессмысленные с точки зрения грамотного продакшна и профессионального кода, программки. Если уж что-то делать в жизни, то делать это хорошо и профессионально. Плохо - оно само получится. А для хорошо нужно усилия прикладывать. Я решил, что хочу быть программистом, а если решил, то хочу быть не просто абы каким программистом, а по-крайней мере, профессиональным, и главное, понимающим свое дело. По сути только для этого я сюда программу то и выложил) 0

Прошу оценить стиль программы

Решение