no a co radeji:

public boolean isExpire() {
   return (this.cached) && (cacheExpire.compareTo(new Date()) < 0);
}

to vam asi pmd snazi vnutit ...

Petr Burdik napsal(a):
> 
> Ahojte,
> zacal jsem pouzivat pmd a v nicem neni problem, jenom v
> CollapsibleIfStatements. Me totiz prijde ze na tom neni nic spatneho
> napsat takovouto podminku. Tedy pokud je to v jedne podmince samozrejme
> a ne ve dvou jak demonstruje priklad. Mam tri takove podminky v celem
> projektu.
> 
> u me:
> 
>     public boolean isExpire() {
>         boolean result = false;
> 
>         if( (this.cached) && (cacheExpire.compareTo(new Date()) < 0) ) {
> result = true; }
> 
>         return result;
>     }
> 
> 
> Metoda vrati true, jestlize je objekt kesovatelny ( fuj ) a jestlize je
> datum expirace mensi nez aktualni datum.
> 
> 
> Odkaz na popis rule:
> http://pmd.sourceforge.net/rules/basic.html#CollapsibleIfStatements
> 
> Pet
> 
> 
> --Using Opera's revolutionary e-mail client: http://www.opera.com/mail/

-- 
Jiří Mareš (mailto:[EMAIL PROTECTED])
ČSAD SVT Praha, s.r.o. (http://www.svt.cz)
Czech Republic

Odpovedet emailem