Shouldn't you replace this line rather than repeat it with a different value below?