Нужна критика кода "React, Redux"

@AlexKOR5 · Регистрация: 15.02.2019

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

Здравствуйте, нужна критика кода "React, Redux".
Правильно ли всё сделано?
Еще нет почему то рендеринга компонента "ResultDisplay", подскажите почему.
https://codepen.io/AlexKor5/pe... itors=1010

PHP/HTML
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
<!DOCTYPE html>
<html lang="en">
 
<head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>React</title>
    <!-- <link rel="stylesheet" href="./style.css"> -->
</head>
 
<body>
<div id="container"></div>
<!-- Redux -->
<script src="https://unpkg.com/redux@3.7.2/dist/redux.min.js"></script>
<!-- React -->
<script crossorigin src="https://unpkg.com/react@17/umd/react.development.js"></script>
<script crossorigin src="https://unpkg.com/react-dom@17/umd/react-dom.development.js"></script>
<script src="https://unpkg.com/babel-standalone@6/babel.min.js"></script>
 
<!-- My scripts -->
<script type="text/babel">
    const {Component} = React;
    const {createStore, combineReducers, compose, applyMiddleware} = Redux
    const SomeContext = React.createContext('top default value');
 
    //Actions
    const addElem = (name, surname, id) => ({
        type: "ADD_ELEMENT",
        name,
        surname,
        id
    })
    const removeElem = (id) => ({
        type: "REMOVE_ELEMENT",
        id
    })
 
    //Reducer
    const reducer = (state = [], action) => {
        switch (action.type) {
            case "ADD_ELEMENT":
                return [
                    ...state,
                    {
                        name: action.name,
                        surname: action.surname,
                        id: action.id
                    }
                ]
            case "REMOVE_ELEMENT":
                return state.filter((elem) => {
                    return elem.id !== action.id
                })
            default:
                return state
        }
    }
 
    const store = createStore(reducer);
    console.log(store.getState());
 
    class App extends Component {
        render() {
            return (
                <React.Fragment>
                    <SomeContext.Provider value={store}>
                        <InnerElem/>
                    </SomeContext.Provider>
                </React.Fragment>
            )
        }
    }
 
    const InnerElem = () =>
        <div>
            <h1>Some Title</h1>
            <MyForm/>
            <ResultDisplay/>
        </div>
 
    class MyForm extends Component {
        constructor(props) {
            super(props);
            this.show = this.show.bind(this);
            this.addData = this.addData.bind(this);
            this.removeData = this.removeData.bind(this);
        }
 
        show() {
            console.log(this._btnSend);
        }
 
        addData(event, store) {
            event.preventDefault();
            const {_addForm} = this;
            store.dispatch(addElem(_addForm.name.value, _addForm.surname.value, _addForm.id.value))
            console.log(store.getState());
        }
 
        removeData(event, store) {
            event.preventDefault();
            const {_removeForm} = this;
            store.dispatch(removeElem(_removeForm.id.value));
            console.log(store.getState());
        }
 
        render() {
            return (
                <SomeContext.Consumer>
                    {
                        store =>
                            <React.Fragment>
                                <h4>Add form</h4>
                                <form action="#" ref={el => this._addForm = el}>
                                    <input type="text" placeholder="Name" name="name" required/><br/>
                                    <input type="text" placeholder="Surname" name="surname" required/><br/>
                                    <input type="text" placeholder="id" name="id" required/><br/>
                                    <input type="submit" value="Sent to state"
                                           ref={el => this._btnSend = el}
                                           onClick={(event) => this.addData(event, store)}
                                    />
                                </form>
                                <br/>
                                <hr/>
                                <h4>Delete form</h4>
                                <form action="#" ref={el => this._removeForm = el}>
                                    <input type="text" placeholder="id" name="id" required/><br/>
                                    <input type="submit" value="Delete from state"
                                           ref={el => this._btnRemove = el}
                                           onClick={(event) => this.removeData(event, store)}
                                    />
                                </form>
                            </React.Fragment>
                    }
                </SomeContext.Consumer>
            )
        }
    }
 
    class ResultDisplay extends Component {
        render() {
            return (
                <div>
                    <span>#################### state_content ######################</span>
                    <div>
                        <SomeContext.Consumer>
                            {
                                store =>
                                    store.getState().map((elem) => {
                                        return (
                                            <span><b>Name:</b>elem.name</span>
                                        )
                                    })
                            }
                        </SomeContext.Consumer>
                    </div>
 
                </div>
            )
        }
    }
 
    ReactDOM.render(
        <App/>
        ,
        document.getElementById(`container`)
    );
</script>
</body>
</html>

@drmedus · 16.03.2021, 18:24

Используй функциональные компоненты вместо классовых и хуки. Будет сильно меньше кода и сильно больше читаемости и понимания происходящего.

Новые блоги и статьи Все статьи Все блоги /
модель ЗдравоСохранения 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 Эта мне ментальная установка, что вот прямо сейчас, мол, мне для полного счастья не хватает (нужное вписать), и когда я этого достигну - тогда и полный кайф. Одна из самых сильных ловушек на пути. . . .	Использование значений реквизитов справочника в документе, с определенными условиями и правами Maks 07.04.2026 1. Контроль срока действия договора Алгоритм из решения ниже реализован на примере нетипового документа "ЗаявкаНаРаботу", разработанного в конфигурации КА2. Задача: уведомлять пользователя, если. . .	Доступность команды формы по условию Maks 07.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: сделать доступной кнопку (команда формы "ЗавершитьСписание") при. . .

@drmedus 27 / 22 / 6 Регистрация: 13.12.2012 Сообщений: 159
	16.03.2021, 18:24
	Используй функциональные компоненты вместо классовых и хуки. Будет сильно меньше кода и сильно больше читаемости и понимания происходящего. 1