What's wrong with this code?

[s_allamneni]

strHTML = strHTML & "<form onSubmit="&q& "return false;" & q&">"
strHTML = strHTML & "<input type=" &q& "button" &q& " value=" &q& "First Page" &q& " onClick=" &q& "self.location.href =" &q& "'products.asp?id=" & catid &q& "Page=1'" &q& "return true;" &q& ">"
strHTML = strHTML & "<input type=" &q& "button" &q& " value=" &q& "Prev Page" &q& " onClick=" &q& "self.location.href =" &q& "'products.asp?id=" & catid &q& "Page=" & (curPage-1) &q& "return true;" &q& ">"
strHTML = strHTML & "<input type=" &q& "button" &q& " value=" &q& "Next Page" &q& " onClick=" &q& "self.location.href =" &q& "'products.asp?id=" & catid &q& "Page=" & (curPage+1) &q& "return true;" &q& ">"
strHTML = strHTML & "<input type=" &q& "button" &q& " value=" &q& "Last Page" &q& " onClick=" &q& "self.location.href =" &q& "'products.asp?id=" & catid &q& "Page=" & numPages & "'" &q& "return true;" &q& ">"
strHTML = strHTML & "</form>"

When i click on the buttons(first,last,previous,next) nothing is happening....i want it to go the products.asp page with the catid(category id ) and page number as the parameters.
thanks for your help

[s_allamneni]

Thank you Dave,
I need to make this html string to sent it to another function. which inturn sent it to the browser. But the code is working fine now.
I'm kinda learning and developing this web site. I highly appreciate your help.