fix: the bug of possible large movement of atom position caused by periodicity in BFGS method

[pxlxingliang]

Reminder

• Have you linked an issue with this pull request?
• Have you noticed possible changes of behavior below or in the linked issue?
• Have you explained the changes of codes in core modules of ESolver, HSolver, ElecState, Hamilt, Operator or Psi? (ignore if not applicable)

Linked Issue

Fix #3600

What's changed?

• Example: My changes might affect the performance of the application under certain conditions, and I have tested the impact on various scenarios...

Any changes of core modules? (ignore if not applicable)

• Example: I have added a new virtual function in the esolver base class in order to ...

[kirk0830]

Cheers,
but the geometry optimizer looks not quite nice cause there is obviously an inheritance relation between base class BFGS_Basic and the Ions_Move_BFGS but I doubt a little about if this design is really meaningful.

[pxlxingliang]

Cheers, but the geometry optimizer looks not quite nice cause there is obviously an inheritance relation between base class BFGS_Basic and the Ions_Move_BFGS but I doubt a little about if this design is really meaningful.

This also confuse me. These codes need be refactored to make the structure clearer.

[dyzheng]

LGTM