Logic looks correct here. But think the formatting needs to be fixed up…