How should I cast for Java generic with multiple bounds?
Asked Answered
W

5

21

Is it possible to cast an object in Java to a combined generic type?

I have a method like:

public static <T extends Foo & Bar> void doSomething(T object) {
    //do stuff
}

Calling this method is no problem if I have a class that implements both interfaces (Foo & Bar).

The problem is when I need to call this method the object I need to pass to it is received as java.lang.Object and I need to cast it to make the compiler happy. But I can't figure out how to make this cast.

edit:

The problem lies in a function like this:

public void problemFunction (Object o) {
  if ( o instanceof Foo && o instanceof Bar) {
      doSomething((Problematic cast) o);
  }
}

}

Wasteful answered 25/11, 2008 at 17:29 Comment(0)
C
29

Java 8 introduces the possibility of casting with additional bounds. You can cast an Object as a class with multiple interfaces (or just as multiple interfaces).

So this:

doSomething((Problematic cast) o);

simply becomes to this:

doSomething((Foo & Bar) o);
Carolyncarolyne answered 9/10, 2015 at 23:59 Comment(0)
S
13

Unfortunately, there is no legal cast that you can make to satisfy this situation. There must be a single type known to implement all of the interfaces that you need as bounds, so that you can cast to it. The might be a type you create for the purpose, or some existing type.

interface Baz extends Foo, Bar { }

public void caller(Object w) {
  doSomething((Baz) w);
}

If other types are known, like Baz, to meet the bounds, you could test for those types, and have a branch in your caller that calls doSomething with a cast to those types. It's not pretty.

You could also use delegation, creating your own class Baz that meets the bounds required by doSomething. Then wrap the object you are passed in an instance of your Baz class, and pass that wrapper to doSomething.

private static class FooBarAdapter implements Foo, Bar {
  private final Object adaptee;
  FooBarAdapter(Object o) {
    adaptee = (Foo) (Bar) o;
  }
  public int flip() { return ((Foo) adaptee).flip(); }
  public void flop(int x) { ((Foo) adaptee).flop(x); }
  public void blort() { ((Bar) adaptee).blort(); }
}

public void problemFunction (Object o) {
  doSomething(new FooBarAdapter(o));
}
Shipworm answered 25/11, 2008 at 18:6 Comment(3)
Good idea. I used it in a slightly different way. I prefer the FooBarAdapter to get to objects with the 2 different types. The == can be used to check if it's the same object (no a must). This way, their is a compile time type check.Diabolize
I combined this delegated pattern anonymous/named inner method classes to "cast" an object I know to extend/implement a set of classes and interfaces to meant the bound requirements. See Jon's answer: https://mcmap.net/q/659006/-anonymous-extending-a-class-and-implementing-an-interface-at-the-same-timeGabe
And this seems to be the limitation of the Java language because in Scala running on the same JVM, this is easy as o.asInstanceOf[Foo with Bar]Oblate
A
7
public static <T extends Foo & Bar> void doSomething(T object)

This seems to denote that you would be performing more than one operation on the object in question.

I would argue that if the desired operations you are performing on the object are distinct enough to be seperated across interfaces, then they are distinct enough to deserve their own methods.

It is likely that you could restructure this code to call seperate methods to perform the desired operation. This may end up making the entire operation more clear from the client's perspective.

Instead of:

public void problemFunction (Object o) {
  if (o instanceof Foo && o instanceof Bar) {
      doSomething((Problematic cast) o);
  }
}

It becomes:

public void problemFunction(Object o) {
  if (o instanceof Foo && o instanceof Bar) {
      fooifySomething((Foo) o);
      baratizeSomething((Bar) o);
  }
}
Anticipant answered 25/11, 2008 at 20:27 Comment(1)
LMAO fooify, that's new to me xDRencontre
C
6

It is possible to "up-cast" to a union type, by the horrible means of encoding the union type into a method's type parameter; for your example you can write

private <Q extends Foo & Bar> Q upcast(final Object in) {
    return (Q) in;
}

// ... elsewhere...

if (myObject instanceof Foo && myObject instanceof Bar) {
    doSomething(upcast(myObject));
}
Charade answered 2/12, 2014 at 16:25 Comment(1)
Thanks. Only annoyance is it generates an unchecked cast. However I used a modified version which takes Foo rather than Object and checks it implements Bar and then returns with the cast as you have done otherwise throwing a ClassCastException.Vichy
C
-1

As a workaround, you could define another interface FooBar that extends Foo and Bar and have your class implement that. It doesn't even have to be a top-level interface - you can declare it private if you don't want to clutter up or retrofit the rest of your code:

private interface FooBar extends Foo, Bar {}
public void problemFunction (Object o) {
  if ( o instanceof Foo && o instanceof Bar) {
      doSomething((FooBar) o);
  }
}
Cherice answered 25/11, 2008 at 18:18 Comment(2)
I believe this won't work since there is no guarantee the o actually has "implements FooBar". I don't think Java will just assume it.Adelaideadelaja
@ArtB The answer says "and have your class implement that.".Reproduction

© 2022 - 2024 — McMap. All rights reserved.