Переписать кусок кода

@antoniogrid · Регистрация: 20.07.2014

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

Всем привет

Можно ли переписать этот код более оптимально?

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
componentDidMount() {
    const url = "https://api.myjson.com/bla";
 
    fetch(url, {
      method: "GET"
    })
      .then(responce => responce.json())
      .then(posts => {
        const data = posts.map(obj => ({
          ProductId: +obj.ProductId,
          Sort: +obj.Sort,
          Title: obj.Title,
          Macronutrients: +obj.Macronutrients,
          Proteins: +obj.Proteins,
          Fats: +obj.Fats,
          Carbohydrates: +obj.Carbohydrates
        }));
 
        this.setState({ posts: data });
      });
  }

@antoniogrid · 27.10.2019, 18:55 **[ТС]**

Всем привет, в реакте

JavaScript
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
import React, { Component } from "react";
import "./App.css";
import ReactTable from "react-table";
import "react-table/react-table.css";
import ExportToExcel from "./ExportToExcel";
 
class App extends Component {
  constructor(props) {
    super(props);
 
    this.state = {
      posts: []
    };
  }
 
  componentDidMount() {
    const url = "https://api.myjson.com/bins/1fjdjc";
 
    fetch(url, {
      method: "GET"
    })
      .then(responce => responce.json())
      .then(posts => {
        const data = posts.map(obj => ({
          ProductId: +obj.ProductId,
          Sort: +obj.Sort,
          Title: obj.Title,
          Macronutrients: +obj.Macronutrients,
          Proteins: +obj.Proteins,
          Fats: +obj.Fats,
          Carbohydrates: +obj.Carbohydrates
        }));
 
        this.setState({ posts: data });
      });
  }
 
  deleteRow(id) {
    const index = this.state.posts.findIndex(post => {
      return post.ProductId === id;
    });
    this.state.posts.splice(index, 1);
    this.setState({ posts: this.state.posts });
  }
 
  render() {
    const columns = [
      {
        Header: "Product ID",
        accessor: "ProductId",
        style: {
          textAlign: "center"
        },
        width: 200,
        maxWidth: 200,
        minWidth: 200
      },
      {
        Header: "Sort",
        accessor: "Sort",
        sortable: true,
        filterable: true,
        style: {
          textAlign: "center"
        },
        width: 200,
        maxWidth: 200,
        minWidth: 200
      },
      {
        Header: "Group",
        accessor: "Macronutrients",
        sortable: true,
        filterable: true,
        style: {
          textAlign: "center"
        },
        width: 200,
        maxWidth: 200,
        minWidth: 200
      },
      {
        Header: "Title",
        accessor: "Title",
        sortable: true,
        filterable: true,
        style: {
          textAlign: "left"
        },
        width: 600,
        maxWidth: 600,
        minWidth: 300
      },
      {
        Header: "Proteins",
        accessor: "Proteins",
        sortable: true,
        filterable: true,
        style: {
          textAlign: "center"
        },
        width: 200,
        maxWidth: 200,
        minWidth: 200
      },
      {
        Header: "Fats",
        accessor: "Fats",
        sortable: true,
        filterable: true,
        style: {
          textAlign: "center"
        },
        width: 200,
        maxWidth: 200,
        minWidth: 200
      },
      {
        Header: "Carbs",
        accessor: "Carbohydrates",
        sortable: true,
        filterable: true,
        style: {
          textAlign: "center"
        },
        width: 200,
        maxWidth: 200,
        minWidth: 200
      },
      {
        Header: "Actions",
        Cell: props => {
          return (
            <button
              style={{ backgroundColor: "red", color: "#fefefe" }}
              onClick={() => {
                this.deleteRow(props.original.id);
              }}
            >
              Delete
            </button>
          );
        },
        sortable: false,
        filterable: false,
        style: {
          textAlign: "center"
        },
        width: 100,
        maxWidth: 100,
        minWidth: 100
      }
    ];
 
    return (
      <ReactTable
        columns={columns}
        data={this.state.posts}
        filterable
        defaultPageSize={15}
        noDataText={"PLease Wait..."}
        //showPagination = {false}
        showPaginationTop
      >
        {(state, filtredData, instance) => {
          this.reactTable = state.pageRows.map(post => {
            return post._original;
          });
          return (
            <div>
              {filtredData()}
              <ExportToExcel posts={this.reactTable} />
            </div>
          );
        }}
      </ReactTable>
    );
  }
}
export default App;

Можно ли упростить этот кусок?

