コードを整理したい

Question

以下の通りコードを書いていますが、完全に重複しています。
これを重複を排除し、綺麗に描きたいのですが、やり方がわからず、ご教示頂けますと助かります。

$(document).on('turbolinks:load', function() {
  $('.thumb li').click(function() {
    var class_name, num;
    class_name = $(this).attr('class');
    num = class_name.slice(5);
    $('.main li').hide();
    $('.item' + num).fadeIn();
  });
  var i, len, ref, results, tag;
  $('#magazine-tags').tagit({
    fieldName: 'adviser[magazine_list]',
    singleField: true
  });
  if (gon.magazine_tags != null){
    ref = gon.magazine_tags;
    results = [];
    for (i = 0, len = ref.length; i < len; i++) {
      tag = ref[i];
      results.push($('#magazine-tags').tagit('createTag', tag));
    }
    return results;
  }
  var i, len, ref, results, tag;
  $('#brand-tags').tagit({
    fieldName: 'adviser[brand_list]',
    singleField: true
  });
  if (gon.brand_tags != null){
    ref = gon.brand_tags;
    results = [];
    for (i = 0, len = ref.length; i < len; i++) {
      tag = ref[i];
      results.push($('#brand-tags').tagit('createTag', tag));
    }
    return results;
  }
});

Answer 1

コードでわかりやすく説明したつもりになったら、やたら長くなりました・・・

繰り返しを纏めます。

$(document).on('turbolinks:load', function() {

  $('.thumb li').click(function() {
    var class_name, num;
    class_name = $(this).attr('class');
    num = class_name.slice(5);
    $('.main li').hide();
    $('.item' + num).fadeIn();
  });

 // 関数にまとめる(名前は適切に変更のこと)
  var initializeMagazineTags = function(){
    var i, len, ref, results, tag;
    $('#magazine-tags').tagit({
      fieldName: 'adviser[magazine_list]',
      singleField: true
    });
    if (gon.magazine_tags != null){
      ref = gon.magazine_tags;
      results = [];
      for (i = 0, len = ref.length; i < len; i++) {
        tag = ref[i];
        results.push($('#magazine-tags').tagit('createTag', tag));
      }
      return results;
    }
  }; 
  
 // 関数にまとめる(名前は適切に変更のこと)
  var initializeBrandTags = function(){
    var i, len, ref, results, tag;

    $('#brand-tags').tagit({
      fieldName: 'adviser[brand_list]',
      singleField: true
    });

    if (gon.brand_tags != null){
      ref = gon.brand_tags;
      results = [];
      for (i = 0, len = ref.length; i < len; i++) {
        tag = ref[i];
        results.push($('#brand-tags').tagit('createTag', tag));
      }
      return results;
    }
  }; 
  
  // 実行
 var magazineTags = initializeMagazineTags（）；
  var brandTags = initializeBrandTags();
});

共通部分の差分を変数にして関数の先頭に変数として定義する。
動作が変わっていないか確認してください。

$(document).on('turbolinks:load', function() {

  $('.thumb li').click(function() {
    var class_name, num;
    class_name = $(this).attr('class');
    num = class_name.slice(5);
    $('.main li').hide();
    $('.item' + num).fadeIn();
  });

 // 関数にまとめる
  var initializeMagazineTags = function(){
    var tagSelector = '#magazine-tags';
    var fieldName = 'adviser[magazine_list]';
    var initialTags = gon.magazine_tags;

    var i, len, ref, results, tag;
    $(tagSelector).tagit({
      fieldName: fieldName,
      singleField: true
    });
    if (initialTags != null){
      ref = initialTags;
      results = [];
      for (i = 0, len = ref.length; i < len; i++) {
        tag = ref[i];
        results.push($(tagSelector).tagit('createTag', tag));
      }
      return results;
    }
  }; 
  
 // 関数にまとめる
  var initializeBrandTags = function(){
    var tagSelector = '#brand-tags';
    var fieldName = 'adviser[brand_list]';
    var initialTags = gon.brand_tags;

    var i, len, ref, results, tag;
    $(tagSelector).tagit({
      fieldName: fieldName,
      singleField: true
    });

    if (initialTags != null){
      ref = initialTags;
      results = [];
      for (i = 0, len = ref.length; i < len; i++) {
        tag = ref[i];
        results.push($(tagSelector).tagit('createTag', tag));
      }
      return results;
    }
  }; 

  // 実行
 var magazineTags = initializeMagazineTags（）；
  var brandTags = initializeBrandTags();
});

