А зачем каждый раз перетирать onclick?