fix Fix NullPtr Safebox

[100%]

Original Code: In the original code, it checks whether an active quest is running for the player before performing an operation on the safebox. If the quest is active, the operation is interrupted, preventing the player from performing the operation on the safebox.

Modified Code: In the modified code, the same condition check is performed but with a more readable approach. The player object (pc) is obtained using quest::CQuestManager::Instance().GetPCForce(ch->GetPlayerID()). It then checks if this object exists (pc != nullptr) and if the player has an active quest (pc->IsRunning()). If so, the operation on the safebox is aborted.

This change makes the code more readable and enhances its clarity, making it easier for other developers to understand how the code works. It also allows for a more flexible modification of the code in the future, in case additional changes to the logic of this part of the game are needed.

 

game -> input_main.cpp

Find:

This is the hidden content, please

• Sign In
• or
• Sign Up

Change to:

This is the hidden content, please

• Sign In
• or
• Sign Up