Jejda,
diky. Ja jsem nikdy neprosel cckem a toto moc nepouzivam. diky moc :)

Pet

On Tue, 03 Apr 2007 13:08:17 +0200, Jiri Mares <[EMAIL PROTECTED]> wrote:


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/




--
Using Opera's revolutionary e-mail client: http://www.opera.com/mail/

Odpovedet emailem