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

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

13.04.2016, 07:48. Показов 744. Ответов 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
Ответ Создать тему
Новые блоги и статьи
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
 SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 12.02.2026
Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 11.02.2026
Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Поддержать - Реклама - Условия использования - Обратная связь
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru