【问题标题】:Write cleaner code for a dynamic object definition为动态对象定义编写更简洁的代码
【发布时间】:2017-12-17 09:59:16
【问题描述】:

我正在使用 vue 并尝试实现动态 axios GET 查询,该查询将根据搜索表单中填写的内容动态生成。现在我在单个文件组件中有这个方法:

更新:

<template lang="pug">
    div
        form(@submit.prevent="search")
            input(type='text', name="user_seq_id", placeholder="Customer ID")
            input(type='text', name="registered", placeholder="Registration date")
            input(type='text', name="email", placeholder="E-Mail")
            input(type='text', name="organization", placeholder="Organization")
            input(type='text', name="firstname", placeholder="Firstname")
            input(type='text', name="lastname", placeholder="Lastname")
            button(type="submit") Submit
</template>

<script>
    export default {
        data () {
            return {
            }
        },
        methods: {
            search(e) {
                let user_seq_id = e.target.elements.user_seq_id.value,
                    registered = e.target.elements.registered.value,
                    email = e.target.elements.email.value,
                    organization = e.target.elements.organization.value,
                    firstname = e.target.elements.firstname.value,
                    lastname = e.target.elements.lastname.value,
                    params = {
                        filter: 'role',
                        role: 'User',
                        sort: 'registered|desc'
                    }

                if ( user_seq_id ) {
                    _.set(params, 'user_seq_id', user_seq_id)
                    params.filter = params.filter + ',user_seq_id'
                }
                if ( registered ) {
                    _.set(params, 'registered', registered)
                    params.filter = params.filter + ',registered'
                }
                if ( email ) {
                    _.set(params, 'email', email)
                    params.filter = params.filter + ',email'
                }
                if ( organization ) {
                    _.set(params, 'organization', organization)
                    params.filter = params.filter + ',organization'
                }
                if ( firstname ) {
                    _.set(params, 'firstname', firstname)
                    params.filter = params.filter + ',firstname'
                }
                if ( lastname ) {
                    _.set(params, 'lastname', lastname)
                    params.filter = params.filter + ',lastname'
                }

                this.$store.commit('users', null)

                return this.$axios.get('/users', {
                        params: {
                            filter: 'role',
                            role: 'User',
                            sort: 'registered|desc'
                        }
                    })
                    .then( ({ data, headers, request, status, statusText }) => {
                        return this.$store.commit('users', data.result)
                    })
            }
        }
    }
</script>

这里有人知道以某种方式简化这个吗?我觉得每次都重复自己很垃圾,我相信有更好的方法来做到这一点。如您所见,我使用“lodash”将属性注入“param”对象。

提前致谢!!!

【问题讨论】:

  • e.targets.elements 示例?
  • 必须添加params .filter?
  • 我更新了代码来验证一下。

标签: javascript vue.js vuejs2 lodash axios


【解决方案1】:

由于您只想重构代码,因此我不考虑您用于设置 params 的逻辑。

你可以试试这个:

methods:{
   search(e) {

        let data = {
            user_seq_id: e.target.elements.user_seq_id.value,
            registered: e.target.elements.registered.value,
            email: e.target.elements.email.value,
            organization: e.target.elements.organization.value,
            firstname: e.target.elements.firstname.value,
            lastname: e.target.elements.lastname.value
       };

       let params = {
                filter: 'role',
                role: 'User',
                sort: 'registered|desc'
          };

    Object.keys(data).forEach((key) => {
        if(data[key]){
            this.modifyParams(params, key, data[key]);
        }
    });


        this.$store.commit('users', null)

        this.$axios.get('/users', {
            params: params
        })
        .then( ({ data, headers, request, status, statusText }) => {
            return this.$store.commit('users', data.result)
        })

    },
     modifyParams(obj, key, value){
        _.set(obj, key, value)
            obj.filter = obj.filter + ',' + key;
    }


}

PS:由于在 JavaScript 中对象是通过引用传递的,原始的 params 对象只会被修改

【讨论】:

  • 我更新了代码,虽然我不知道,清理它有什么帮助?!这些参数用于帮助 mongoDB 在数据库中进行查询。 params 字符串将用于稍后在我的 api 中动态构建 find()-Object。
  • 但是感谢您的建议。至少它有助于更​​动态地设置它:)
  • @PatrickDahm 你不希望你的代码重复。所以我重构了代码看起来更精简。
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 2015-10-27
  • 2018-06-05
  • 2012-05-08
  • 2021-11-17
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多