Криворукий код и его критика

SlavaSokolov · Регистрация: 14.04.2016

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

Необходима критика криворукого кода. Желательно прилагать пояснения что не так, или хотя бы наставить чайник на путь истинный). Генератор паролей. Использовал два класса.

Кликните здесь для просмотра всего текста

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
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
import javafx.application.Application;
import javafx.event.ActionEvent;
import javafx.event.EventHandler;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.MenuBar;
import javafx.scene.control.TextField;
import javafx.scene.layout.AnchorPane;
import javafx.stage.Stage;
 
 
import java.awt.*;
import java.awt.datatransfer.StringSelection;
 
 
public class Main extends Application {
    Logic logic = new Logic();
 
    private Stage stage; private AnchorPane pane; private Scene scene;
 
    private MenuBar mainMenu;
 
    private TextField passwordField, lenpassword, quantityABC, quantityNumbers;
 
    private Button generat, clearpassword, copypassword, clearQABC, clearQNumber, clearlenPassword;
 
 
 
    @Override
    public void start(Stage primaryStage) throws Exception{
 
        this.stage = primaryStage;
        pane = new AnchorPane();
        scene = new Scene(pane, 710, 360);
        stage.setScene(scene);
        stage.setResizable(false);
 
        mainMenu = new MenuBar();
        pane.getChildren().add(mainMenu);
        mainMenu.setPrefSize(720, 30); mainMenu.setLayoutX(0); mainMenu.setLayoutY(0);
        
        creatPasswordMenu();
        creatProperties();
        initialize();
        stage.show();
 
    }
 
    public void creatPasswordMenu(){
 
        passwordField = new TextField();
        pane.getChildren().add(passwordField);
        passwordField.setPrefSize(720, 30); passwordField.setLayoutX(0); passwordField.setLayoutY(30);
 
        copypassword = new Button("Копировать");
        pane.getChildren().add(copypassword);
        copypassword.setPrefSize(240, 30); copypassword.setLayoutX(0); copypassword.setLayoutY(60);
 
        generat = new Button("Генерировать");
        pane.getChildren().add(generat);
        generat.setPrefSize(240, 30); generat.setLayoutX(240); generat.setLayoutY(60);
 
        clearpassword = new Button("Очистить");
        pane.getChildren().add(clearpassword);
        clearpassword.setPrefSize(240, 30); clearpassword.setLayoutX(480); clearpassword.setLayoutY(60);
 
    }
 
    public void creatProperties(){
 
        lenpassword = new TextField();
        pane.getChildren().add(lenpassword);
        lenpassword.setPrefSize(240, 30); lenpassword.setLayoutX(0); lenpassword.setLayoutY(100);
 
        clearlenPassword = new Button("Очистить");
        pane.getChildren().add(clearlenPassword);
        clearlenPassword.setPrefSize(120, 30); clearlenPassword.setLayoutX(240); clearlenPassword.setLayoutY(100);
 
        quantityABC = new TextField();
        pane.getChildren().add(quantityABC);
        quantityABC.setPrefSize(240, 30); quantityABC.setLayoutX(0); quantityABC.setLayoutY(140);
 
        clearQABC = new Button("Очистить");
        pane.getChildren().add(clearQABC);
        clearQABC.setPrefSize(120, 30); clearQABC.setLayoutX(240); clearQABC.setLayoutY(140);
 
        quantityNumbers = new TextField();
        pane.getChildren().add(quantityNumbers);
        quantityNumbers.setPrefSize(240, 30); quantityNumbers.setLayoutX(0); quantityNumbers.setLayoutY(180);
 
        clearQNumber = new Button("Очистить");
        pane.getChildren().add(clearQNumber);
        clearQNumber.setPrefSize(120, 30); clearQNumber.setLayoutX(240); clearQNumber.setLayoutY(180);
    }
 
    public void initialize(){
        clearpassword.setOnAction(new EventHandler<ActionEvent>() {
            @Override
            public void handle(ActionEvent event) {
                passwordField.setText("");
            }
        });
        clearlenPassword.setOnAction(new EventHandler<ActionEvent>() {
            @Override
            public void handle(ActionEvent event) {
                lenpassword.setText("");
            }
        });
        clearQNumber.setOnAction(new EventHandler<ActionEvent>() {
            @Override
            public void handle(ActionEvent event) {
                quantityNumbers.setText("");
            }
        });
        clearQABC.setOnAction(new EventHandler<ActionEvent>() {
            @Override
            public void handle(ActionEvent event) {
                quantityABC.setText("");
            }
        });
        generat.setOnAction(new EventHandler<ActionEvent>() {
            @Override
            public void handle(ActionEvent event) {
                    butGenerat();
            }
        });
        copypassword.setOnAction(new EventHandler<ActionEvent>() {
            @Override
            public void handle(ActionEvent event) {
                StringSelection ss = new StringSelection(passwordField.getText());
                Toolkit.getDefaultToolkit().getSystemClipboard().setContents(ss, null);
            }
        });
    }
 
    public void butGenerat() {
        try {
            logic.sabc = quantityABC.getText();
            logic.slength = lenpassword.getText();
            logic.snumbers = quantityNumbers.getText();
        } catch (Exception e){
            passwordField.setText("Ошибка ввода данных!");
        }
        passwordField.setText(logic.generator());
    }
 
    public static void main(String[] args) {
        launch(args);
    }
}

Кликните здесь для просмотра всего текста

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
public class Logic {
 
    private String arrayabc = "qwertyuiopasdfghjklzxcvbnm", arraynumbers = "0987654321";
 
    String snumbers, sabc, slength;
 
    int numbers, abc, length;
 
    public String generator(){
        String password = "";
        boolean act;
        try {
            act = check();
        } catch (Exception e){
            act = false;
        }
        if (act == true){
            password = buildpassword();
            return password;
        } else {
            return "Ошибка ввода данных!";
        }
    }
 
    private boolean check(){
        numbers = Integer.parseInt(snumbers);
        abc = Integer.parseInt(sabc);
        length = Integer.parseInt(slength);
 
        if ((length >= numbers + abc) && !(numbers == 0 && abc == 0)){
            return true;
        } else {
            return false;
        }
    }
 
    private String buildpassword(){
        String password = "";
        int act;
 
        for (int index = 0; index < length; index++){
            act = (int) (Math.random() * 2);
            System.out.println(act);
            if (act == 0 && abc != 0){
                password += arrayabc.charAt((int) (Math.random() * arrayabc.length()));
                abc++;
            } else if (numbers != 0){
                password += arraynumbers.charAt((int) (Math.random() * arraynumbers.length()));
                numbers++;
            }
        }
        
        return password;
    }
}

@ntlinuxnt · 10.03.2017, 02:16

Не по теме:

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

Необходима критика криворукого кода.

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

Курите и будет вам счастье.

Новые блоги и статьи Все статьи Все блоги /
Установка Emscripten SDK (emsdk) и CMake на Windows для сборки C и C++ приложений в WebAssembly (Wasm) 8Observer8 30.01.2026 Чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. Система контроля версиями Git. . .	Подключение Box2D v3 к SDL3 для Android: физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .
Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL3_image 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .	Влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .	Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android 8Observer8 26.01.2026 Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .	Установка Android SDK, NDK, JDK, CMake и т.д. 8Observer8 25.01.2026 Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .