fix bug: href is not correct
separate Controller from View