Is this a bad practice to catch a non-specific exception such as System.Exception? Why?

Question

I am currently doing a code review and the following code made me jump. I see multiple issues with this code. Do you agree with me? If so, how do I explain to my colleague t

Answer 1

the princeple is only to catch the exception you can handle. such as, if you know how to deal with findnotfound, you catch the filenotfoundexception, otherwiese, do NOT catch it and let it be thrown to the upper layer.