Anyone good at programming logic?

Question

anonymous · Answer

Need help with optimizing the above pseudocode to make it more efficient. 

start

read Customer_Name
perform Calculate_SafetyLevel()
perform Evaluate_History()
print Customer_Name
print 'Credit History of' Client_Name 'is' History
print 'Safety Level of' Client_Name 'is' Safety_Level

stop

Calculate_SafetyLevel() 

read Customer_Income
if Customer_Income >= $5000

Safety_Level = Excellent

elseif Customer_Income >= $3000

Safety_Level = Good

elseif Customer_Income >= $1500

Safety_Level = Ok

endif

return


Evaluate_History()

read Customer_History

if Customer_History = Clean

History = Good

elseif Customer_History = Pending_Balance

History = OK

elseif Customer_History = Defaulted

History = Risky

endif

return

tyteen4a03 · Answer

The logic looks OK to me, however Calculate_SafetyLevel will not do anything when the customer's Customer_Income is lower than $1500. Safety_Level will remain undefined.