JavaScript代码分析-柏拉图的可维护性等级

Question

我已经使用JavaScript一段时间了，刚开始使用Plato分析我的代码。 我不确定它如何计算可维护性，但是下面的代码返回的可维护性得分为69.3。 我想念什么？ 尝试添加评论，并且没有改变。

/*globals jQuery*/
var App = App|| {};
App.AnimateSearch = (function ($) {
    'use strict';

    var searchContainer = $('[search-container]'),
        emptySearchMessage = $('.empty-search-message');

    function animateEmptyMessage() {
        emptySearchMessage.css({
            'opacity': 0,
            'transform': 'scale(0.5)',
            '-webkit-transform': 'scale(0.5)',
            '-moz-transform': 'scale(0.5)'
        });

        emptySearchMessage.fadeIn().animate({
            'opacity': 1,
            'transform': 'scale(1)',
            '-webkit-transform': 'scale(1)',
            '-moz-transform': 'scale(1)'
        }, 300);
    }

    function animateSearch(customClass) {
        searchContainer = typeof customClass === 'undefined' ? searchContainer : $(customClass);
        searchContainer.css({ 'margin-top': '100px', 'opacity': 0 });

        setTimeout(function () {
            searchContainer.stop().animate({ 'margin-top': '0', 'opacity': 1 }, 300);
        }, 500);
    }

    return {
        animateEmptyMessage: animateEmptyMessage,
        animateSearch: animateSearch
    };
}(jQuery));

感谢您的帮助/建议！

Answer 1

可维护性是许多不同参数的函数。 通常大约70的可维护性是完全可以接受的。 〜70 +是好的，30-70在警告区域，低于30通常是一个问题。 如果要提高分数，请尝试将其中某些CSS属性移动到带有动画的CSS类中。

您还仅依靠jquery在init中进行选择并在搜索中选择一个自定义容器。 最初的选择可以作为参数传递，并且对自定义类的支持似乎有点animateSearch ，这可能是为了支持事后添加的东西（为什么？因为函数名称为animateSearch ，然后支持可能与搜索无关的任何任意类）。

这些更改都将改善“可维护性”，但同样，〜70不一定是问题。 分数仅是重要的，因为它们与应用程序中的其他代码以及开发人员的舒适度有关。

也就是说，此代码很简单，但是随着添加一个解决方案，很容易失控。 通用动画解决方案将是一种抽象。 使用CSS动画并根据约定或某些应用程序级别的框架简单地添加类是另一种方法。 例如一些骨干组件，角度指令或Web组件，它们根据空虚情况或其他方式管理自己的状态。 这些单独的实现也很可能会在分数方面保持“可维护性”，但是这限制了重要的范围和潜在的未来发展。

如果文件从未更改，并且在写入后也不需要理解，那么可维护性就无关紧要。 如果文件将进行很多更改或需要完全理解才能解决所有将来的代码，则优先考虑可维护性。