for - object java example

La mejor manera de "negar" una instancia de

Question

Estaba pensando si existe una forma mejor / mejor de negar una instanceof de Java. En realidad, estoy haciendo algo como:

if(!(str instanceof String)) { /* do Something */ }

Pero creo que debería existir una sintaxis "hermosa" para hacer esto.

¿Alguien sabe si existe y cómo se ve la sintaxis?

EDIT: por hermosa, podría decir algo como esto:

if(str !instanceof String) { /* do Something */ } // compilation fails

Answer 1

No sé lo que imaginas cuando dices "hermoso", pero ¿qué pasa con esto? Personalmente creo que es peor que la forma clásica que publicaste, pero a alguien le puede gustar ...

if (str instanceof String == false) { /* ... */ }

Answer 2

No, no hay mejor manera; el tuyo es canonico

Answer 3

Podrías usar el método Class.isInstance :

if(!String.class.isInstance(str)) { /* do Something */ }

... pero sigue siendo negado y bastante feo.

Answer 4

Por lo general, usted no quiere solo una cláusula if sino también else .

if(!(str instanceof String)) { /* do Something */ }
else { /* do something else */ }

Se puede escribir como

if(str instanceof String) { /* do Something else */ }
else { /* do something */ }

O puede escribir el código para que no necesite saber si es una cadena o no. p.ej

if(!(str instanceof String)) { str = str.toString(); }

Se puede escribir como

str = str.toString();

Answer 5

Si lo encuentra más comprensible, puede hacer algo como esto con Java 8:

Predicate<Object> isInstanceOfTheClass =
objectToTest -> objectToTest instanceof TheClass;
Predicate<Object> isNotInstanceOfTheClass =
isInstanceOfTheClass.negate(); // or objectToTest -> !(objectToTest instanceof TheClass)
if (notInstanceOfTheClass.test(myObject)) {
// do something
}

Answer 6

Si puedes usar importaciones estáticas, y tu código moral lo permite

public class ObjectUtils {
private final Object obj;
private ObjectUtils(Object obj) {
this.obj = obj;
}
public static ObjectUtils thisObj(Object obj){
return new ObjectUtils(obj);
}
public boolean isNotA(Class<?> clazz){
return !clazz.isInstance(obj);
}
}

Y entonces...

import static notinstanceof.ObjectUtils.*;
public class Main {
public static void main(String[] args) {
String a = "";
if (thisObj(a).isNotA(String.class)) {
System.out.println("It is not a String");
}
if (thisObj(a).isNotA(Integer.class)) {
System.out.println("It is not an Integer");
}
}
}

Este es solo un ejercicio de interfaz fluida, ¡nunca lo usaría en el código de la vida real!
¡Sigue tu estilo clásico, no confundirá a nadie más leyendo tu código!

Answer 7

ok solo mis dos centavos, use un método de cadena:

public static boolean isString(Object thing) {
return thing instanceof String;
}
public void someMethod(Object thing){
if (!isString(thing)) {
return null;
}
log.debug("my thing is valid");
}