Разве это плохая практика - распаковывать «это» в React?

Question

Я всегда распаковываю это , чтобы функция рендеринга выглядела аккуратнее. Это плохая практика?

Пример:

class Zoom extends React.Component {
    // ...

    render() {
        const {view, zoomOutScreen} = this;
        const {navigation} = this.props;

        return (
            <Wrapper
                innerRef={view}
                animation='zoomIn'
                duration={200}
                useNativeDriver={true}
            >
                <Component
                    navigation={navigation}
                    zoomOutScreen={zoomOutScreen}
                />
            </Wrapper>
        );
    }
}

Answer 1

Неплохая практика, но она более читабельна и позволяет избежать анти-паттернов в React. Он дает пользователю информацию о том, какие объекты связаны с этим компонентом, а что еще относится к глобальным объектам. Так что предпочтительнее не разрушать его.

Answer 2

Это ни хорошо, ни плохо, это прежде всего вопрос стиля.

В некоторых сценариях использования может быть очень небольшая объективная разница, но об этом не стоит думать. (Если вы используете свойства несколько раз, кэширование их к локальной константе может ускорить их поиск. Если вы этого не сделаете, кэширование их к локальной константе - ненужный шаг. Но опять же, в 99,9999999999% случаев это просто не будет иметь никакого значения в реальном мире. Делайте то, что кажется ясным.)

---

Примечание: если хотите, вы можете объединить два задания по деструктуризации:

const {view, zoomOutScreen, props: {navigation}} = this;