JavaScript
1
2
3
4
5
6
7
8
9
10
then(responce => responce.json())
      .then(posts => {
        const data = posts.map(obj => ({
          ProductId: +obj.ProductId,
          Sort: +obj.Sort,
          Title: obj.Title,
          Macronutrients: +obj.Macronutrients,
          Proteins: +obj.Proteins,
          Fats: +obj.Fats,
          Carbohydrates: +obj.Carbohydrates

?

Комментарий модератора

Правила форума

5. Запреты и ограничения.
5.5 Запрещено размещать тему в нескольких подразделах одного раздела одновременно (кросспостинг), а также дублировать тему в одном разделе.

@amr-now · 27.10.2019, 20:19

antoniogrid, привет.
А там упрощать нечего. Строится новая коллекция из новых анонимных объектов.
Ничего особенного в этом куске не происходит.

@drmedus · 28.10.2019, 09:49

можно.
1) вместо then использовать async / await
2) сам fetch не должен находиться в componentDidMount. Нужно сделать для него обертку и пользоваться уже ей.
3) в then слишком большой кусок кода. Опять же оборачиваем в функцию и выносим. Это очень сильно повысит читабельность.

@antoniogrid · 28.10.2019, 23:11 **[ТС]**

А можете показать?

@drmedus · 29.10.2019, 09:17

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

А можете показать?

могу, но не буду. Это максимально простые вещи. Уверен, что вы справитесь.

@Casper-SC · 02.11.2019, 22:56

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

2) сам fetch не должен находиться в componentDidMount. Нужно сделать для него обертку и пользоваться уже ей.

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

А можете показать?

Что-то в этом духе

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
export default class ApiBase {
    encodeQueryData(data) {
        const result = [];
        for (const item in data)
        {
            if (data.hasOwnProperty(item)) {
                result.push(encodeURIComponent(item) + '=' + encodeURIComponent(data[item]));
            }
        }
        return result.join('&');
    }
 
    buildUrl(url, encodedGetParameters) {
        return `${url}?${encodedGetParameters}`
    }
}

JavaScript
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
import axios from 'axios'
import ApiBase from "./ApiBase"
 
export default class ServerApi extends ApiBase {
    constructor(config) {
        super();
        this.config = config;
    }
 
    singIn(login, password) {
        const authData = {
            login: login,
            password: password
        };
        return axios.post(this.config.authUrl, authData, {
            baseURL: this.config.baseUrl,
            withCredentials: true,
            headers: {
                "Content-Type": "application/json",
                "Accept": "*/*",
                "Cache-Control": "no-cache",
            }
        });
    };
 
    getValues() {
        return axios.get(this.config.valuesUrl, {
            baseURL: this.config.baseUrl,
            withCredentials: true,
            headers: {
                "Accept": "*/*",
                "Cache-Control": "no-cache",
            }
        });
    };
}

Добавлено через 2 минуты
В данном случае возвращается из методов Promise.

Добавлено через 2 минуты
У тебя может быть много разных сервисов, а не один ServerApi, поэтому есть ApiBase, от которого можно унаследоваться другим сервисом и вызывать его методы.

Добавлено через 1 минуту
Согласись, удобнее вызывать метод сервиса позволяющего получить доступ к API, чем копипастить код по компонентам.

Добавлено через 2 минуты
На самом деле, кстати, возвращать Promise библиотеки Axios тоже не совсем правильно, так как ты остаёшься привязан к реализации. Нужно возвращать данные, но тоже асинхронно.

Добавлено через 6 минут
Я не знаю, как принято в JavaScript (хотя общие принципы везде должны быть одинаковые), но в C# я бы возвращал именно данные (Task<T>, где T - тип возвращаемых данных, а Task - аналог Promise из JS) из такого класса-сервиса (ServerApi), а в случае ошибки кидал бы эксепшен.
Вот тебе пример того о чём я говорю (кидание эксепшена):
vue-realworld-example-app - api.service.js
Под возвратом именно данных я подразумеваю, что возвращается не response с какой-то инфой лишней, а только данные.

Добавлено через 1 минуту
response с лишней инфой - это (TypeScript):

JavaScript
1
2
3
4
5
6
7
8
export interface AxiosResponse<T = any>  {
  data: T;
  status: number;
  statusText: string;
  headers: any;
  config: AxiosRequestConfig;
  request?: any;
}

@drmedus · 03.11.2019, 21:14

я имел ввиду лишь то, что внутри componentDidMount вместо fetch нужно вызывать обертку, что-то вроде getData(). А вот уже getData реализует внутри себя получение. Это и читабельность повысит и избавит от потенциального дублирования кода.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer 8Observer8 08.02.2026 Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .	SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств 8Observer8 07.02.2026 Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .	SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .
SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .

@amr-now 6496 / 3907 / 2006 Регистрация: 14.06.2018 Сообщений: 6,781
	27.10.2019, 20:19
	antoniogrid, привет. А там упрощать нечего. Строится новая коллекция из новых анонимных объектов. Ничего особенного в этом куске не происходит. 0

@drmedus 27 / 22 / 6 Регистрация: 13.12.2012 Сообщений: 159
	28.10.2019, 09:49
	можно. 1) вместо then использовать async / await 2) сам fetch не должен находиться в componentDidMount. Нужно сделать для него обертку и пользоваться уже ей. 3) в then слишком большой кусок кода. Опять же оборачиваем в функцию и выносим. Это очень сильно повысит читабельность. 1

@antoniogrid 3 / 3 / 2 Регистрация: 20.07.2014 Сообщений: 654
	28.10.2019, 23:11 [ТС]
	А можете показать? 0

@drmedus 27 / 22 / 6 Регистрация: 13.12.2012 Сообщений: 159
	03.11.2019, 21:14
	я имел ввиду лишь то, что внутри componentDidMount вместо fetch нужно вызывать обертку, что-то вроде getData(). А вот уже getData реализует внутри себя получение. Это и читабельность повысит и избавит от потенциального дублирования кода. 0