2つの関数の後半が同じ物になりました。
動作が変わっていないか確認してください。

$(document).on('turbolinks:load', function() {

  $('.thumb li').click(function() {
    var class_name, num;
    class_name = $(this).attr('class');
    num = class_name.slice(5);
    $('.main li').hide();
    $('.item' + num).fadeIn();
  });

  // 共通する部分を関数にまとめる
 var initializeTags = function(tagSelector, fieldName, initialTags){
    var i, len, ref, results, tag;
    $(tagSelector).tagit({
      fieldName: fieldName,
      singleField: true
    });
    if (initialTags != null){
      ref = initialTags;
      results = [];
      for (i = 0, len = ref.length; i < len; i++) {
        tag = ref[i];
        results.push($(tagSelector).tagit('createTag', tag));
      }
      return results;
    }
  };

 // 関数にまとめる
  var initializeMagazineTags = function(){
    var tagSelector = '#magazine-tags';
    var fieldName = 'adviser[magazine_list]';
    var initialTags = gon.magazine_tags;

    // 関数呼び出しに変更する
    return initializeTags(tagSelector, fieldName, initialTags);
  }; 
  
 // 関数にまとめる
  var initializeBrandTags = function(){
    var tagSelector = '#brand-tags';
    var fieldName = 'adviser[brand_list]';
    var initialTags = gon.brand_tags;

    // 関数呼び出しに変更する
    return initializeTags(tagSelector, fieldName, initialTags);
  }; 

  // 実行
 var magazineTags = initializeMagazineTags（）；
  var brandTags = initializeBrandTags();
});

冗長系はなくなったと思います。
動作が変わっていないか確認してください。

initializeMagazineTagsとinitializeBrandTagsはもはや何もしていません。
共通関数を直接呼び出すように変更します。

$(document).on('turbolinks:load', function() {

  $('.thumb li').click(function() {
    var class_name, num;
    class_name = $(this).attr('class');
    num = class_name.slice(5);
    $('.main li').hide();
    $('.item' + num).fadeIn();
  });

  // 共通する部分を関数にまとめる
 var initializeTags = function(tagSelector, fieldName, initialTags){
    var i, len, ref, results, tag;
    $(tagSelector).tagit({
      fieldName: fieldName,
      singleField: true
    });
    if (initialTags != null){
      ref = initialTags;
      results = [];
      for (i = 0, len = ref.length; i < len; i++) {
        tag = ref[i];
        results.push($(tagSelector).tagit('createTag', tag));
      }
      return results;
    }
  };

  // マガジンのタグを初期化する
 var magazineTags = initializeTags(
        '#magazine-tags', 'adviser[magazine_list]', gon.magazine_tags);
  // ブランドのタグを初期化する
  var brandTags = initializeTags(
        '#brand-tags', 'adviser[brand_list]', gon.brand_tags); 
});

共通関数を直接呼び出すように変更します。

冗長化を減らすというならこのような手順がよいと思います。

この先の改善点としては、initializeTagsは３つのことを行っていて、一つの関数にまとめていいのかという問題があります。つまり、保存先のfield設定、タグの初期値の設定、現在のタグの取得です。

保存先のタグのfiled設定に関してはもともと１行なので、共通の関数の中に隠す必要があるか疑問です。
現在のタグの取得も本来の目的外で分けたほうがわかりやすいかもしれません。

Comments

[innjera]

とても丁寧なご説明ありがとう御座います！理解が深まりました！

Answer 2

jQuery日本語リファレンス
selector1, selector2, ..., selectorN
http://semooh.jp/jquery/api/selectors/multiple/

Comments

[innjera]

参考サイトのご教示有難うございました。参考にさせて頂きます。