Я использую ReactJS + nextJS + Semantic-UI-React

У меня такой ответ json.

[
    {
        "artist": "Cardi B",
        "dob": "01/01/1990",
        "albums":[
            {
                "title": "Invasion of privacy",
                "year": "2018"
            }
        ],
        "country": "usa"
    },
    {
        "artist": "Michael Jackson",
        "dob": "01/01/1950",
        "albums":[
            {
                "title": "Thriller",
                "year": "1981"
            },
            {
                "title": "Blood On The Dance Floor",
                "year": "1995"
            }           
        ],
        "country": "usa"
    }
]

Я хотел бы показать каждого художника в таблице. Альбомы исполнителя будут показаны при щелчке по какой-либо ячейке строки исполнителя.

У меня 2 компонента. Один для исполнителя и один для альбома. Компонент AlbumTable - это, по сути, другая таблица, заключенная в <Table.Row>. В основном приведенный ниже код работает нормально.

toggleVisibility(index){
    let oldSt = this.state.AlbumStatus;
    oldSt[index] = !oldSt[index];
    this.setState({AlbumStatus: oldSt});
}


renderResult(){
    let rows = this.props.artists.map((r, index) =>{
        return(
             <Table.Body key={index}>
                  <Table.Row >
                        <Table.Cell>{r.artist}</Table.Cell>
                        <Table.Cell>{r.dob}</Table.Cell>
                        <Table.Cell onClick={() => this.toggleVisibility(index)}>+</Table.Cell>                 
                        <Table.Cell>{r.country}</Table.Cell>                            
                  </Table.Row>
                  <AlbumRow result={r.albums} hiddenStatus={!this.state.AlbumStatus[index]} />
              </Table.Body>
        );
    });
    return rows;
}


render(){
    return(
      <Table > 
            <Table.Header >
              <Table.Row>
                <Table.HeaderCell >Artists</Table.HeaderCell>
                <Table.HeaderCell >Dob</Table.HeaderCell>
                <Table.HeaderCell >Albums</Table.HeaderCell>                                    
              </Table.Row>
            </Table.Header>
            {this.renderResult()}
       </Table>             
    );      
}

Мое требование - показывать альбомы только при нажатии на знак «+» в строке исполнителя. Я могу переключать видимость, используя приведенный выше код.

Проблема:

Если я щелкну знак «+» в строке «Майкл Джексон», отобразятся его соответствующие альбомы. однако все альбомы артиста также перерисованы. Ожидается ли это ?? Будет ли это вызывать проблемы с производительностью для больших наборов данных? Каков наилучший подход, не сильно влияющий на производительность?

0
KitKarson 3 Июн 2018 в 02:07

1 ответ

Лучший ответ

Вероятно, это не большая проблема, если вы не заметите, что работа с большими наборами данных тормозит. Когда дело доходит до производительности, сначала измерьте, если вы считаете, что у вас есть проблема (подробнее об измерении производительности React). Не тратьте время на чрезмерную оптимизацию, пока не поймете, что у вас есть проблема.

Однажды может быть предложено разбить каждую строку на PureComponent :

import React, { PureComponent } 'react';

class AlbumTableRow extends PureComponent {
    render() {
        const {
            key,
            artist,
            dob,
            country
            albums,
            albumsHiddenStatus,
            onToggleVisibility,
        } = this.props;
        return(
            <Table.Body key={key}>
                <Table.Row >
                    <Table.Cell>{artist}</Table.Cell>
                    <Table.Cell>{dob}</Table.Cell>
                    <Table.Cell onClick={onToggleVisibility}>+</Table.Cell>
                    <Table.Cell>{country}</Table.Cell>
                </Table.Row>
                <AlbumRow result={albums} hiddenStatus={albumsHiddenStatus} />
            </Table.Body>
        );
    }
}

export default AlbumTableRow;

PureComponents должны повторно отображаться только при изменении значений свойства. Затем вы должны изменить код родительского компонента следующим образом:

getOnToggleVisibility = (index) => () => this.toggleVisibility(index);

renderResult(){
    let rows = this.props.artists.map((r, index) =>{
        return (
            <AlbumTableRow
                key={index}
                artist={r.artist}
                dob={r.dob}
                country={r.country}
                albums={r.albums}
                albumsHiddenStatus={!this.state.AlbumStatus[index]}
                onToggleVisibility={this.getOnToggleVisibility(index)}
            />
        );
    });
    return rows;
}

Обратите внимание, что я добавил метод getOnToggleVisibility, который принимает этот индекс и возвращает функцию onToggleVisibility, которая знает правильный индекс для использования.

0
Dave Lunny 3 Июн 2018 в 00:26