在if / else条件中使用id是不好的做法吗?

时间:2015-04-23 20:33:57

标签: javascript angularjs if-statement

我对if / else条件有疑问,我的数据库保存了我用来实现if / else条件的区域ID 5657。

1-使用id是不好的做法吗?

2-在这种情况下实现if / else条件的正确方法是什么?

ctrl.js

if( geoLocation.items ) {
    $.each(geoLocation.parent(), function( index, location ) {
        if( location.id === 5657 ) {
            var disableItemId = 'disabled' + location.id;
            // Get the model
            var model = $parse( disableItemId );
            // Assigns a value to it
            model.assign( $scope, false );
        }
    });
    $.each( geoLocation.items, function( index, location ) {
        var disableItemId = 'disabled' + location.id;
        // Get the model
        var model = $parse( disableItemId );
        // Assigns a value to it
        model.assign( $scope, false );
    });
}

1 个答案:

答案 0 :(得分:-1)

理想情况下 - 在您的位置的数据级别,您将添加一个名为disabled或某事物的列/字段,并标记您想要达到该逻辑的记录。代码不是使用硬编码ID,而是代码如下:

if (location.disabled)

当然,如果您没有更新数据源/对象的访问权限或能力,那么您可能会像在那里那样坚持这样做。