Clean up is_closed and use H5I_INVALID_HID

https://github.com/QMCPACK/qmcpack/pull/5526#discussion_r2096554038
Some thoughts. It can be better if we write a check_valid function and hide include and H5I_INVALID_HID on the cpp side.

Start with the discussion in pull request #5526 and locate the current handling of is_closed, H5I_INVALID_HID, and the related include. Determine whether a check_valid function can replace the existing checks while keeping the include and constant on the C++ side; done means the cleanup is implemented and existing behavior remains valid.

Written by the indexing model from the issue text.