Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Форум программистов и сисадминов Киберфорум > Форум программистов > C/C++ > C++ > С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.75/4: Рейтинг темы: голосов - 4, средняя оценка - 4.75
3 / 3 / 8
Регистрация: 14.10.2012
Сообщений: 288

Как правильно вынести функции в методы?

13.04.2016, 07:48. Показов 731. Ответов 6
Метки нет (Все метки)
Студворк — интернет-сервис помощи студентам
Программа работает с двумя текстовыми файлами, создает массив объектов из двух объектов(в первом- строчки из первого файла, во втором- второго). Всё работает. Осталось сделать так, чтобы код соответствовал концепции ООП, т.е. внести поправки + где я память не освободил?
модуль array.h с описанием класса:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20

#pragma once
 
#include "strings.h"
 
class StringArray{
    char **items;
    int logLength;
public:
    StringArray();
    StringArray(int length);
    StringArray(const StringArray &copy_object);
    ~StringArray();
 
    void addString(char *str, int i);
    char *atIndex(int index);
    const int getLength();
 
    StringArray *concat(StringArray *array_1, StringArray *array_2);
    StringArray *intersect(StringArray *array_1, StringArray *array_2);
};
array.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

#include "array.h"
 
StringArray::StringArray()
{
    items = NULL;
    logLength = 0;
}
StringArray::StringArray(int length)
{
    logLength = length;
    items = new char*[length];
    for(int i  = 0; i < length; i++)
        items[i] = "";
}
 
StringArray ::StringArray(const StringArray &copy_object)
{
    this->items = copy_object.items;
    this->logLength = copy_object.logLength;
}
 
StringArray::~StringArray()
{
    for(int i = 0; i < logLength; i++)
        delete items[i];
}
 
void StringArray::addString(char *str, int i)
{
    char *copy = new char[strlen(str)];
    strcpy(copy, str);
    items[i] = copy;
}
 
char *StringArray::atIndex(int index)
{
    if (index > logLength)
        return NULL;
    return items[index];
}
 
const int StringArray::getLength ()
{
    return logLength;
}
 
char *strconcat(char *src, char *dest)
{
    char *result = new char[strlen(src) + strlen(dest) + 1];
    int i;
 
    for (i = 0; *src != '\0' && *src != '\n'; i++, src++)
        result[i] = *src;
 
    for (; *dest != '\0' && *dest != '\n'; i++, dest++)
        result[i] = *dest;
 
    result[++i] = '\0';
 
    return result;
}
 
StringArray *StringArray::concat(StringArray *array_1, StringArray *array_2)
{
    StringArray *result = new StringArray(logLength);
    int length;
    if (array_1->logLength >= array_2->logLength)
        length = array_1->logLength;
    else
        length = array_2->logLength;
 
    for (int i = 0; i < length; i++)
    {
        char *con = new char[strlen(array_1->atIndex(i))+strlen(array_2->atIndex(i))];
        con = strconcat(array_1->atIndex(i), array_2->atIndex(i));
            result->addString(con, i) ;
    }
    return result;
}
 
