Firstly evals have to evaluate true or false - what you have here appears to return a string - presumably you are actually comparing this to some value? One, not particularly elegant or efficient method is to do the check within your eval, eg: eval(ClaimsHistoryHelper.getClaimDrug(drugRefNo) == null ? false : drugName == (ClaimsHistoryHelper.getClaimDrug(drugRefNo)).getDrugName()) If your ClaimsHistoryHelper is doing a time consuming operation (eg going to a db etc), you can probably make a very quick and simple efficiency saving by having it cache the last result it searched for/returned (or depending on how many drugRefNo you are looking up a more complete caching solution). Thomas ************************************************************************************** This message is confidential and intended only for the addressee. If you have received this message in error, please immediately notify the postmaster(a)nds.com and delete it from your system as well as any copies. The content of e-mails as well as traffic data may be monitored by NDS for employment and security purposes. To protect the environment please do not print this e-mail unless necessary. NDS Limited. Registered Office: One London Road, Staines, Middlesex, TW18 4EX, United Kingdom. A company registered in England and Wales. Registered no. 3080780. VAT no. GB 603 8808 40-00 **************************************************************************************