不良反应.js性能与相关数据和双向绑定

时间:2015-04-10 20:20:30

标签: javascript reactjs

为了熟悉react.js,我写了一篇类似Facebook的新闻流,包括用户评论。我尝试使用户名可更改,因此我在用户对象和feed组件之间使用了双向绑定。但它并不是非常有效,我不知道这是一个反应问题,还是我有一个概念问题。

问题案例:

Here是一个关于JSFiddle的简单示例。在页面顶部,您可以更改用户名并对所有名称进行更新。

我从我自己的API获取数据,格式如下:

{
    "feeds": [
        {
            "_id": "feed_1",
            "user": "user_1",
            "text": "Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam nonumy eirmod tempor invidunt ut labore et dolore magna aliquyam",
            "comments": [
                {
                    "_id": "comment_1",
                    "user": "user_2",
                    "content": "Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam nonumy eirmod tempor invidunt ut labore et dolore magna aliquyam erat, sed diam voluptua."
                },
                ...
            ]
        },
        ...
    ],
    "users": [
        {
            "_id": "user_1",
            "name": "Allan Moreno"
        },
        {
            "_id": "user_2",
            "name": "Edward Mendez"
        }
        ...
    ]
}

所以我有一个包含feed的数组和一个包含用户的数组。我遍历Feed以生成反应Feed组件。提要和评论从 getUserById 函数中获取用户,并将其作为react属性传递。

var StreamContainer = React.createClass({
    getUserById: function(userId) {
        for(var i=0; i<this.state.users.length; i++) {
            if(this.state.users[i]._id === userId) {
                return this.state.users[i];
            }
        }
    },

    getInitialState: function() {
        return {
            "feeds": [...],
            "users": [...]
        };
    },

    render: function() {
        var getUserById = this.getUserById;

        return (
            <div className="stream-container">
                <FeedList getUserById={getUserById} feeds={this.state.feeds} />
            </div>
        );
    }
});

var FeedList = React.createClass({
    render: function() {
        var getUserById = this.props.getUserById;

        var feedNodes = this.props.feeds.map(function(feed) {
            return [<Feed key={feed._id} feed={feed} getUserById={getUserById} />];
        });

        return (
            <div>
                <strong>Feeds:</strong>
                <ol className="stream-items">
                    {feedNodes}
                </ol>
            </div>
        );
    }
});

var Feed = React.createClass({
    render: function() {
        var getUserById = this.props.getUserById;

        return (
            <li className="stream-item">
                <strong>{getUserById(this.props.feed.user).name}:</strong> <em>{this.props.feed.text}</em>
                <FeedCommentList comments={this.props.feed.comments} getUserById={getUserById} />
            </li>
        );
    }
});

React.render(
    <StreamContainer />,
    document.getElementsByTagName('body')[0]
);

现在我试图让它在一些性能测试中有点互动。我已添加了所有用户的列表,以动态更改其名称。对于双向绑定,我使用ReactLink

var StreamContainer = React.createClass({
    mixins: [React.addons.LinkedStateMixin],

    ...

    render: function() {
        ...

        var valueLink = this.linkState('users');
        var handleChange = function(e) {
            valueLink.requestChange(e.target.value);
        };

        return (
            <div className="stream-container">
                <UserList users={this.state.users} valueLink={valueLink} />
                <FeedList getUserById={getUserById} feeds={this.state.feeds} />
            </div>
        );
    }
});

var UserList = React.createClass({
    render: function() {
        var valueLink = this.props.valueLink;

        var users = this.props.users.map(function(user, index) {
            var thisValueLink = {};

            var handleChange = function(newValue) {
                valueLink.value[index].name = newValue;
                valueLink.requestChange(valueLink.value);
            };

            thisValueLink.requestChange = handleChange;
            thisValueLink.value = valueLink.value[index].name;

            return [<User key={user._id} user={user} valueLink={thisValueLink} />];
        });

        return (
            <div>
                <strong>Users:</strong>
                <ul>
                    {users}
                </ul>
            </div>
        );
    }
});

var User = React.createClass({
    render: function() {
        return (
            <li>                            
                <input type="text" valueLink={this.props.valueLink} /> {this.props.user.name} 
            </li>
        );
    }
});

但现在我遇到了性能问题。如果我更改用户的名称,则必须检查所有名称以进行更新。因此,如果我有100个Feed,您可以看到输入和更新之间的延迟。

是否有更好的方法将用户传递给子组件?或者可能是一种更高效的方式来处理双向绑定?

1 个答案:

答案 0 :(得分:0)

您可以在StreamContainer渲染中创建一个用户名词典对象,以传递给需要查找用户名的子项:

var userNames = {};
this.state.users.forEach(function(user) {
    userNames[user._id] = user.name;
});

<FeedList usernames={userNames} feeds={this.state.feeds} />

然后它只是一个简单的键查找,例如:

var Feed = React.createClass({
    render: function() {
        return (
            <li className="stream-item">
                <strong>{this.props.usernames[this.props.feed.user]}:</strong> <em>{this.props.feed.text}</em>
                <FeedCommentList comments={this.props.feed.comments} usernames={this.props.usernames} />
            </li>
        );
    }
});



var FeedCommentList = React.createClass({
    render: function() {
        var _this = this;
        var commentNodes = this.props.comments.map(function(comment) {
            return [
                <FeedComment 
                    key={comment._id} 
                    username={_this.props.usernames[comment.user]} 
                    comment={comment} />
            ];
        });

...

updated jsfiddle