forループ内にアンカータグを書き込もうとしており、onclickイベントで関数にパラメーターを送信しています。
ここで問題となるのは、関数が変数内のパラメーターを渡さず、代わりに「Uncaught ReferenceError：I is notdefined」というエラーが発生することです。ここで、Iは変数catidoneの値です。
catlisttxt += '<li>'+catidone+'<a href="#" onclick="return DisplayQues('+ catidone +')">'+data.categories[i].maincatname+'</a></li>';