This is wrong, only execute one time