看个jquery写法，有什么问题？

[临摹]

//这是个在树形中根据href查找节点，并替换成新的href的函数
//jdjbNew:新节点id
//jdjbOld：旧节点id
function editJd(jdjbNew,jdjbOld){
//旧节点href
var nodehref_old = "zdList?jdjb=" + jdjbOld;
//新节点href
var nodehref_new = "zdList?jdjb=" + jdjbNew;
//在树形DIV中查找a href=nodehref_old的链接（关键是这句进不去，这样写有问题吗？有别的写法吗？）
//这里虽然用了each，但只会循环一次，因为这个href肯定只会出现一次
$("#treeDemo").find("a[href='" + nodehref_old + "']").each(function(){
//调换成新的链接
$(this).attr("href",nodehref_new);
});
}

[夜雨_Jason]

没有问题的呀

<div id="treeDemo">
	<!-- 第一个a将会连接到zdList?jdjb=id2 -->
	<a href="zdList?jdjb=id1" id="id1">id1</a>
	<a href="zdList?jdjb=id2" id="id2">id2</a>
</div>
<script src="http://code.jquery.com/jquery-1.8.3.min.js"></script>
<script>
function editJd(jdjbNew,jdjbOld){
    //旧节点href
    var nodehref_old = "zdList?jdjb=" + jdjbOld;
    //新节点href
    var nodehref_new = "zdList?jdjb=" + jdjbNew;
    //在树形DIV中查找a href=nodehref_old的链接（关键是这句进不去，这样写有问题吗？有别的写法吗？）
    //这里虽然用了each，但只会循环一次，因为这个href肯定只会出现一次
    $("#treeDemo").find("a[href='" + nodehref_old + "']").each(function(){
        //调换成新的链接
        $(this).attr("href",nodehref_new);
    });
}
editJd("id2","id1");
</script>

[人生难得一只鸡]

写法虽然没有什么问题，但是拿href作为条件来查找元素 本身就不靠谱，特别是相对路径在各大浏览器的解析也是不一致的