0

我有一个javascript函数。第三个条件不起作用，当它单独在循环中时它起作用，但在其他条件下不起作用，我不知道为什么。有人可以帮忙吗？？

function verif(ref1, ref2, ref3, ref4, ref5, ref6) {
    document.forms["form"].intitule.value = ref1;
    document.forms["form"].montant.value = ref2;
    document.getElementById(ref3).selected = true;

    for (var i = 1; i <= 400; i++) {
        if (document.getElementById("newDay").options[i].text == ref4) {
            document.getElementById("newDay").options[i].selected = true;
        }
        if (document.getElementById("newMonth").options[i].text == ref5) {
            document.getElementById("newMonth").options[i].selected = true;
        }
        if (document.getElementById("newYear").options[i].text == ref6) {
            document.getElementById("newYear").options[i].selected = true;
        }
    }
}
4

2 回答 2

0

将每个条件添加到它自己的 try catch 中，看看是否有任何条件抛出错误。我添加了警报以帮助您调试。

function verif(ref1, ref2, ref3, ref4, ref5, ref6) {
    document.forms["form"].intitule.value = ref1;
    document.forms["form"].montant.value = ref2;
    document.getElementById(ref3).selected = true;

    for (var i = 1; i <= 400; i++) {
        try{
        if (document.getElementById("newDay").options[i].text == ref4) {
            document.getElementById("newDay").options[i].selected = true;
        }
        }catch(err){
          alert(err);
        }
        try{
        if (document.getElementById("newMonth").options[i].text == ref5) {
            document.getElementById("newMonth").options[i].selected = true;
        }
        }catch(err){
          alert(err);
        }
        try
        {
        if (document.getElementById("newYear").options[i].text == ref6) {
            document.getElementById("newYear").options[i].selected = true;
        }
        }catch(err){
          alert(err);
        }
    }
}
于 2013-08-15T10:25:28.900 回答
0

看起来您正在尝试为所有3项目执行一个循环。这样做的好处是所需的迭代次数较少，但是您没有包含任何代码来确保您不会尝试查找undefined <option> s。接下来，考虑一下使用一个循环的好处是如何真正节省迭代次数，因此有两个循环，第二个在第一个结束的地方拾取，仍然具有最小的迭代次数。

因此，如果您想节省循环迭代次数，您可以尝试这样的事情，其中​​循环中的项目会根据距离i和已经找到的内容而减少。不过，这确实会导致一些代码重复。请注意循环是如何按月、日、年的顺序排序的，因为12 < 31 < x（我假设x是 about ），因此确保您不会尝试查找，例如第20 个月，哪个当然不会存在。400

function verif(ref1, ref2, ref3, ref4, ref5, ref6) {
    document.forms["form"].intitule.value = ref1;
    document.forms["form"].montant.value = ref2;
    document.getElementById(ref3).selected = true;

    var i = 0, found = {day: 0, month: 0, year: 0},
        elm = {
            days: document.getElementById("newDay"),
            months: document.getElementById("newMonth"),
            years: document.getElementById("newYear")
        };
    // months length = min(days, months, years)
    while (++i < elm.months.length && !found.months) {
        if (elm.days.options[i].text == ref4) {
            found.days = elm.days.options[i].selected = true;
        }
        if (elm.months.options[i].text == ref5) {
            found.months = elm.months.options[i].selected = true;
        }
        if (elm.years.options[i].text == ref6) {
            found.years = elm.years.options[i].selected = true;
        }
    }
    // days length = min(days, years)
    while (++i < elm.days.length && !found.days) {
        if (elm.days.options[i].text == ref4) {
            found.days = elm.days.options[i].selected = true;
        }
        if (elm.years.options[i].text == ref6) {
            found.years = elm.years.options[i].selected = true;
        }
    }
    // only years left
    while (++i < elm.years.length && !found.years) {
        if (elm.years.options[i].text == ref6) {
            found.years = elm.years.options[i].selected = true;
        }
    }
}

您可能还注意到我没有包含任何break语句，这是因为如果我们找到仍将在以后的循环中查找的内容，我们仍然想搜索特定于当前循环的项目，我们不想跳过同一迭代的其他检查，我们也不想在以后的循环上浪费时间。因此，取而代之break的是一段执行类似操作的代码。<condition>

于 2013-08-15T12:16:36.283 回答