Vue.js计算的属性未更新

时间:2017-03-08 18:21:33

标签: javascript vue.js single-page-application

我正在使用Vue.js计算属性,但遇到了一个问题:在正确的时间调用计算方法 IS ,但忽略了计算方法返回的值!

我的方法

computed: {
    filteredClasses() {
        let classes = this.project.classes
        const ret = classes && classes.map(klass => {
            const klassRet = Object.assign({}, klass)
            klassRet.methods = klass.methods.filter(meth => this.isFiltered(meth, klass))
            return klassRet
        })
        console.log(JSON.stringify(ret))
        return ret
    }
}

console.log语句打印出的值是正确的,但是当我在模板中使用filteredClasses时,它只使用第一个缓存值并且永远不会更新模板。这是由Vue chrome devtools确认的(filteredClasses在初始缓存后永远不会更改。)

有人能告诉我为什么会这样吗?

Project.vue

<template>
<div>
    <div class="card light-blue white-text">
        <div class="card-content row">
            <div class="col s4 input-field-white inline">
                <input type="text" v-model="filter.name" id="filter-name">
                <label for="filter-name">Name</label>
            </div>
            <div class="col s2 input-field-white inline">
                <input type="text" v-model="filter.status" id="filter-status">
                <label for="filter-status">Status (PASS or FAIL)</label>
            </div>
            <div class="col s2 input-field-white inline">
                <input type="text" v-model="filter.apkVersion" id="filter-apkVersion">
                <label for="filter-apkVersion">APK Version</label>
            </div>
            <div class="col s4 input-field-white inline">
                <input type="text" v-model="filter.executionStatus" id="filter-executionStatus">
                <label for="filter-executionStatus">Execution Status (RUNNING, QUEUED, or IDLE)</label>
            </div>
        </div>
    </div>
    <div v-for="(klass, classIndex) in filteredClasses">
        <ClassView :klass-raw="klass"/>
    </div>
</div>
</template>

<script>
import ClassView from "./ClassView.vue"

