One could argue does it make sense to use the parameters if an exception is raised? In the end, the method was not able to fulfill its purpose (you raised an exception). So does it make sense to process the parameters? In my opinion no.
I try too to...
This can also be simplified to
date_ext = |{ sy-datum DATE = ENVIRONMENT }|.
Big advantage with this expression is you dont need an intermediate variable if you want to pass a formatted date into a method!