0

我编写了这些函数来在单击表格的特定单元格时处理数据。我有 6 个函数来处理事件的数据，但每个函数都是相同的，它们只是具有不同的 ID。这是我的代码：

    //COMMENT HANDLING
    $("#mondayCommentLink").click(function () {
        var mondayhtmls = $("#mondayComment");
        var input = $("<input type='text' id='mondayCommentText' name='mondayCommentText' size='10' />");
        input.val(data.days[0].comment);
        mondayhtmls.html(input);
    });

    $("#tuesdayCommentLink").click(function () {
        var tuesdayhtmls = $("#tuesdayComment");
        var inputt = $("<input type='text' id='tuesdayCommentText' name='tuesdayCommentText' size='10' />");
        inputt.val(data.days[1].comment);
        tuesdayhtmls.html(inputt);
    });

    $("#wednesdayCommentLink").click(function () {
        var htmls = $("#wednesdayComment");
        var input = $("<input type='text' id='wednesdayCommentText' name='wednesdayCommentText' size='10' />");
        input.val(data.days[2].comment);
        htmls.html(input);
    });

    $("#thursdayCommentLink").click(function () {
        var htmls = $("#thursdayComment");
        var input = $("<input type='text' id='thursdayCommentText' name='thursdayCommentText' size='10' />");
        input.val(data.days[3].comment);
        htmls.html(input);
    });

    $("#fridayCommentLink").click(function () {
        var htmls = $("#fridayComment");
        var input = $("<input type='text' id='fridayCommentText' name='fridayCommentText' size='10' />");
        input.val(data.days[4].comment);
        htmls.html(input);
    });

    $("#saturdayCommentLink").click(function () {
        var htmls = $("#saturdayComment");
        var input = $("<input type='text' id='saturdayCommentText' name='saturdayCommentText' size='10' />");
        input.val(data.days[5].comment);
        htmls.html(input);
    });

有没有一种方法可以简化这段代码，或者让它比让 6 个单独的函数处理每个特定的单元格事件更优雅？

4

5 回答 5

2

如果每个链接都有一个公共类（或者您可以给它们一个），那么您可以将其用作选择器。此外，data-day="1"在标签中添加类似的内容，数字对应于位置（0 代表星期一，1 代表星期二......）。

然后，使用以下命令开始您的功能：

var day = this.id.match(/(\w+)CommentLink/)[1],
    comment = data.dats[this.getAttribute("data-day")].comment;

然后你可以做类似的事情$("#"+day+"Comment")，id='"+day+"CommentText'等等。

于 2013-06-06T16:12:52.240 回答
2

将属性 data-dow="2" 和 data-day="wednesday" 添加到每个评论链接，删除 id="wednesdayCommentLink" 并添加 class="commentLink"。

<a class="wednesdayCommentLink">link</a>

变成

<a class="commentLink" data-dow="2" data-day="wednesday">link</a>

对#wednesdayComment (.comment) 和#wednesdayCommentText (.commentText) 执行相同的操作，为每个添加 data- 属性。

然后，使用这个 JavaScript：

$(".commentLink").click(function () {
    var dow = $(this).attr('data-dow');
    var day = $(this).attr('data-day');

    var input = $('<input type="text" class="commentText" data-dow="'+dow+'" data-day="'+day+'" name="'+day+'CommentText" size="10" />');
    input.val(data.days[dow].comment);

    $('.comment[data-dow="'+dow+'"]').html(input);
});
于 2013-06-06T16:18:28.187 回答
1

我敢肯定有很多事情可以做。但是在重构时，我建议从小处着手。所以首先只需将事件处理程序的公共代码移动到一个单独的函数中：

var handler = function(day,commentId,commentTextId) {
  return function () {
    var input = $("<input type='text' id='"+commentTextId+"' name='"+commentTextId+"' size='10' />");
    input.val(day.comment);
    $('#'+commentId).html(input);
  };
};

然后你可以像这样设置你的事件：

$("#mondayCommentLink").click(handler(data.days[0],"mondayComment","mondayCommentText"));
$("#tuesdayCommentLink").click(handler(data.days[1],"tuesdayComment","tuesdayCommentText"));
// and so on
于 2013-06-06T16:22:52.170 回答
0 
var dayNums = {"monday": 0, "tuesday": 1, "wednesday": 2, "thursday": 3, "friday": 4, "saturday": 5};
$("#mondayCommentLink, #tuesdayCommentLink, #wednesdayCommentLink, #thursdayCommentLink, #fridayCommentLink, #saturdayCommentLink").click(function () {
    var day = $(this).attr("id").replace(/CommentLink$/, "");
    var htmls = $("#" + day + "Comment");
    var input = $("<input type='text' id='" + day + "CommentText' name='" + day + "CommentText' size='10' />");
    input.val(data.days[dayNums[day]].comment);
    htmls.html(input);
});
于 2013-06-06T16:15:35.027 回答
0

我要做的是为commentLinks创建一个类，这样您就只能为每个元素编写一次单击事件，并且我不会删除id来执行以下操作：

jQuery('.commentLink').click(function(){
  var elementId = jQuery(this).attr('id');
  var sectionName = elementId.substring(0, elementId.lastIndexOf('Link'));
  var htmls = jQuery('#' + sectionName);
  var input = jQuery("<input type='text' id='" + sectionName + "Text' name='" + sectionName + "Text' size='10' />");
  htmls.html(input);
});

就这样。

于 2013-06-06T16:32:59.290 回答