if(var i("counter").value..
Это на каком языке? Сравнивать переменную, которую только что создал - странное дело.. да и блок по идентификатору получать надо через
document.getElementById('counter')
В итоге:
function prev(id){
var counter = document.getElementById('counter'+(id===undefined?'':id)).value;
document.getElementById('counter'+(id===undefined?'':id)).value = counter.value>1?counter--:counter;
}
function next(id){
var counter = document.getElementById('counter'+(id===undefined?'':id)).value;
document.getElementById('counter'+(id===undefined?'':id)).value = counter.value<20?counter++:counter;
}