Рефакторинг кода(упрощение для понимания)

@rubbashka · Регистрация: 19.10.2014

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

Есть работающая лабораторная работа, я ее сделал, не показывал преподавателю и не буду, не хочу показывать г***код. Даже две лабораторных работы.
НО, написал я их полностью в main, не выделяя методы и не создавая новые классы, что очень грустно
Сейчас я понимаю, что если мне полностью переписанную программу каким-нибудь профи покажут на форуме, то я не пойму кто как и зачем и почему и что куда делось, что перенеслось. Хотелось бы последовательно ее рефакторить(это же так называется?).
Для начала: может кто помочь только выделить методы, не создавая отдельных классов? Я пытался сам, но джавка начинала ругаться на то что где-то что-то не объявлено или объявлено не там, и я забил.
Как я понимаю, можно создать методы для всех выводов, а еще отдельный метод для вычисления иксов.
и потом эти методы вызывать в main. Так же должна выглядеть хорошая программа?
Если неправильно использую термины или неправильное понимание java программирования, сделайте замечание, пожалуйста
Еще, выскажите замечания по поводу моего кода(непонятно назвал переменные(большие/маленькие буквы), плохо комментирую, etc..), пожалуйста

Для начала текст только первой лабораторной: Составить программу, решающую системы линейных уравнений методом Гаусса. Исходная матрица уравнения является нижнетреугольной. Для хранения структуры данных использовать одномерные массивы.

Java
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
import java.util.*;
 
public class GaussianElemination {
 
    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);
        System.out.print("Введите размерность массива: ");
        int dimM = sc.nextInt();
        System.out.println("Введите нижнетреугольную матрицу размерности " + dimM);
        double matrix[][] = new double [dimM][dimM];
        
        int k = dimM;
        
        //Заполнение треугольной матрицы по строкам слева направо
        for (int i = 0; i < dimM; i++){
            for (int j = 0 ; j < dimM - k + 1; j++){
                System.out.print("Введите элемент a(" + i + ", " + j + "): ");
                matrix[i][j] = sc.nextDouble();                
            }
            k = k - 1;
        }
        
        //Заполнение столбца свободных членов
        double free[] = new double [dimM];
        for (int i = 0; i < dimM; i++){
            System.out.print("Введите свободный член B[" + i + "]: ");
            free[i] = sc.nextDouble();
        }
        
        //Расширенная матрица
        double augmatrix[][] = new double [dimM][dimM + 1];
        for (int i = 0; i < dimM; i++){
            for (int j = 0; j < dimM; j++){
                augmatrix[i][j] = matrix[i][j];
            }            
        }
        for (int i = 0; i < dimM; i++){
            augmatrix[i][dimM] = free[i];
        }
        
        //Размерность одномерного массива и его объявление
        int dimA = (dimM * (dimM  + 1)) / 2; 
        double array[] = new double [dimA];
        
        //Забивание треугольного массива в одномерный
        for (int i = 0; i < dimM; i++){
            for (int j = 0; j < dimM; j++ ){
                int x = i * (i + 1) / 2 + j;
                array[x] = matrix[i][j];
            }
        }
         
        /* ===== Вычисление иксов ===== 
        Для каждого икса я вывел формулу, например
        x[0] = B[0]/arr[0]
        x[1] = (B[1] - (x[0]*arr[1]))/arr[2]
        x[2] = (B[2] - (x[0]*arr[3] + x[1]*arr[4]))/arr[5] и т.д.        
        Где B[i] это свободные коэффициенты, а arr[j] - это элементы
        одномерного массива который получен из исходной нижнетреугольной
        матрицы*/
        double x[] = new double [dimM];
        int count = 1; // Счетчик для коэффициенов перед иксами
        int count2 = 2;// Вспомогательные
        int count3 = 3;// счетчики
        // Для начала всем иксам придаем нулевое значение
        for (int i = 0; i < dimM; i++){
            x[i] = 0;
        }
        x[0] = free[0]/array[0];
        for (int i = 0; i < dimM - 1; i++){
            for (int j = i; j >= 0; j--){// Вычисляем сумму после B[i] - ...
                x[i + 1] = x[i + 1] + x[i - j]*array[count];
                count += 1;
            }
            //Довычисляем x по форумуле x = (b-sum)/k
            count = count + 1;
            x[i + 1] = (free[i + 1] - x[i + 1])/array[count2];
            count2 += count3;
            count3 += 1; 
        }
        
        //Вывод исходной треугольной матрицы
        System.out.println("Ваша матрица: "); 
        for (int i = 0; i < dimM; i++){
            for (int j = 0; j < dimM; j++){
                System.out.printf("%8.0f", matrix[i][j]);                
            }
            System.out.println();
        }    
               
        //Вывод расширенной матрицы (т.е. со свободными членами)
        System.out.println("Расширенная матрица: "); 
        for (int i = 0; i < dimM; i++){
            for (int j = 0; j < dimM + 1; j++){
                System.out.printf("%8.0f", augmatrix[i][j]);                
            }
            System.out.println();
        }
        
        //Вывод одномерного массива
        System.out.println("Одномерный массив: "); 
        for (int i = 0; i < dimA; i++){
            System.out.printf("%8.0f", array[i]);
            System.out.print(" ");
        }
        System.out.println();          
        
        //Вывод ответа
        System.out.println("Ответ: ");
        for (int i = 0; i < dimM; i++){
            System.out.print("x[" + i + "] равен: ");
            System.out.printf("%8.2f", x[i]);
            System.out.println();
        }
    } 
}

