Syntax error

I am always forgetting to add put a fi to close a if