StringArray *StringArray::intersect (StringArray *array_1, StringArray *array_2)
{
    StringArray *result = new StringArray(logLength);
    char *con;
    for(int i = 0; i < logLength; i++)
    {
        if(strcmp(array_1->items[i], array_2->items[i]) == 0)
            result->addString("", i);
        else
        {
            con = new char[strlen(array_1->atIndex(i))+strlen(array_2->atIndex(i))];
            con = strconcat(array_1->atIndex(i), array_2->atIndex(i));
            result->addString(con, i) ;
        }
    }
    return result;
}
Напоследок main.cpp и по нему больше всего вопросов. Куда вынести функцию считывания строки, вывода? И как поправить код, чтобы не так ужасно выглядел(особенно в 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
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
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190

#include <iostream>
#include <cstdio>
#include "stdlib.h"
#include "array.h"
#include "strings.h"
#include <fstream>
 
using namespace std;
 
int menu()
{
    int choise;
    cout << endl;
    cout << endl;
    cout << "                         MENU" << endl;
    cout << "1. Element-by-element concatenation of two arrays." << endl;
    cout << "2. The ordering of the rows in lexicographical order." << endl;
    cout << "3. Merge two arrays by removing duplicate rows." << endl;
    cout << "4. Display the entire array" << endl;
    cout << "5. Display the specified string" << endl;
    cout << "6. Exit." << endl;
    cout << endl;
    cout << "Choose an option: ";
    cin >> choise;
 
    return choise;
}
 
char* get_line(FILE* source)
{
    char* result;
    char string[5] = "";
    void *res;
 
    result = (char*)calloc(1, sizeof(char));
    if (!result)
    {
        return NULL;
    }
 
    while(strchr(result,'\n') == NULL && !feof(source))
    {
        fgets(string, 5, source);
        res = realloc(result,(strlen(result) + strlen(string))+1);
        if (!res)
        {
            free(result);
            return NULL;
        }
        else
            result = (char*)res;
        result = strcat(result, string);
    }
 
    if (strlen(result) == 0)
    {
        free(result);
        return NULL;
    }
 
    return result;
}
 
void printAll(StringArray *array)
{
    for (int i = 0; i < array->getLength(); i++)
        cout << array->atIndex(i) << endl;
}
 
int main()
{
     StringArray **mass;
     mass = new StringArray*[2];
     int count_row_1;
     int count_row_2;
 
     FILE *source_1 = fopen("1.txt","r");
     if(!source_1)
     {
         cout << "File not found" << endl;
         return -1;
     }
 
 
     FILE *source_2 = fopen("2.txt","r");
     if(!source_2)
     {
         cout << "File not found" << endl;
         fclose(source_1);
         return -1;
     }
 
     if(fscanf(source_1, "%d", &count_row_1) != 1)
     {
         cout << "Read error!" << endl;
         fclose(source_1);
         fclose(source_2);
         return -1;
     }
 
     if(fscanf(source_2, "%d", &count_row_2) != 1)
     {
         cout << "Read error!" << endl;
         fclose(source_1);
         fclose(source_2);
         return -1;
     }
 
     if(count_row_1 >= count_row_2)
        count_row_2 = count_row_1;
     else
        count_row_1 = count_row_2;
 
     char *buf_str = get_line(source_1);
     if(!buf_str)
     {
         cout << "Read error" << endl;
         return -1;
     }
 
     int i = 0;
     mass[0] = new StringArray(count_row_1);
     while(buf_str && !feof(source_1))
     {
         buf_str = get_line(source_1);
         mass[0]->addString(buf_str, i);
         i++;
     }
     fclose(source_1);
 
 
     mass[1] = new StringArray(count_row_2);
     i = 0;
     buf_str = get_line(source_2);
     if(!buf_str)
     {
         cout << "Read error" << endl;
         return -1;
     }
     while(buf_str && !feof(source_2))
     {
         buf_str = get_line(source_2);
         mass[1]->addString(buf_str, i);
         i++;
     }
 
    int choice;
    while(choice != 6)
    {
        choice = menu();
        cout << endl;
        switch (choice)
        {
            case 1:
                {
                    cout<<"       Element-by-element concatenation of two arrays."<<endl;
                    StringArray *mass3 = (**mass).concat(mass[0], mass[1]);
                    printAll(mass3);
                    break;
                }
            case 3:
                {
                    cout<<"____________"<<endl;
                    StringArray *mass4 = (**mass).intersect(mass[0], mass[1]);
                    printAll(mass4);
                    break;
                }
            case 4:
                {
                    printAll(mass[0]);
                    cout<<"____________"<<endl;
                    printAll(mass[1]);
                    break;
                }
            case 5:
                {
                    int row;
                    cout << "Input number string: ";
                    cin >> row;
                    cout << mass[0]->atIndex(row) << endl;
                    cout << mass[1]->atIndex(row) << endl;
                    break;
                }
        }
    }
 
    fclose(source_2);
    delete buf_str;
    return 0;
}
Добавлено через 30 минут
Вот, подправил,только как эту функцию(lala) сделать методом класса?
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
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181

#include <iostream>
#include <cstdio>
#include "stdlib.h"
#include "array.h"
#include "strings.h"
#include <fstream>
 
using namespace std;
 
char* get_line(FILE* source)
{
    char* result;
    char string[5] = "";
    void *res;
 
    result = (char*)calloc(1, sizeof(char));
    if (!result)
    {
        return NULL;
    }
 
    while(strchr(result,'\n') == NULL && !feof(source))
    {
        fgets(string, 5, source);
        res = realloc(result,(strlen(result) + strlen(string))+1);
        if (!res)
        {
            free(result);
            return NULL;
        }
        else
            result = (char*)res;
        result = strcat(result, string);
    }
 
    if (strlen(result) == 0)
    {
        free(result);
        return NULL;
    }
 
    return result;
}
 
int menu()
{
    int choise;
    cout << endl;
    cout << endl;
    cout << "                         MENU" << endl;
    cout << "1. Element-by-element concatenation of two arrays." << endl;
    cout << "2. The ordering of the rows in lexicographical order." << endl;
    cout << "3. Merge two arrays by removing duplicate rows." << endl;
    cout << "4. Display the entire array" << endl;
    cout << "5. Display the specified string" << endl;
    cout << "6. Exit." << endl;
    cout << endl;
    cout << "Choose an option: ";
    cin >> choise;
 
    return choise;
}
 
void printAll(StringArray *array)
{
    for (int i = 0; i < array->getLength(); i++)
        cout << array->atIndex(i) << endl;
}
 
int lala(FILE *f, StringArray **mass, int count_r, int j)
{
    mass[j] = new StringArray(count_r);
    char *buf_str = get_line(f);
    if(!buf_str)
    {
        cout << "Read error" << endl;
        return -1;
    }
    int i = 0;
    mass[j] = new StringArray(count_r);
    while(buf_str && !feof(f))
    {
        buf_str = get_line(f);
        mass[j]->addString(buf_str, i);
        i++;
    }
    return 0;
}
 
int main()
{
    StringArray **mass;
    mass = new StringArray*[2];
    int count_row_1;
    int count_row_2;
    int count_row_for_alloc;
 
    FILE *source_1 = fopen("1.txt","r");
    if(!source_1)
    {
        cout << "File not found" << endl;
        return -1;
    }
 
    FILE *source_2 = fopen("2.txt","r");
    if(!source_2)
    {
        cout << "File not found" << endl;
        fclose(source_1);
        return -1;
    }
 
    if(fscanf(source_1, "%d", &count_row_1) != 1)
    {
        cout << "Read error!" << endl;
        fclose(source_1);
        fclose(source_2);
        return -1;
    }
 
    if(fscanf(source_2, "%d", &count_row_2) != 1)
    {
        cout << "Read error!" << endl;
        fclose(source_1);
        fclose(source_2);
        return -1;
    }
 
    if(count_row_1 >= count_row_2)
       count_row_for_alloc = count_row_1;
    else
       count_row_for_alloc = count_row_2;
 
    lala(source_1, mass, count_row_for_alloc, 0);
    fclose(source_1);
 
    lala(source_2, mass, count_row_for_alloc, 1);
    fclose(source_2);
 
    int choice;
    while(choice != 6)
    {
        choice = menu();
        cout << endl;
        switch (choice)
        {
            case 1:
                {
                    cout<<"       Element-by-element concatenation of two arrays."<<endl;
                    StringArray *mass3 = (**mass).concat(mass[0], mass[1]);
                    printAll(mass3);
                    break;
                }
            case 3:
                {
                    cout<<"____________"<<endl;
                    StringArray *mass4 = (**mass).intersect(mass[0], mass[1]);
                    printAll(mass4);
                    break;
                }
            case 4:
                {
                    printAll(mass[0]);
                    cout<<"____________"<<endl;
                    printAll(mass[1]);
                    break;
                }
            case 5:
                {
                    int row;
                    cout << "Input number string: ";
                    cin >> row;
                    cout << mass[0]->atIndex(row) << endl;
                    cout << mass[1]->atIndex(row) << endl;
                    break;
                }
        }
    }
 
    return 0;
}
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
 13.04.2016, 07:48
Ответы с готовыми решениями:

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

Как вынести функции во внешний файл
День добрый. Недавно начал изучать микроконтроллеры и написание кода на Си\С++ Освоил основы передачи данных по последовательному порту и...

Как правильно наследовать методы класса
#include &lt;iostream&gt; using namespace std; class Pair { protected: int first; // первое число int second; ...

6
return (true);
 Аватар для mimicria
1977 / 1112 / 221
Регистрация: 19.04.2011
Сообщений: 2,346
13.04.2016, 08:05
В конструкторе инициализации память выделяется только под массив указателей, под сами элементы нет.
В деструкторе наоборот, причём delete вызван неправильно (без [])
В конструкторе копирования идёт простое присвоение указателей, а значит объект-копия будет ссылаться на ту же память, что и копируемый объект.
0
694 / 304 / 99
Регистрация: 04.07.2014
Сообщений: 851
13.04.2016, 08:51
Цитата Сообщение от Nike1995 Посмотреть сообщение
C++
1
2
3
4
5

StringArray ::StringArray(const StringArray &copy_object)
{
   this->items = copy_object.items;
   this->logLength = copy_object.logLength;
}
После это будет существовать два объекта, использующие один и тот же участок памяти, и, как следствие, они оба попытаются её освободить.
0
3 / 3 / 8
Регистрация: 14.10.2012
Сообщений: 288
13.04.2016, 12:33  [ТС]
Я понимаю, что ошибки есть, но как исправить. весь код написан мной, но пару моментов я не знаю как делать
0
694 / 304 / 99
Регистрация: 04.07.2014
Сообщений: 851
13.04.2016, 13:19
Nike1995, Начинай с исправления сказанных тебе ошибок, так же добавь, например, test.cpp - простую программу для тестирования твоего класса, в которой будет проверка класса при работе с одним файлом без взаимодействия с человеком.
0
3 / 3 / 8
Регистрация: 14.10.2012
Сообщений: 288
13.04.2016, 14:35  [ТС]
mimicria, вот, переделал деструктор.
C++
1
2
3
4
5
6
7

StringArray::~StringArray()
{
    for (int i = 0; i < logLength; i++)
        delete []items[i];
 
    delete items;
}
Как переделать конструктор копирования? в конструкторе помять под каждый элемент выделется в другом месте т.к. строка считывается из файла и ее размер заранее не известен
0
return (true);
 Аватар для mimicria
1977 / 1112 / 221
Регистрация: 19.04.2011
Сообщений: 2,346
13.04.2016, 14:48
Цитата Сообщение от Nike1995 Посмотреть сообщение
в конструкторе помять под каждый элемент выделется в другом месте т.к. строка считывается из файла и ее размер заранее не известен
Это ошибка в логике. Надо сначала считать строку из файла, узнать её размер, а потом создавать экземпляр класса, инициализируя его этим размером и строкой.
Цитата Сообщение от Nike1995 Посмотреть сообщение
Как переделать конструктор копирования?
По аналогии выделить памяти столько же, сколько в объекте-оригинале (через new) и скопировать строку из оригинала, например strcpy()
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
 13.04.2016, 14:48
Помогаю со студенческими работами здесь

Как правильно вписать функцию в h-файл и правильно её отразить в основной функции?
Создать внешнюю библиотеку, в которой написана функция вывода символьного массива обратном порядке. Подключить данную библиотеку к ...

Как правильно вызвать методы, чтобы передать весь массив объектов
#include &lt;iostream&gt; using namespace std; class f { public: int a; f() { a = 0;

Вынести методы из классов Panel и PictureBox (явная реализация методов базовых абстрактных классов)
Тема: Множественное наследование. Явная реализация методов базовых абстрактных классов. Как вынести методы из классов Panel и...

Вынести функции из класса
Здравствуйте нужно вынести функции из класса в сам main подскажите как реализовать: #include &lt;iostream&gt; #include...

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


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

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