EF/UIView 在查找model值的时候有个隐含的bug

[acelan86]

      function getProperty(target, path) {
            var value = target;
            for (var i = 0; i < path.length; i++) {
                value = value[path[i]];
            }

            return value;
        }

ef/UIView这块代码有点隐藏的bug

   getProperty({}, ['a', 'b']); //会报错，因为obj.a === undefined

因为模板中怎么写程序是没法预知的，所以，这里的查找放到try里面更好些

[acelan86]

如果不想用try，预判下value是否为对象，默认给值，大概如下：

function getProperty(target, path) {
            var value = target;
            for (var i = 0; i < path.length; i++) {
                if (!(value && 'object' == typeof value)) {
                    value = {};
                }
                value = value[path[i]];
            }

            return value;
        }

[otakustay]

我觉得本质问题是模板这块不会抛出错来，这边异常是应该的，不然无法判断是真undefined还是别的问题……

但我看了下esui/main的代码，似乎应该是会抛出来异常的才对，你这边在控制台看不到相关的错误吗？

[acelan86]

看得到，我就是看到了跟进去才发现这个问题的。

我的情景是这样的，创建跟编辑的业务我写成同一个模板，当我新建的时候model是没有任何数据的，这时候form填充数据的时候访问到了多级的属性报错，这种是不应该发生的吧

[acelan86]

表单回填遇到没有数据不应该报错，应该为空，或者不应该在这个层面吃掉错误，我相信灰大有更好的做法

[otakustay]

大概知道情况了，这得分2种情况来看：

1. 表单回填这种，等于是我预期会出现为空的情况
2. 另一种情况，我并不预期会为空，这里相当于校验一下给模板的数据结构是不是对的

我觉得应该分开来讨论，不如加一种符号……

[acelan86]

我的观点哈，加一种语法或者符号都会增加使用难度，我在没有文档的情况下真心遇到好多问题，这些是阻碍推广的关键，另外这种模板数据我觉得只是展现，完全可以降低报警级别，变成warnning就好了，或者可以提供模板块的统一语法配置，但是一定不是默认的

[otakustay]

我并不知道这个变化直接上破坏向前兼容性会不会有影响

如果我们在UIView上加个开关控制这个行为，但默认行为依旧保持现状呢