export default {
    name: "ProjectView",

    props: {
        projectId: {
            type: String,
            default() {
                return this.$route.params.id
            }
        }
    },

    data() {
        return {
            project: {},
            filter: {
                name: "",
                status: "",
                apkVersion: "",
                executionStatus: ""
            }
        }
    },

    async created() {
        // Get initial data
        const res = await this.$lokka.query(`{
            project(id: "${this.projectId}") {
                name
                classes {
                    name
                    methods {
                        id
                        name
                        reports
                        executionStatus
                    }
                }
            }
        }`)

        // Augment this data with latestReport and expanded
        const reportPromises = []
        const reportMeta     = []
        for(let i = 0; i < res.project.classes.length; ++i) {
           const klass = res.project.classes[i];
           for(let j = 0; j < klass.methods.length; ++j) {
               res.project.classes[i].methods[j].expanded = false
               const meth = klass.methods[j]
               if(meth.reports && meth.reports.length) {
                   reportPromises.push(
                       this.$lokka.query(`{
                           report(id: "${meth.reports[meth.reports.length-1]}") {
                               id
                               status
                               apkVersion
                               steps {
                                   status platform message time
                               }
                           }
                       }`)
                       .then(res => res.report)
                    )
                    reportMeta.push({
                        classIndex: i,
                        methodIndex: j
                    })
                }
            }
        }

        // Send all report requests in parallel
        const reports = await Promise.all(reportPromises)

        for(let i = 0; i < reports.length; ++i) {
           const {classIndex, methodIndex} = reportMeta[i]
           res.project.classes[classIndex]
                      .methods[methodIndex]
                      .latestReport = reports[i]
       }

       this.project = res.project

       // Establish WebSocket connection and set up event handlers
       this.registerExecutorSocket()
   },

   computed: {
       filteredClasses() {
           let classes = this.project.classes
           const ret = classes && classes.map(klass => {
                const klassRet = Object.assign({}, klass)
                klassRet.methods = klass.methods.filter(meth => this.isFiltered(meth, klass))
                return klassRet
            })
            console.log(JSON.stringify(ret))
            return ret
        }
    },

    methods: {
        isFiltered(method, klass) {
            const nameFilter = this.testFilter(
                this.filter.name,
                klass.name + "." + method.name
            )
            const statusFilter = this.testFilter(
                this.filter.status,
                method.latestReport && method.latestReport.status
           )
           const apkVersionFilter = this.testFilter(
               this.filter.apkVersion,
               method.latestReport && method.latestReport.apkVersion
           )
           const executionStatusFilter = this.testFilter(
               this.filter.executionStatus,
               method.executionStatus
           )
           return nameFilter && statusFilter && apkVersionFilter && executionStatusFilter
       },
       testFilter(filter, item) {
           item = item || ""
           let outerRet = !filter ||
           // Split on '&' operator
           filter.toLowerCase().split("&").map(x => x.trim()).map(seg =>
               // Split on '|' operator
               seg.split("|").map(x => x.trim()).map(segment => {
                   let quoted = false, postOp = x => x
                   // Check for negation
                   if(segment.indexOf("!") === 0) {
                       if(segment.length > 1) {
                           segment = segment.slice(1, segment.length)
                           postOp = x => !x
                       }
                   }
                   // Check for quoted
                   if(segment.indexOf("'") === 0 || segment.indexOf("\"") === 0) {
                       if(segment[segment.length-1] === segment[0]) {
                           segment = segment.slice(1, segment.length-1)
                           quoted = true
                       }
                   }
                   if(!quoted || segment !== "") {
                       //console.log(`Item: ${item}, Segment: ${segment}`)
                       //console.log(`Result: ${item.toLowerCase().includes(segment)}`)
                       //console.log(`Result': ${postOp(item.toLowerCase().includes(segment))}`)
                   }
                   let innerRet = quoted && segment === "" ?
                       postOp(!item) :
                       postOp(item.toLowerCase().includes(segment))

                   //console.log(`InnerRet(${filter}, ${item}): ${innerRet}`)

                   return innerRet
               }).reduce((x, y) => x || y, false)
           ).reduce((x, y) => x && y, true)

           //console.log(`OuterRet(${filter}, ${item}): ${outerRet}`)
           return outerRet
       },
       execute(methID, klassI, methI) {
           this.project.classes[klassI].methods[methI].executionStatus = "QUEUED"
           // Make HTTP request to execute method
           this.$http.post("/api/Method/" + methID + "/Execute")
           .then(response => {
           }, error =>
               console.log("Couldn't execute Test: " + JSON.stringify(error))
           )
       },
       registerExecutorSocket() {
           const socket = new WebSocket("ws://localhost:4567/api/Executor/")

           socket.onmessage = msg => {
               const {methodID, report, executionStatus} = JSON.parse(msg.data)

               for(let i = 0; i < this.project.classes.length; ++i) {
                   const klass = this.project.classes[i]
                   for(let j = 0; j < klass.methods.length; ++j) {
                       const meth = klass.methods[j]
                       if(meth.id === methodID) {
                           if(report)
                               this.project.classes[i].methods[j].latestReport = report
                           if(executionStatus)
                               this.project.classes[i].methods[j].executionStatus = executionStatus
                           return
                       }
                   }
               }
           }
       },
       prettyName: function(name) {
           const split = name.split(".")
           return split[split.length-1]
       }
   },

   components: {
       "ClassView": ClassView
   }
}
</script>

<style scoped>
</style>

8 个答案:

答案 0 :(得分:11)

之前我遇到过类似的问题,并使用常规方法而不是计算属性来解决它。只需将所有内容移动到一个方法中即可返回。 Official docs.

答案 1 :(得分:3)

您需要为v-for中的列表项指定唯一键值。像这样......