@MLPMan · 20.10.2014, 16:51

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

Как я понимаю, можно создать методы для всех выводов, а еще отдельный метод для вычисления иксов.

Для начала хотя бы так. (ниже набросал пример)

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

Я пытался сам, но джавка начинала ругаться на то что где-то что-то не объявлено или объявлено не там, и я забил.

Зря, передача данных в параметрах медотов - один из краеугольных камней проганья, рекомендую вернутся к этому.

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

Для хранения структуры данных использовать одномерные массивы.

Сделайте структуру данных, в которой всё будет в одномерных массивах, но доступ к элементам, удобным способом, типа такого:

Java
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
class SysEqMembers {
        
    private double matrix[][], 
                   free[], 
                   augmatrix[][], 
                   array[]; 
    private int dimM;
    
    public static void showArray(double arr) {
        for (int i = 0; i < arr.length; i++){
            System.out.printf("%8.0f", arr[i]);
            System.out.print(" ");
        }
        System.out.println(); 
    }
    
    protected show1DArr() {
        //Вывод одномерного массива
        System.out.println("Одномерный массив: ");
        showArray(array);
    }
    
    public  inputFromConsole() {
        Scanner sc = new Scanner(System.in);
        System.out.print("Введите размерность массива: ");
        int dimM = sc.nextInt();
        System.out.println("Введите нижнетреугольную матрицу размерности " + dimM);
        matrix = new double [dimM][dimM];
        
        int k = dimM;
        
        //Заполнение треугольной матрицы по строкам слева направо
        for (int i = 0; i < dimM; i++){
            for (int j = 0 ; j < dimM - k + 1; j++){
                System.out.print("Введите элемент a(" + i + ", " + j + "): ");
                matrix[i][j] = sc.nextDouble();                
            }
            k = k - 1;
        }
        
        //Заполнение столбца свободных членов
        free = new double [dimM];
        for (int i = 0; i < dimM; i++){
            System.out.print("Введите свободный член B[" + i + "]: ");
            free[i] = sc.nextDouble();
        }
        
        //Расширенная матрица
        augmatrix = new double [dimM][dimM + 1];
        for (int i = 0; i < dimM; i++){
            for (int j = 0; j < dimM; j++){
                augmatrix[i][j] = matrix[i][j];
            }            
        }
        for (int i = 0; i < dimM; i++){
            augmatrix[i][dimM] = free[i];
        }
        
        //Размерность одномерного массива и его объявление
        int dimA = (dimM * (dimM  + 1)) / 2; 
        array = new double [dimA];
        
        //Забивание треугольного массива в одномерный
        for (int i = 0; i < dimM; i++){
            for (int j = 0; j < dimM; j++ ){
                int x = i * (i + 1) / 2 + j;
                array[x] = matrix[i][j];
            }
        }
    }
    
    int getEquationCount() {
        return dimM;
    }   
    
    //columnIndex - иксы
    double getCoef(int eqationIndex, int columnIndex) {
        return matrix[eqationIndex][columnIndex]; //Но в задании надо доставать элемент из одномерного массива
    }
    
    
    double getFree(int eqationIndex) {
        return free[eqationIndex];
    }
 
}
 
/* ----------------------------------------------------------------------------------- */
 
// абстрактный решатель уравнений
interface EquationSysSolver {
    double[] solve(SysEqMembers input);
}
 
/* ----------------------------------------------------------------------------------- */
 
// решатель методом Гаусса
class GaussianElemination implements EquationSysSolver {
    @Override
    double[] solve(SysEqMembers input) {
        /* ===== Вычисление иксов ===== */
    }
}
 
 
/* ----------------------------------------------------------------------------------- */
 
//.......
 
public static void main(String[] args) {
         //Ввод данных
    SysEqMembers in = new SysEqMembers();
    in.inputFromConsole();
    // Вычисление
    EquationSysSolver s = new GaussianElemination();    
    double[] result = s.solve(in);
    //Вывод ответа
    System.out.println("Ответ: ");
    SysEqMembers.showArray(result);
    
}

С отступами и каментами полный порядок, разбирать было не очень сложно.

Новые блоги и статьи Все статьи Все блоги /
сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .
Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .	Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и источниками (напряжения, ЭДС и тока). Найти токи и напряжения во всех элементах. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и. . .