Is it bad practice to assume the proper (left to right) execution of boolean expressions for simplicity?
Reliant code:
Code:
if(
!Is_Next_Match(Parser, TOKEN_WEIGHT) ||
!Next_Number(Parser, j, INT) ||
j != i ||
!Next_Number(Parser, Character->Meshes[Mesh_Index]->Weights[i]->Joint ||
!Next_Number(Parser, Character->Meshes[Mesh_Index]->Weights[i]->Bias ||
!Next_Number(Parser, Character->Meshes[Mesh_Index]->Weights[i]->Position[_X_]) ||
!Next_Number(Parser, Character->Meshes[Mesh_Index]->Weights[i]->Position[_Y_]) ||
!Next_Number(Parser, Character->Meshes[Mesh_Index]->Weights[i]->Position[_Z_])
)
goto Error_Currupt_File;
Non-reliant code:
Code:
if(!Is_Next_Match(Parser, TOKEN_WEIGHT))
goto Error_Currupt_File;
if(!Next_Number(Parser, j, INT))
goto Error_Currupt_File;
....