Jak warunkowo opakować komponent React?

86

Mam komponent, który czasami trzeba będzie renderować jako plik, <anchor>a innym razem jako plik <div>. propCzytam w celu ustalenia tego jest this.props.url.

Jeśli istnieje, muszę wyrenderować komponent opakowany w plik <a href={this.props.url}>. W przeciwnym razie zostanie po prostu renderowany jako plik <div/>.

Możliwy?

Oto, co teraz robię, ale czuję, że można to uprościć:

if (this.props.link) {
    return (
        <a href={this.props.link}>
            <i>
                {this.props.count}
            </i>
        </a>
    );
}

return (
    <i className={styles.Icon}>
        {this.props.count}
    </i>
);

AKTUALIZACJA:

Oto ostateczne zamknięcie. Dzięki za wskazówkę, @Sulthan !

import React, { Component, PropTypes } from 'react';
import classNames from 'classnames';

export default class CommentCount extends Component {

    static propTypes = {
        count: PropTypes.number.isRequired,
        link: PropTypes.string,
        className: PropTypes.string
    }

    render() {
        const styles = require('./CommentCount.css');
        const {link, className, count} = this.props;

        const iconClasses = classNames({
            [styles.Icon]: true,
            [className]: !link && className
        });

        const Icon = (
            <i className={iconClasses}>
                {count}
            </i>
        );

        if (link) {
            const baseClasses = classNames({
                [styles.Base]: true,
                [className]: className
            });

            return (
                <a href={link} className={baseClasses}>
                    {Icon}
                </a>
            );
        }

        return Icon;
    }
}
Brandon Durham
źródło
Możesz również przenieść się const baseClasses =do tej if (this.props.link)gałęzi. Ponieważ używasz ES6, możesz także nieco uprościć, const {link, className} = this.props;używając linki classNamejako zmiennych lokalnych.
Sulthan
Człowieku, kocham to. Uczymy się coraz więcej o ES6 i zawsze poprawia to czytelność. Dzięki za dodatkową wskazówkę!
Brandon Durham
1
Co to jest „ostateczne zablokowanie”?
Chris Harrison,

Odpowiedzi:

93

Po prostu użyj zmiennej.

var component = (
    <i className={styles.Icon}>
       {this.props.count}
    </i>
);

if (this.props.link) {
    return (
        <a href={this.props.link} className={baseClasses}>
            {component}
        </a>
    );
}

return component;

lub możesz użyć funkcji pomocniczej do renderowania zawartości. JSX to kod jak każdy inny. Jeśli chcesz zredukować powielanie, użyj funkcji i zmiennych.

Sulthan
źródło
22

Utwórz HOC (komponent wyższego rzędu) do pakowania elementu:

const WithLink = ({ link, className, children }) => (link ?
  <a href={link} className={className}>
    {children}
  </a>
  : children
);

return (
  <WithLink link={this.props.link} className={baseClasses}>
    <i className={styles.Icon}>
      {this.props.count}
    </i>
  </WithLink>
);
Wykonaj Async
źródło
4
HOC powinien umrzeć powoli: P
Jamie Hutber
Termin HOCjest ohydny. To tylko funkcja umieszczona pośrodku. Naprawdę wypieram tę nagle modną nazwę „HPC”. co jest tak ważnego w prostej funkcji, która jest umieszczona pomiędzy ... starą koncepcją od dziesięcioleci.
vsync
12

Oto przykład przydatnego komponentu, z którego korzystałem (nie wiem, komu go akredytować), który spełnia swoje zadanie:

const ConditionalWrap = ({ condition, wrap, children }) => (
  condition ? wrap(children) : children
);

Przypadek użycia:

<ConditionalWrap condition={someCondition}
  wrap={children => (<a>{children}</a>)} // Can be anything
>
  This text is passed as the children arg to the wrap prop
