我创建了一个PHP脚本,该脚本通过AJAX从XML文档构建表.例如:
Everyday ItalianGiada De Laurentiis
2005
30.00
Harry PotterJ K. Rowling
2005
29.99
将创建一个包含标题,作者,年份和价格列以及附加删除列的表格.在解析XML时,在上述情况下,我已将tr id设置为当前XML元素(0和1)的tr id.
if (isset($_POST['rowNumber'])) {
$rowNumber = $_POST['rowNumber'];
$file = $_POST['file'];
$dom = new DOMDocument();
$dom->load("../XML/".$file);
$xml = $dom->documentElement;
//PROBLEM HERE
$xml->childNodes->item(0)->parentNode->removeChild($xml->childNodes->item($rowNumber));
$handle = fopen("../XML/".$file, 'w');
fwrite($handle, $dom->saveXML());
}
我在页面加载时构建表,然后在每次删除时构建表.问题是不正确的行被删除,我不知道为什么.
其他测试…
删除点击次数的50%的第一个节点:
$xml->childNodes->item(0)->parentNode->removeChild($xml->childNodes->item(0));
始终删除第一个节点:
$xml->childNodes->item(0)->parentNode->removeChild($xml->childNodes->item(1));
删除点击次数的50%的第二个节点:
$xml->childNodes->item(0)->parentNode->removeChild($xml->childNodes->item(2));
始终删除第二个节点:
$xml->childNodes->item(0)->parentNode->removeChild($xml->childNodes->item(3));
删除点击次数的50%的第三个节点:
$xml->childNodes->item(0)->parentNode->removeChild($xml->childNodes->item(4));
始终删除第三个节点:
$xml->childNodes->item(0)->parentNode->removeChild($xml->childNodes->item(5));
添加了我的AJAX代码
$('#generatedTable a.delete').live('click', function (e) {
e.preventDefault();
//TABLE ROW ID TO BE DELETED. CAN ALERT THIS FINE.
var trID = $(this).closest('tr').attr('id');
$.ajax({
url: "functions/xmlDelete.php",
type: "POST",
dataType: "json",
data: "rowNumber="+ trID + "&fileName=" + fileName,
success: function(data) {
$.ajax({
url: "functions/xmlParser.php",
type: "POST",
dataType: "json",
data: "fileName="+ fileName,
success: function(data) {
$('#xmlTable').html(data.table);
$('#xmlTable').fadeIn('fast');
oTable = $('#generatedTable').dataTable({
"bJQueryUI": true,
"bPaginate": false,
"bLengthChange": false,
"bFilter": false,
"bSort": false,
"bInfo": true
});
}
});
}
});
} );
这是我的表行的样子,我可以毫无问题地警告trID.
1999 Grammy NomineesManyUSAGrammy10.201999 Edit Delete谁能帮忙解释一下我在这里看到的内容.谢谢!
解决方法:
您没有考虑到childNodes()包括所有节点,而不仅仅是元素.
对于您提供的xml文档,$dom-> documentElement-> childNodes-> item(0)是< bookstore>末尾之间的空白.和< book>的开头,而不是第一个< book>节点.
现在您知道了为什么DOM如此令人讨厌.
我建议您使用DOMXPath或SimpleXML而不是遍历childNodes来收集Element索引.
DOMXPath解决方案
if (isset($_POST['rowNumber'], $_POST['file']) and ctype_digit($_POST['rowNumber'])) {
$rowNumber = $_POST['rowNumber'];
$file = '../XML/'.$_POST['file'];
$dom = new DOMDocument();
$dom->load($file);
$root = $dom->documentElement;
$xp = new DOMXPath($dom);
$books = $xp->query('*', $root);
if ($books->item($rowNumber)) {
$root->removeChild($books->item($rowNumber));
// Note that "$root->childNodes->item(0)->parentNode" is completely unnecessary.
// You already have the parent node ($root), so just use it directly!
} else {
echo "Row does not exist";
}
$dom->save($file);
}
SimpleXML解决方案
if (isset($_POST['rowNumber'], $_POST['file']) and ctype_digit($_POST['rowNumber'])) {
$rowNumber = (int) $_POST['rowNumber']; // casting to int is necessary!!
$file = '../XML/'.$_POST['file'];
$sxe = simplexml_load_file($file);
unset($sxe->book[$rowNumber]);
// or, if you don't want to make element name assumptions:
// $children = $sxe->children();
// unset($children[$rowNumber]);
$sxe->asXML($file);
}
问题
但是,整个方法存在两个严重问题.
>您正在通过文件发布变量接受不受信任的输入.读取和写入驱动器上的任何XML文件都不会有任何用户.您应该具有预定义的有效文件值列表,这些文件值$_POST [‘file’]必须匹配,然后才能处理请求.
>您不考虑并发性.这以两种方式体现出来:
>如果两个用户同时进行编辑,而一个用户删除了书0,那么当第二个用户删除书0时,他将删除的书本与他打算的不同,即在他看来是书1!解决方案是,每本书都需要一个明确的,唯一的标识符,您可以在XML文件和html界面中使用该标识符.您不能使用索引编号来标识节点.
>您对XML文件的读写不是原子的,也不使用任何文件锁定.因此,有人可能会在另一个PHP进程正在写入文件的过程中读取XML,从而给读者提供了不完整的XML文件.您必须执行原子写操作(在* nix系统上,先写一个唯一的临时文件,然后再写link()到真实文件名),或者必须使用flock()或其他某种锁定机制打开该文件.
因为管理并发读写(尤其是快速方式)非常困难,所以我建议您放弃这种“将文件作为数据库”方法,而使用真实的数据库.
(顺便说一句,您的HTML无效.id属性在文档中必须是唯一的,但是您有两个id =“ 0”.)
标签:dom,ajax,nodes,xml,php
来源: https://codeday.me/bug/20191101/1985171.html