Problema con IF: Campos vacios y null

[luna690]

¿Que tengo mal?

Código:

if ( (
                this.getEan()!=null && !"".equals(this.getEan())) 
                || (this.getISBN()!=null && !"".equals(this.getISBN()) 
              ) 
                && brand==true 
                && this.getId() != null
                && this.getName(locale) != null
                && this.getProduct().getSummary().get(locale) != null
                && this.getCondition() != null
                && this.getConvenientCurrentPrice() != null
                && this.getConvenientFirstDocument() != null
                && this.getConvenientFirstDocument().getBinaryId() != null
                && this.getConvenientFirstDocument().getFilename() != null
                && this.rest(locale) != null
            ) { return true; }
        return false;

Necesito que los campos obligatorios: brand, id, nombre, descripcion del producto, condicion, precio, first-document no sean campos vacios.
Y, necesito que haya uno de los dos campos (ean/isbn) llenos.

Que tengo mal?

[luis_vega]

En las condiciones de nombre, condicion, precio, etc... estas comprobando que no son nulos, aun podrian ser vacios. No se si eso es lo que quieres comprobar.

Ademas tendrias que añadir aislar las comprobaciones sobre ean e ISBN con un parenteis porque el operador && tiene mayor precedencia que el ||.
Tal como esta ahora, si Ean contiene algo, el resultado de todo el if es true independientemente del resto de campos porque la condicion a evaluar sera tipo "(True or ....)" y eso siempre da true

Código:

if ( 
     ((this.getEan()!=null && !"".equals(this.getEan())) 
     || 
     (this.getISBN()!=null && !"".equals(this.getISBN())))
                && brand==true 
                && this.getId() != null
                && this.getName(locale) != null
                && this.getProduct().getSummary().get(locale) != null
                && this.getCondition() != null
                && this.getConvenientCurrentPrice() != null
                && this.getConvenientFirstDocument() != null
                && this.getConvenientFirstDocument().getBinaryId() != null
                && this.getConvenientFirstDocument().getFilename() != null
                && this.rest(locale) != null
            ) { return true; }
        return false;

[luna690]

Muchas gracias Luis,
en el caso que cambiara todos los NULL por !"".equals(this.getId()) ¿estaría comprobando tanto si es NULL como si es campo vacío?

[luis_vega]

Entiendo que me estas preguntando si condiciones tipo (!"".equals(cadena)) son true tanto si cadena no es nulo como si no es vacia.
No. Me temo que no. De hecho para (cadena==null) devolveria true, y para cadena con contenido tambien devolveria true.