還能被英語語法難住，切

Would be a good idea to extract (String) mVelContext.get("ord_state") in a variable and then make the invokeOrderExtension call on line 365. 

In fact, this is already done on line 189. It would have to be moved out of the if block.

Would have been a good idea to group these initializations in one place to avoid confusion. The servlet as it is looks quite messy. Also makes it consistent

Order.invokeOrderExtension((String)mVelContext.get("ord_id"), ctl_nbr, (String) mVelContext.get("ord_state"), new_ord_state, true, env);

Two of these arguments are variables, rest are returns from function calls. Minor issue, but helps in the long run in terms of readability.

分析下里面的語法，以後每次遇到時可以都心知肚明