Javascript 函数内部的排序脚本不'；我不能正常工作

我试图将一个脚本转换成一个函数，因为一遍又一遍地重复相同的代码是非常烦人的

脚本选择一个容器，该容器将很快容纳一个新的手风琴。容器中的每个手风琴都有一个唯一的数据排序id（需要从1到10排序） 然后，它检测是否有id小于或大于它的手风琴，并在它们之间添加自身

脚本工作正常，而不在函数中

这个没有函数的脚本可以正常工作

var$container=$（'.panel group[data group holder=“”+dataPlace+“]”）面板组，
$d=$container.find（'div.panel-default'），
$n=$（''）；
var$a=$d.filter（函数（）{
返回$（this）.data（'sort'）

此函数无法正常工作（无法排序）

函数placeSort（thisContainer、containerFind、placeHtml、returnCompareOneFind、returnCompareOne、returnCompareTwo、removeClone）{
var$container=$（thisContainer），
$d=$container.find（containerFind），
$n=$（placeHtml）；
if（returnCompareOneFind==“false”）{
var$a=$d.filter（函数（）{
return$（this）.data（returnCompareOne）

HTML

主要群体
01
02
03
04
05
06
07
08

---

创建一个JSFIDLE会花费太多的时间来重新创建它。很抱歉，我缺少它。

您的问题可能归结为试图传递此

的问题

尝试一下：

function addGroupAccordion(self, dataSort, dataPlace, dataHtml) {
    ...
    var removeClone = $(self).parents('div:eq(0)').remove();
    ...
}

$(document).on('click','.draggable-groups .main-groups',function(){
    ...
    var self = this; // 'self' is more standardized
    addGroupAccordion(self, dataSort, dataPlace, dataHtml);
    ...
});

这里的情况是，当您设置
var that=$（this）时$（）中使用code>。在
函数上，您存储了包含
此
的jQuery对象，然后当您试图使用
$（that）
在
addGroupAccordion
函数中使用它时，您要求jQuery执行：
$（$（this））
您是否定义了
$（that）
？在
var removeClone=$（that）.parents（'div:eq（0）'）.remove（）中将其复制到jsfiddler man。你不需要写它。不管怎样，函数外的代码和函数中的代码都有很大的变化。“不一样了。”塞尔吉奥我把我的第一个帖子修好了。谢谢你的注意。你能发布这段代码所使用的html吗？你是否调用了
addgroupaccordio（）
？我相信这是真的。我没有意识到“自我”，我会尝试使用这种方法。在那之前，我会接受这个答案，谢谢！
    function placeSort(thisContainer, containerFind, placeHtml, returnCompareOneFind, returnCompareOne, returnCompareTwo, removeClone) {
        var $container = $(thisContainer),
            $d = $container.find(containerFind),
            $n = $(placeHtml);

        if(returnCompareOneFind == "false") {
            var $a = $d.filter(function () {
                return $(this).data(returnCompareOne) < returnCompareTwo;
            }).last();
        } else {
            var $a = $d.filter(function () {
                return $(this).find(returnCompareOneFind).data(returnCompareOne) < returnCompareTwo;
            }).last();
        }

        removeClone;

        if ($a.length) $a.after($n);
        else $container.prepend($n);
    }

function addGroupAccordion(that, dataSort, dataPlace, dataHtml) {
    var thisContainer = '.panel-group[data-group-holder="' + dataPlace + '"]';
    var containerFind = 'div.panel-default';
    var placeHtml = '<div class="panel panel-default dragged-true" data-sort="' + dataSort + '"><div class="panel-heading"><h4 class="panel-title"><a class="accordion-toggle sub" data-sort="' + dataSort + '" data-toggle="collapse" href="#main-' + dataSort + '"><div class="remove set"><i class="icon-remove"></i></div>' + dataHtml + ' <b>(<span class="sub-count">0</span>)</b><i class="icon-angle-down"></i></a></h4></div><div id="main-' + dataSort + '" class="panel-collapse in"><div class="panel-body" data-accept="' + dataSort + '"></div></div></div>';
    var returnCompareOneFind = "false";
    var returnCompareOne = 'sort';
    var returnCompareTwo = dataSort;
    var removeClone = $(that).parents('div:eq(0)').remove();

    placeSort(thisContainer, containerFind, placeHtml, returnCompareOne, returnCompareTwo, removeClone);
}

$(document).on('click','.draggable-groups .main-groups',function(){
        var dataSort = $(this).attr("data-sort");
        var dataPlace = $(this).parents('div:eq(1)').attr("data-group");
        var dataHtml = $(this).html();
        var that = $(this);
        addGroupAccordion(that, dataSort, dataPlace, dataHtml);
        var groupHolder = dataPlace;
        countChildren(groupHolder);
    });

<div class="panel panel-default">
  <div class="panel-heading main">
    <h4 class="panel-title">
      <a class="accordion-toggle collapsed main side-js" data-parent="#main-panel" data-panel="00" data-toggle="collapse" href="#panel-00">
        <span>0</span>Tellija kulud 
        <b>(<span class="cat-count">0</span>)</b>
        <i class="icon-angle-down"></i>
      </a>
    </h4>
  </div>
  <div id="panel-00" class="panel-collapse collapse">
    <div class="panel-body">

      <div class="panel-group" data-group-holder="00">

      </div>

    </div>
  </div>
</div>

<h3>Main groups</h3>
  <div class="row draggable-groups" data-group="00">
    <div class="col-md-6">
      <div class="main-groups" data-sort="01">01</div>
    </div>
    <div class="col-md-6">
      <div class="main-groups" data-sort="02">02</div>
    </div>
    <div class="col-md-6">
      <div class="main-groups" data-sort="03">03</div>
    </div>
    <div class="col-md-6">
      <div class="main-groups" data-sort="04">04</div>
    </div>
    <div class="col-md-6">
      <div class="main-groups" data-sort="05">05</div>
    </div>
    <div class="col-md-6">
      <div class="main-groups" data-sort="06">06</div>
    </div>
    <div class="col-md-6">
      <div class="main-groups" data-sort="07">07</div>
    </div>
    <div class="col-md-6">
      <div class="main-groups" data-sort="08">08</div>
    </div>
  </div>

function addGroupAccordion(self, dataSort, dataPlace, dataHtml) {
    ...
    var removeClone = $(self).parents('div:eq(0)').remove();
    ...
}

$(document).on('click','.draggable-groups .main-groups',function(){
    ...
    var self = this; // 'self' is more standardized
    addGroupAccordion(self, dataSort, dataPlace, dataHtml);
    ...
});