javascript - ShrinkSafe 缩小工具提示 : debugging Javascript code via JSLint: style or real errors?

Question

当我尝试通过 ShrinkSafe 缩小我的 Javascript 文件时，一个很棒的缩小工具，它输出一个空文件，这意味着我的 Javascript 文件显然包含一个错误，尽管在五个浏览器中一切都完美。所以我尝试了JSLint ，另一个很棒的网站，发现了数百个样式错误，我纠正了其中的大部分，但我不知道如何修复这些错误。哪些是样式提示，哪些可能是遗留的潜在错误？

非常感谢您对这些问题的风格或纠错提示。

JS LINT 说的是这些

# Errors
    
Problem at line 38 character 55: **Missing radix parameter.**

    if (settings.crossLinking && location.hash && parseInt(location.hash.slice(...

Problem at line 39 character 32: **Missing radix parameter**.

    var currentPanel = parseInt(location.hash.slice(1));

Problem at line 46 character 30: 'currentPanel' **is already defined**.

    var currentPanel = settings.firstPanelToLoad;

Problem at line 47 character 24: 'offset' **is already defined**.

    var offset = - (panelWidth*(currentPanel - 1));

Problem at line 53 character 30: 'currentPanel' **is already defined**.

    var currentPanel = 1;

Problem at line 108 character 33: **Missing radix parameter**.

    if (sliderCount === parseInt($(this).attr("rel").slice(12))) {

Problem at line 111 character 35: **Missing radix parameter**.

    targetPanel = parseInt($(this).attr("href").slice(1));

Problem at line 125 character 55: **Missing radix parameter**.

    if (settings.crossLinking && location.hash && parseInt(location.hash.slice(...

Problem at line 137 character 45: 'currentPanel' **used out of scope**.

    panelHeight = $('.panel:eq(' + (currentPanel - 1) + ')', slider).height();

Problem at line 150 character 73: **Expected** ';' **and instead saw** 'slider'.

    panelHeight = $('.panel:eq(' + x + ')', slider).height()

Problem at line 161 character 32: 'offset' **is already defined**.

    var offset = - (panelWidth*currentPanel);

Problem at line 168 character 69: 'offset' **used out of scope**.

    $('.panel-container', slider).animate({ marginLeft: offset }, settings.sl...

Answer 1

缺少基数——绝对应该修复； parseInt 不应该在没有基数参数的情况下使用，因为很容易引入错误(以“0”开头的字符串被解释为八进制)。

已定义 - 有时它可能是一个错误，因为您正在覆盖较早的值(您可能需要)。典型的临时变量，但如果您知道自己在做什么并且确实想重新定义变量，则安全。

超出范围使用——检查这是否真的是你想要的；这只是一个警告。

预期的“;”等等 -- 你可能错过了结尾“;”在声明的末尾。 JavaScript 会自动将其插入到行尾，但依赖此行为绝不是一个好主意，因为它会默默地插入“;”有时在错误的位置，使您的代码无法调试。

我没有看到任何明显的语法错误。 Shrinksafe 应该可以很好地缩小它。我建议您检查您的 Shrinksafe 参数，看看是否缺少任何内容。

还要考虑闭包编译器(简单模式)，它产生比 Shrinksafe 更严格的压缩。 Closure Compiler 就像 JSLint + Shrinksafe 合二为一。在线网络服务链接:http://closure-compiler.appspot.com/home