web-dev-qa-db-ja.com

JSLintのスペースとタブの混合エラー

JSLintを使用して以下を実行しました。

$(document).ready(function() {

    /*
        Add paragraph on page load
    */

    // Get all header elements
    var header = document.getElementsByTagName('h1'),
        parent,
        newP,
        text;

    // Loop through the elements
    for (var i=0, m = header.length; i < m; i++) {
        parent = header[i].parentNode;
        newP = document.createElement("p");
        text = document.createTextNode('This paragraph was inserted with JavaScript!');
        newP.appendChild(text);
        // Insert the new P element after the header element in its parent node
        parent.insertBefore(newP, header[i].nextSibling);   
    }

    // so much easier with jQuery!
    //$('.section > h1').after('<p>I am a new paragraph &amp; I have been added to the page with javascript!</p>');

    /*
        Toggle show/hide
    */

    // display show/hide link - hidden by default if JS disabled as functionality is not available
    $('#content > .section > h2 > a').removeClass('hide');

    // hide What's new on page load - all content will be available if JS disabled  
    $('#content > .section > ul').addClass('hide');

    // show/hide content on click event
    $('#content > .section > h2 > a').live('click',function() {

        $('#content > .section > ul').toggle();

        return false;

    });

    /*
        JSON
    */

    var $jsonURL = 'scripts/response.json';

    $.ajax({
        type: 'GET',
        url: $jsonURL,
        dataType: "json",
        success: function(data){

            $.each(data.data, function(i, data){

                var $html = '';

                var $string = '';

                if (data.type == 'comment') {
                    $string = 'file';
                } else {
                    $string = 'workspace';
                }

                $html += '<li class="' + data.type + '">';

                $html += '<strong>New ' + data.type + '</strong> was added to the ' + $string + ' ';

                $html += '<a href="' + data.target + '">' + data.target + '</a> ';

                $html += '<a href="' + data.workspace + '">' + data.workspace + '</a>';

                $html += ' by <a href="#">' + data.user + '</a>'; 

                $html += '</li>';   

                $('#content > .section > ul').append($html);    

            });

        },
        error:function (xhr, ajaxOptions, thrownError){
            alert(xhr.status);
            alert(thrownError);
        }           
    }); 

});

そして、このエラーが発生しています：

Error:

Problem at line 89 character 4: Mixed spaces and tabs.

}

Implied global: $ 1,31,34,37,39,51,57,81, document 1,8,16,17, alert 87,88

修正方法が本当にわからない？

javascriptjqueryjslint
23
RyanP13

このエラーは、インデントがスペースとタブの両方の組み合わせを使用している場合に発生します。たとえば、{SPACE}{SPACE}{TAB}{SPACE}または{TAB}{SPACE}{TAB}。なぜそれがエラーではなく警告ではないのか本当にわかりませんが、解決策は行を再確認して、スペースのみを使用するようにすることですORタブ。

タブとスペースの混在に関する問題は、ファイルを別のアプリケーションで表示すると、インデントの問題が発生する可能性があることです。たとえば、1人のユーザーが2つのスペースに等しいタブを構成している場合、別のユーザーは最初のユーザーのファイルを開いて、インデントが不均一になることがあります。どちらか一方を使用すると、コードが読みやすくなります。

興味深いことに、Stack Overflowはタブを4つのスペースに正規化するため、ここからコードをコピーしてJSLintに貼り付けると、問題が解決します。

34
Andy E

また、JSHintで使用可能な「smarttabs」オプションの使用を検討することもできます（JSHintは、JSLintのドロップイン代替品ですが、より優れています）。

この記事は非常に洞察に富んでおり、タブとスペースのトレードオフを客観的に説明しており（件名について多くのことを言うことができることにさえ気付いていませんでした）、スマートタブロジックの動作を示しています。

http://www.emacswiki.org/emacs/SmartTabs

基本的に、「インデント」にタブを使用する場合、スペースが「位置合わせのみ」に使用されている限り、つまりスペースの前に正しい数のインデントタブがある限り、「位置合わせ」にスペースを使用できます。

これにより、このコードスニペットが有効になります（ "---->"はタブを表します）。

function foo() {
---->var a = 4,
---->    b = 5,
---->    c = 6;
}

'.jshintrc'というファイルでこれを行うことができます。

{
    "smarttabs": true
}

または、コメント付きでソースコードに設定できます。

/*jslint smarttabs:true */

または、タブを完全に破棄することもできます...（宗教戦争が続く）。

個人的には、JSLintの派生プロジェクトであるJSHintを使用しています。ほとんどの目的で、これらは同じツールです。 http://jshint.com/docs/#options 。それをお勧めします。ほとんどのオプションは2つのツール間で共通です。

タブも使用しません。ずっと。選択肢があれば、私は2スペースの人です。

11
Dave Dopson