Thus line won't work, tell me why? and refactor to make if useful