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

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

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

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

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

Новые блоги и статьи Все статьи Все блоги /
Видеокарта простаивает ночами? Вот 4 проекта, которые загрузят её наукой Programma_Boinc 10.04.2026 Видеокарта простаивает ночами? Вот 4 проекта, которые загрузят её наукой Если на Windows стоит дискретная NVIDIA или AMD — можно отдать её вычислительную мощность реальным исследованиям. . . .	Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .	модель ЗдравоСохранения 8. Подготовка к разному выполнению заданий anaschu 08.04.2026 https:/ / github. com/ shumilovas/ med2. git main ветка * содержимое блока дэлэй из старой модели теперь внутри зайца новой модели 8ATzM_2aurI	Блокировка документа от изменений, если он открыт у другого пользователя Maks 08.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в конфигурации КА2. Задача: запретить редактирование документа, если он открыт у другого пользователя. / / . . .
Система безопасности+живучести для сервера-слоя интернета (сети). Двойная привязка. Hrethgir 08.04.2026 Далее были размышления о системе безопасности. Сообщения с наклонным текстом - мои. А как нам будет можно проверить, что ссылка наша, а не подделана хулиганами, которая выбросит на другую ветку и. . .	Модель ЗдрввоСохранения 7: больше работников, больше ресурсов. anaschu 08.04.2026 работников и заданий может быть сколько угодно, но настроено всё так, что используется пока что только 20% kYBz3eJf3jQ	Дальние перспективы сервера - слоя сети с космологическим дизайном интефейса карты и логики. Hrethgir 07.04.2026 Дальнейшее ближайшее планирование вывело к размышлениям над дальними перспективами. И вот тут может быть даже будут нужны оценки специалистов, так как в дальних перспективах всё может очень сильно. . .	Горе от ума kumehtar 07.04.2026 Эта мне ментальная установка, что вот прямо сейчас, мол, мне для полного счастья не хватает (нужное вписать), и когда я этого достигну - тогда и полный кайф. Одна из самых сильных ловушек на пути. . . .