Как сократить программу, сделав код более компактным?

@dmuriam · Регистрация: 24.07.2015

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

При это нужно удалить и не использовать функции Multip_2 и Subtr_2.
Но нужно по-прежнему вывести на экран вектора float CX1[SIZE]; float X2[SIZE];

Прикреплен файл с похожим примером, см стр. 17 - 18

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
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
#include <cmath>
#include <iostream>
#include <conio.h>
#include <iomanip> // для setprecision (сколько знаков после запятой в действительном числе) ;  setw (отступы)
#include <locale>
#define SIZE 4
 
 
using namespace std;
 
 
float Var(int n); //для расчета с
void Multip(float C[SIZE][SIZE], float B[SIZE], float CB[SIZE]); // умножение матрицы на вектор 
void Subtr(float B[SIZE], float CB[SIZE], float X1[SIZE]); // вычитание двух векторов 
void Multip_2(float C[SIZE][SIZE], float X1[SIZE], float CB[SIZE]); // умножение для X2
void Subtr_2(float B[SIZE], float CX1[SIZE], float X2[SIZE]); // вычитание для X2 
 
 
 
int main()
{
    setlocale(LC_ALL, "rus");
     
    
    float c;
    int i,j;
    float C[SIZE][SIZE];
    float normC, max1=0;
    float normB, max2=0;
    int steps; 
    float CB[SIZE];  
    float X1[SIZE];
    float CX1[SIZE];
    float X2[SIZE];
    
    
        
    c = Var(5);
    
    
    float arr1[SIZE][SIZE] = {{ 0.95+c, 0.26+c, -0.17+c, 0.27+c },
                              { -0.15+c, 1.26+c, 0.36+c, 0.42+c },
                              { 0.26+c, -0.54+c, -1.76+c, 0.31+c },
                              { -0.44+c, 0.29+c, -0.78+c, -1.78+c }};
                              
                              
                              
                                
    //вывод матрицы изначальной arr1    
    cout << "изначальная матрица arr1 \n" << endl; 
        for (i = 0; i<SIZE; i++)          //цикл по строкам
        {
            for (j = 0; j<SIZE; j++)      //цикл по столбцам
            {
                    cout << setw(12) << fixed << setprecision(3) << arr1[i][j]; 
            }
            cout << endl;
        }
        cout << endl;
    
//////////////////////////////////////////////////////////////////////////////////////////////////////
 
    //вывод матрицы — (она же det arr1)
    cout << "матрица С \n" << endl; 
    for (i = 0; i<SIZE; i++)          //цикл по строкам
        {
            for (j = 0; j<SIZE; j++)  //цикл по столбцам
            {           
                C[i][j] = arr1[i][j] / arr1[i][i];   //вычисление матрицы — из матрицы arr1  //???????? КАК ЭТО?
                if (i == j) C[i][j] = 0;            //присваивание 0 диагонали
                
                cout << setw(12) << fixed << setprecision(3) << C[i][j]; 
            }
            cout << endl; 
        }
        cout << endl;
 
//////////////////////////////////////////////////////////////////////////////////////////////////////
    
    float B[SIZE] = { 2.480, -3.160, 1.520, -1.290 };   
    
    cout << "изначальный вектор В \n" << endl;
    for (i = 0; i<SIZE; i++)  
    {   
        cout << setw(12) << fixed << setprecision(3) << B[i]; 
        cout << endl;   
    }
    cout << endl;
                                        
    
    cout << "вектор неВ \n" << endl;
    for (i = 0; i<SIZE; i++)  
    {
        B[i] = B[i] / arr1[i][i];   
        cout << setw(12) << fixed << setprecision(3) << B[i]; 
        cout << endl;   
    }
    cout << endl;
    
//////////////////////////////////////////////////////////////////////////////////////////////////////
    
    cout << "норма матрицы С = " ;  
    for (int i=0; i<SIZE; i++)
    {
        normC = fabs(C[i][0]) + fabs(C[i][1]) + fabs(C[i][2]) + fabs(C[i][3]); //за одну итерацию суммируются значения в одной строке (fabs - взятие модуля для float)
        if (normC > max1)   //если сумма значений строки > 0      //???????? ЗАЧЕМ ЭТО?
        max1 = normC;       
    }
    cout << fixed << setprecision(3) << max1 << endl;           
 
///////////////////////////////////////////////////////////////////////////////////////////////////////
    
    cout << "норма вектора неВ = " ;    
    for (int i=0; i<SIZE; i++)
    {
        normB = fabs(B[i]);  //за одну итерацию взятие модуля одного элемента
        if (normB > max2)    //если значение элемента > 0         //???????? ЗАЧЕМ ЭТО?
        max2 = normB;   
    }
    cout << fixed << setprecision(3) << B[i] << endl;           
    cout << "\n\n";
    
///////////////////////////////////////////////////////////////////////////////////////////////////////
/*
//  Inaccuracy = round(log((0.0001 / max2)*(1 - max1)) / log(max1) - 1);
 
    
 
    cout << "кол-во шагов погрешности = " << steps << endl;
    cout << endl;
*/
//////////////////////////////////////////////////////////////////////////////////////////////////////
    
    cout << "----------------------------------------------\n" ;
    cout << "результат умножения матрицы С на вектор неВ \n\n" ; 
 
    Multip(C, B, CB);
    
    for (i = 0; i<SIZE; i++)  
    {
        cout << setw(12) << fixed << CB[i] << endl;
    }
    cout << endl;
    
//////////////////////////////////////// 
 
    cout << "результат вычитания вектора CнеB из вектора неB \n" ;
    cout << "то есть это вектор X1 \n" ;
    
    Subtr(B, CB, X1);
          
        for (i = 0; i<SIZE; i++)  
    {
        cout << setw(12) << fixed << X1[i] << endl;
    }
    cout << endl;
    
///////////////////////////////////////////////////////////////////////////////////////////////////////
    
    cout << "----------------------------------------------\n" ;
    cout << "результат умножения матрицы С на вектор X1 \n\n" ; 
    
    Multip_2(C, X1, CX1);
    
    for (i = 0; i<SIZE; i++)  
    {   
        cout << setw(12) << fixed << CX1[i] << endl;
    }
    cout << endl;
    
////////////////////////////////////////
 
    cout << "результат вычитания вектора CX1 из вектора неB \n" ;
    cout << "то есть это вектор X2 \n" ;
    
    Subtr_2(B, CX1, X2);
          
        for (i = 0; i<SIZE; i++)  
    {
        cout << setw(12) << fixed << X2[i] << endl;
    }
    cout << endl;
    
/////////////////////////////////////////////////////////////////////////////////////////////////////// 
    cout << "----------------------------------------------\n" ;
    
    getch();
    return 0;
}
 
/* ф-ция для рассчета с */
float Var(int n)
{
    return (float)n*0.01;
}
 
 
/* ф-ция умножения матрицы С на вектор неB */
void Multip(float C[SIZE][SIZE], float B[SIZE], float CB[SIZE])
{
    for (int i=0; i < SIZE; i++)
    {
        CB[i] = 0;
        for (int j=0; j < SIZE; j++)
        {
            CB[i] += C[i][j] * B[i];
        }
    }  
}
/* ф-ция вычитания вектора CнеB из вектора неB */
void Subtr(float B[SIZE], float CB[SIZE], float X1[SIZE])
{
    for (int i=0; i < SIZE; i++)
    {
        X1[i] = 0;
        for (int j=0; j < SIZE; j++)
        {
            X1[i] = B[i] - CB[i];
        }
    } 
    cout << endl; 
}
 
 
/* ф-ция умножения матрицы С на вектор X1 */
void Multip_2(float C[SIZE][SIZE], float X1[SIZE], float CX1[SIZE])
{
    for (int i=0; i < SIZE; i++)
    {   
        CX1[i] = 0;
        for (int j=0; j < SIZE; j++)
        {
            CX1[i] += C[i][j] * X1[i];
        }
    }  
}
/* ф-ция вычитания вектора X1 из вектора неB */
void Subtr_2(float B[SIZE], float CX1[SIZE], float X2[SIZE])
{
    for (int i=0; i < SIZE; i++)
    {
        X2[i] = 0;
        for (int j=0; j < SIZE; j++)
        {
            X2[i] = B[i] - CX1[i];
        }
    } 
    cout << endl; 
}

@denistymchuk · 13.01.2018, 18:57

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
#include<iostream>
//ваши инклуды
#define SIZE 4
#define f(i) for(int i = 0;i < SIZE;i++)//больше циклов for я у вас не нашел
//теперь используйте такой вызов f(i)или f(i) в зависимости от нужной переменой в цикле, а еще лучше написать функцию //вывода массива по вашему правилу
void printArray(int* arr)//для тех выводов , где выводиться результат умножения
{
    f(i)
        cout << setw(12) << fixed << arr[i] << "\n";
    cout << "\n"
}
//А там где вложенный цикл напишите так
f(i)
    f(j)
        //ваши действия

@outoftime · 20.05.2021, 15:31

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

обобщить их , при этом сделать это удобней

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

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

нужно удалить и не использовать функции Multip_2 и Subtr_2.

Это логическая задача. К кодингу отношения не имеет. Делайте сами.

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

#define SIZE 4

Это Си а не С++ style.

216: X1[i] = 0;
for (int j=0; j < SIZE; j++)
{
X1[i] = B[i] - CB[i];

Не имеет смысла. Логично предположить, что здесь должно аккумулироваться значение на подобии строки 206. Если это так, у вас 2 абсолютно идентичных функции Multip и Subtr, которые различаются применяемым оператором. Следовательно, нужно сделать одну функцию, которая будет принимать оператор в качестве аргумента. Сразу меньше кода - профит.

Циклы: 139, 152, 165, 178 - copy/paste. Это называется индуский стиль программирования. Вы же знаете что такое функция, используйте её чтобы разбить вашу программу на логические блоки. Придеживайтесь принципа DRY. На случай если вы читаете это глубоком будущем, вики отстой, так как там все статьи меняют чуть ли не каждую пятницу. Чтобы быть уверенным что ссылка ведёт на нужный материал дубль на английском (сообщество с 1995 года существует, надеюсь ещё столько пробудет, а то и больше).

Новые блоги и статьи Все статьи Все блоги /
Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	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 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.

Как сократить программу, сделав код более компактным?

Решение