more-movie.js页面的一些简化

Question

data: {

    navigateTitle: "",

    movies:[],//重要

    requesUrl:"",

    totalCount:0

  },

processDoubanData:function(moviesDouban){

    var movies = this.data.movies;//重要

    for(var idx in moviesDouban.subjects){

      var subject = moviesDouban.subjects[idx];

      var title = subject.title;

      if(title.length>=6){

        title = title.substring(0,6) + "...";

      }

      var temp = {

        stars:util.convertToStarsArray(subject.rating.stars),

        title:title,

        average:subject.rating.average,

        coverageUrl:subject.images.large,

        movieId:subject

      }

      movies.push(temp);//重要

    }

    this.setData({

      totalCount:this.data.totalCount += 20

    })

    this.setData({

      movies:movies//重要

    });

    wx.hideNavigationBarLoading();

    wx.stopPullDownRefresh();

  },

关于加载数据这里，看老师写得好复杂，我标记重要这里是可以简化代码的

Answer 1

7七月 2017-01-18 17:38:45

好的，谢谢

Answer 2

7七月 2017-01-17 18:16:58

concat是要将每一次下拉加载更多的数据合并在一起，这个操作怎么去掉呢？

• 提问者 慕粉3921807 #1

  你不是写了movies.push(temp)这个了嘛，每次加载，movies就会多加内容
  然后 this.setData({ movies:movies});就行了嘛

Answer 3

7七月 2017-01-17 17:12:18

下面是最终版的moremovie代码，可以在这上面指出哪些代码可以简化

Answer 4

7七月 2017-01-17 17:11:47

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

// pages/movies/more-movie/more-movie.js var app = getApp() var util = require('../../../utils/util.js') Page({   data: {     movies: {},     navigateTitle: "",     requestUrl: "",     totalCount: 0,     isEmpty: true,   },   onLoad: function (options) {     var category = options.category;     this.data.navigateTitle = category;     var dataUrl = "";     switch (category) {       case "正在热映":         dataUrl = app.globalData.doubanBase +           "/v2/movie/in_theaters";         break;       case "即将上映":         dataUrl = app.globalData.doubanBase +           "/v2/movie/coming_soon";         break;       case "豆瓣Top250":         dataUrl = app.globalData.doubanBase + "/v2/movie/top250";         break;     }     wx.setNavigationBarTitle({       title: this.data.navigateTitle     })     this.data.requestUrl = dataUrl;     util.http(dataUrl, this.processDoubanData)   },     onScrollLower: function (event) {     var nextUrl = this.data.requestUrl +       "?start=" + this.data.totalCount + "&count=20";     util.http(nextUrl, this.processDoubanData)     wx.showNavigationBarLoading()   },     onPullDownRefresh: function (event) {     var refreshUrl = this.data.requestUrl +       "?star=0&count=20";     this.data.movies = {};     this.data.isEmpty = true;     this.data.totalCount = 0;     util.http(refreshUrl, this.processDoubanData);     wx.showNavigationBarLoading();   },     processDoubanData: function (moviesDouban) {     var movies = [];     for (var idx in moviesDouban.subjects) {       var subject = moviesDouban.subjects[idx];       var title = subject.title;       if (title.length >= 6) {         title = title.substring(0, 6) + "...";       }       // [1,1,1,1,1] [1,1,1,0,0]       var temp = {         stars: util.convertToStarsArray(subject.rating.stars),         title: title,         average: subject.rating.average,         coverageUrl: subject.images.large,         movieId: subject.id       }       movies.push(temp)     }     var totalMovies = {}       //如果要绑定新加载的数据，那么需要同旧有的数据合并在一起     if (!this.data.isEmpty) {       totalMovies = this.data.movies.concat(movies);     }     else {       totalMovies = movies;       this.data.isEmpty = false;     }     this.setData({       movies: totalMovies     });       this.data.totalCount += 20;     wx.hideNavigationBarLoading();     wx.stopPullDownRefresh()   },     onReady: function (event) {     wx.setNavigationBarTitle({       title: this.data.navigateTitle     })   },     onMovieTap: function (event) {     var movieId = event.currentTarget.dataset.movieid;     wx.navigateTo({       url: '../movie-detail/movie-detail?id=' + movieId     })   }, })

Answer 5

7七月 2017-01-17 17:11:33

你好。。。我没看出来你怎么简化的。关于写法的问题，讲课程和自己写项目是不一样的。讲课宁愿多写一些代码，也尽量不用简化。还是谢谢你。如果能指出如何简化就最好了。

Answer 6

提问者 慕粉3921807 2017-01-17 17:45:42

var movies = [];//这里如果写成var movies = this.data.movies;下面的几行就是多余的

if (!this.data.isEmpty) {

      totalMovies = this.data.movies.concat(movies);

    }

    else {

      totalMovies = movies;

      this.data.isEmpty = false;

    }