<ClassView :klass-raw="klass" :key="klass.id"/>

否则,Vue不知道要使用哪些项目。这里的解释https://vuejs.org/v2/guide/list.html#key

答案 2 :(得分:2)

当值未定义时,我遇到了这个问题,然后计算无法检测到它的变化了。我通过给它一个空值或随机值来修复它,因为它将得到更新。希望能帮助到你。

答案 3 :(得分:1)

如果您打算在project.classes.someSubProperty更改时更新计算属性,则在定义计算属性时必须存在该子属性。 Vue无法检测到属性的添加或删除,只能更改现有属性。

当使用带有空state对象的Vuex存储时,这使我很痛苦。我对状态的后续更改不会导致依赖于状态的计算属性得到重新评估。向Veux状态添加具有空值的显式键可以解决该问题。

我不确定在您的情况下显式键是否可行,但这可能有助于解释为什么计算所得的属性过时。

Vue重新激活文档,了解更多信息: https://vuejs.org/v2/guide/reactivity.html#Change-Detection-Caveats

答案 4 :(得分:1)

如果在返回之前添加console.log,则可能可以在filteredClasses中看到计算值。

但是DOM由于某些原因不会更新。

然后您需要强制重新渲染DOM。

重新呈现的最佳方法是添加密钥作为计算值,如下所示。

<div
  :key="JSON.stringify(filteredClasses)" 
  v-for="(klass, classIndex) in filteredClasses"
>
  <ClassView
    :key="classIndex"
    :klass-raw="klass"
  />
</div>

警告

  

请勿将非原始值(例如对象和数组)用作键。请使用字符串或数字值。

这就是为什么我将数组filteredClasses转换为字符串的原因。 (可以有其他array-> string转换方法)

我也想说“建议尽可能使用v-for提供键属性”。

答案 5 :(得分:1)

对于这种情况,我有一种解决方法,我不知道您是否喜欢它。我在data()下放置一个整数属性(我们称其为trigger),每次我在计算属性中使用的对象更改时,它都会增加1。因此,这种方式,计算属性每次都会更新对象发生变化。

示例:

export default {
data() {
  return {
    trigger: 0, // this will increment by 1 every time obj changes
    obj: { x: 1 }, // the object used in computed property
  };
},
computed: {
  objComputed() {
    // do anything with this.trigger. I'll log it to the console, just to be using it
    console.log(this.trigger);

    // thanks to this.trigger being used above, this line will work
    return this.obj.y;
  },
},
methods: {
  updateObj() {
    this.trigger += 1;
    this.obj.y = true;
  },
},
};

这里正在link

答案 6 :(得分:0)

如果您在vue注册了对象以使其具有反应性之后将属性添加到返回的对象中,则当这些新属性更改时,它将不知道听这些属性。这是一个类似的问题:

let classes = [
    {
        my_prop: 'hello'
    },
    {
        my_prop: 'hello again'
    },
]

如果我将此数组加载到vue实例中,vue会将这些属性添加到其反应系统中,并能够监听它们的更改。但是,如果我从计算函数中添加新属性:

computed: {
    computed_classes: {
        classes.map( entry => entry.new_prop = some_value )
    }
}

new_prop所做的任何更改都不会导致vue重新计算该属性,因为我们从未在vues反应系统中实际添加classes.new_prop

要回答您的问题,您需要使用 之前存在的所有反应性属性构造对象,即使它们只是null。真正在使用vues反应系统的人都应该阅读以下链接:https://vuejs.org/v2/guide/reactivity.html

答案 7 :(得分:0)

对于在 Vue3 上遇到这个问题的其他人,我刚刚解决了这个问题,并且能够通过包装我从 {{1}使用提供的 this.$forceUpdate() 函数在引用中使用 [并且需要是反应式的] 函数,如下所示:

setup()

执行此操作后,对包装值的每个更改都会立即反映出来!