0

当通过 javascript 添加 td 元素时，我有一个运行良好的 onclick javascript 函数。删除按钮工作正常。但是当我用 php 创建元素并单击删除时，我得到：

SyntaxError: syntax error
var header = document.getElementById(

即使计数器工作正常。这是范围问题吗？这是我的代码：

JS

var labor_count = 0;
var part_count = 0;

function incrementLabor(count) {
    labor_count = parseInt(count);
    console.log(labor_count);
}

HTML/PHP

public function getTableRow()
{
    return '
<tr>
<td>
    <input
        type="text"
        name="work_description[]"
        value="'.$this->description.'">
</td>
<td>
    <input
        class="hours calculate"
        type="text"
        name="hours[]"
        value="'.$this->hours.'"
        onchange="calculate();"
        size="2">
</td>
<td>
    <input
        class="rate calculate"
        type="text"
        name="rate[]"
        value="'.$this->rate.'"
        onchange="calculate();"
        size="2">
</td>
<td>
    <input
        type="button"
        value="Remove"
        onclick="
            (function () {
                var parent = this.parentNode.parentNode; //get the row node
                table.deleteRow(parent.rowIndex); //Delete the row index behind it.
                labor_count--;
                console.log(labor_count);
                if (labor_count === 0) {
                    var header = document.getElementById("labor_header");
                    header.parentNode.removeChild(header);
                })()
            };" >
    </td>
</tr>

';
}

这是工作功能：

function removeIt(element) {
        var parent = element.parentNode.parentNode; //get the row node
        var table = parent.parentNode;
        table.deleteRow(parent.rowIndex); //Delete the row index behind it.
        labor_count--;
        console.log(labor_count);
        if (labor_count === 0) {
            var header = document.getElementById("labor_header");
            header.parentNode.removeChild(header);
        }
}
4

2 回答 2

3

getElementById( 之后的引号是关闭 onclick=" 引号。我建议命名你的函数，然后在 onclick 中调用它

<script>
function removeIt(element) {
    var parent = element.parentNode.parentNode; //get the row node
    table.deleteRow(parent.rowIndex); //Delete the row index behind it.
    labor_count--;
    console.log(labor_count);
    if (labor_count === 0) {
        var header = document.getElementById("labor_header");
        header.parentNode.removeChild(header);
    }
} 
</script>

<input
type="button"
value="Remove"
onclick="removeIt(this)"
>

此外，在 php 中，您应该将 htmlspecialchars() 放在 $this->description、$this->hours 和 $this->rate 周围，以防它们包含引号或其他特殊字符。

于 2013-08-20T23:06:13.717 回答
1

您的 javascript 格式不正确。应该

(function () {
    var parent = this.parentNode.parentNode; //get the row node
    table.deleteRow(parent.rowIndex); //Delete the row index behind it.
    labor_count--;
    console.log(labor_count);
    if (labor_count === 0) {
        var header = document.getElementById("labor_header");
        header.parentNode.removeChild(header);
    }
})();

除此之外，我真的建议不要使用 PHP 生成 javascript。将 javascript 留在 html 文件中并只生成一些 html 元素会好得多。

另外，我建议不要使用内联事件处理程序。相反，在页面加载时使用 javascript 将事件处理程序附加到 html 元素。

这是一个可以帮助您的链接。

于 2013-08-20T23:01:09.390 回答