</ConditionalWrap>
Antoniego
źródło
2
Kredyt powinien prawdopodobnie znaleźć się tutaj: gist.github.com/kitze/23d82bb9eb0baabfd03a6a720b1d637f
Roy Prins
Widziałem to z kitze. Ale nie byłem pewien, czy wpadł na pomysł kogoś innego
Antony'ego
Ani ja. To był pierwszy wynik, który się pojawił i założyłem, że to źródło - a przynajmniej bliższe;).
Roy Prins
Powinieneś używać wrapdeklaratywnego, a nie funkcji, aby zachować bardziej "Reagowanie" -spirit
vsync
Jak uczynić go bardziej deklaratywnym @vsync? Myślałem, że rekwizyty do renderowania są zgodne z duchem Reacta?
antony
10

Istnieje inny sposób użycia zmiennej referencyjnej

let Wrapper = React.Fragment //fallback in case you dont want to wrap your components

if(someCondition) {
    Wrapper = ParentComponent
}

return (
    <Wrapper parentProps={parentProps}>
        <Child></Child>
    </Wrapper>

)
Avinash
źródło
Możesz skondensować pierwszą połowę dolet Wrapper = someCondition ? ParentComponent : React.Fragment
mpoisot
To jest niesamowite, ale czasami chcesz zachować kod deklaratywny , co oznacza, że ​​zwraca tylko JSX
vsync
React.Fragment can only have 'key' and 'children'
Pojawia
@vsync musisz dodać warunek dla rekwizytów, a także coś takiego jak propId = {someCondition? parentProps: undefined} ..
Avinash
1
Wiem :) Pisałem to ze względu na dokumentację dla innych, którzy przychodzą tu z tym problemem, więc Google zapisze tę stronę w swoich wynikach wyszukiwania dla tych słów kluczowych
vsync
1

Możesz również użyć funkcji util, takiej jak ta:

const wrapIf = (conditions, content, wrapper) => conditions
        ? React.cloneElement(wrapper, {}, content)
        : content;
Tomek
źródło
0

Powinieneś użyć JSX if-else, jak opisano tutaj . Coś takiego powinno działać.

App = React.creatClass({
    render() {
        var myComponent;
        if(typeof(this.props.url) != 'undefined') {
            myComponent = <myLink url=this.props.url>;
        }
        else {
            myComponent = <myDiv>;
        }
        return (
            <div>
                {myComponent}
            </div>
        )
    }
});
user2027202827
źródło
-2

Funkcjonalny komponent, który renderuje 2 komponenty, jeden jest opakowany, a drugi nie.

Metoda 1:

// The interesting part:
const WrapIf = ({ condition, With, children, ...rest }) => 
  condition 
    ? <With {...rest}>{children}</With> 
    : children

 
    
const Wrapper = ({children, ...rest}) => <h1 {...rest}>{children}</h1>


// demo app: with & without a wrapper
const App = () => [
  <WrapIf condition={true} With={Wrapper} style={{color:"red"}}>
    foo
  </WrapIf>
  ,
  <WrapIf condition={false} With={Wrapper}>
    bar
  </WrapIf>
]

ReactDOM.render(<App/>, document.body)
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/16.6.3/umd/react.production.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/react-dom/16.6.3/umd/react-dom.production.min.js"></script>

Można tego również użyć w następujący sposób:

<WrapIf condition={true} With={"h1"}>

Metoda 2:

// The interesting part:
const Wrapper = ({ condition, children, ...props }) => condition 
  ? <h1 {...props}>{children}</h1>
  : <React.Fragment>{children}</React.Fragment>;   
    // stackoverflow prevents using <></>
  

// demo app: with & without a wrapper
const App = () => [
  <Wrapper condition={true} style={{color:"red"}}>
    foo
  </Wrapper>
  ,
  <Wrapper condition={false}>
    bar
  </Wrapper>
]

ReactDOM.render(<App/>, document.body)
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/16.6.3/umd/react.production.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/react-dom/16.6.3/umd/react-dom.production.min.js"></script>

vsync
źródło