198

I want to return two objects from a Java method and was wondering what could be a good way of doing so?

The possible ways I can think of are: return a HashMap (since the two Objects are related) or return an ArrayList of Object objects.

To be more precise, the two objects I want to return are (a) List of objects and (b) comma separated names of the same.

I want to return these two Objects from one method because I dont want to iterate through the list of objects to get the comma separated names (which I can do in the same loop in this method).

Somehow, returning a HashMap does not look a very elegant way of doing so.

James P.
  • 19,313
  • 27
  • 97
  • 155
Jagmal
  • 5,726
  • 9
  • 35
  • 35
  • 1
    Is the List and the CSVs basically diffent views of the same data? It sounds like what you need is an Object where you have a single `List` reference and a sort of lookup table. – James P. Jul 02 '11 at 09:29
  • Related question: https://stackoverflow.com/questions/53453058/returning-multiple-primitive-objects-in-java-unrecommended – John McClane Nov 25 '18 at 19:55

25 Answers25

139

If you want to return two objects you usually want to return a single object that encapsulates the two objects instead.

You could return a List of NamedObject objects like this:

public class NamedObject<T> {
  public final String name;
  public final T object;

  public NamedObject(String name, T object) {
    this.name = name;
    this.object = object;
  }
}

Then you can easily return a List<NamedObject<WhateverTypeYouWant>>.

Also: Why would you want to return a comma-separated list of names instead of a List<String>? Or better yet, return a Map<String,TheObjectType> with the keys being the names and the values the objects (unless your objects have specified order, in which case a NavigableMap might be what you want.

Evan Mulawski
  • 54,662
  • 15
  • 117
  • 144
Joachim Sauer
  • 302,674
  • 57
  • 556
  • 614
  • The reason for returning comma separated list is: If I dont create the list here, I would have to do this in the caller by looping through the objects (CS value is needed). Perhaps, I am pre-optimizing unnecessarily. – Jagmal Jan 19 '09 at 14:08
  • 3
    I've always wondered why Java doesn't have a Pair class for this reason. – David Koelle Jan 19 '09 at 14:08
  • Jagmal: yes, if the only rason for returning the comma-separated list is this optimization, then forget about it. – Joachim Sauer Jan 19 '09 at 14:13
  • This only works well if the items you'd like to return are of the same class, or at least have a close common ancestor. I mean, using Object in place of WhateverTypeYouWant isn't very neat. – David Hanak Jan 19 '09 at 15:59
  • @David: I agree that using Object here isn't very neat, but then again return objects without any common ancestor (except Object of course) isn't very neat as well. I'd even say it's a code smell, if you need that. – Joachim Sauer Jan 19 '09 at 16:16
  • Joachim -- I doubt you'll see this, since the post is so old, but I'm wondering if you could explain why you feel that it is a "code smell" to have return objects of multiple types. What kinds of problems does this cause, or what types of poor design does it indicate? Thanks. – J. Taylor Apr 26 '11 at 15:19
  • @jrtayloriv: the problem with returning objects of different types is that you can't consistently handle the returned objects: **either** the returned type depends on the input provided (in which case it should really be two different methods) or is independent from it (in which case you should return a common base class that has enough features to be useful). – Joachim Sauer Apr 26 '11 at 15:23
  • @DavidKoelle You mean something like a bidimap? – Weckar E. Sep 11 '17 at 09:45
74

If you know you are going to return two objects, you can also use a generic pair:

public class Pair<A,B> {
    public final A a;
    public final B b;

    public Pair(A a, B b) {
        this.a = a;
        this.b = b;
    }
};

Edit A more fully formed implementation of the above:

package util;

public class Pair<A,B> {

    public static <P, Q> Pair<P, Q> makePair(P p, Q q) {
        return new Pair<P, Q>(p, q);
    }

    public final A a;
    public final B b;

    public Pair(A a, B b) {
        this.a = a;
        this.b = b;
    }

    @Override
    public int hashCode() {
        final int prime = 31;
        int result = 1;
        result = prime * result + ((a == null) ? 0 : a.hashCode());
        result = prime * result + ((b == null) ? 0 : b.hashCode());
        return result;
    }

    @Override
    public boolean equals(Object obj) {
        if (this == obj) {
            return true;
        }
        if (obj == null) {
            return false;
        }
        if (getClass() != obj.getClass()) {
            return false;
        }
        @SuppressWarnings("rawtypes")
        Pair other = (Pair) obj;
        if (a == null) {
            if (other.a != null) {
                return false;
            }
        } else if (!a.equals(other.a)) {
            return false;
        }
        if (b == null) {
            if (other.b != null) {
                return false;
            }
        } else if (!b.equals(other.b)) {
            return false;
        }
        return true;
    }

    public boolean isInstance(Class<?> classA, Class<?> classB) {
        return classA.isInstance(a) && classB.isInstance(b);
    }

    @SuppressWarnings("unchecked")
    public static <P, Q> Pair<P, Q> cast(Pair<?, ?> pair, Class<P> pClass, Class<Q> qClass) {

        if (pair.isInstance(pClass, qClass)) {
            return (Pair<P, Q>) pair;
        }

        throw new ClassCastException();

    }

}

Notes, mainly around rustiness with Java & generics:

  • both a and b are immutable.
  • makePair static method helps you with boiler plate typing, which the diamond operator in Java 7 will make less annoying. There's some work to make this really nice re: generics, but it should be ok-ish now. (c.f. PECS)
  • hashcode and equals are generated by eclipse.
  • the compile time casting in the cast method is ok, but doesn't seem quite right.
  • I'm not sure if the wildcards in isInstance are necessary.
  • I've just written this in response to comments, for illustration purposes only.
jamesh
  • 19,863
  • 14
  • 56
  • 96
David Hanak
  • 10,754
  • 3
  • 31
  • 39
  • I usually have this class knocking about in each codebase I work on. I'd also add: a hashCode/equals implementation, and possibly a static isInstance() and cast() method. – jamesh Jan 21 '09 at 13:49
  • Sure, there are many ways to make this class smarter and more convenient to use. The version above includes what's just enough in a one-shot declaration. – David Hanak Jan 21 '09 at 16:05
  • @jamesh: do you mind to write your Pair in full details here? I'd like to know how it looks like after supplying "a hashCode/equals implementation, and possibly a static isInstance() and cast() method." Thank you. – Qiang Li Aug 13 '11 at 00:38
  • @QiangLi - I usually generate the hashcode & equals methods. Instance method isInstance takes two classes and makes sure that the instance's a & b are instances of those classes. Cast takes a Pair,?> and carefully casts it to a Pair. The implementations should be fairly easy (hint: Class.cast(), Class.isInstance()) – jamesh Aug 15 '11 at 22:04
  • @Jamesh, I would really appreciate if you can please spell what you said out here. Maybe posted as another answer? I would of course give you an upvote! I am just not sure the way to do what you said. Thank you! – Qiang Li Aug 15 '11 at 22:58
  • 3
    That's a very nice `Pair` implementation. One minor change I'd make: add a message to the `ClassCastException`. Otherwise debugging becomes a nightmare if this fails for some reason. (and the rawtypes suppress-warnings would be unnecessary if you'd cast to `Pair,?>` (which works, because you only need `Object` methods from `a` and `b`). Do you mind if I tweak the code? – Joachim Sauer Jun 05 '13 at 12:47
  • IMHO, better names for fields `a` and `b` of a `Pair` are `first` and `second`, as that states what is known about those fields. On the other hand, for such a basic class, some people prefer shorter field names - so `a` and `b` are not bad choices. – ToolmakerSteve Jun 13 '15 at 22:54
32

In the event the method you're calling is private, or called from one location, try

return new Object[]{value1, value2};

The caller looks like:

Object[] temp=myMethod(parameters);
Type1 value1=(Type1)temp[0];  //For code clarity: temp[0] is not descriptive
Type2 value2=(Type2)temp[1];

The Pair example by David Hanak has no syntactic benefit, and is limited to two values.

return new Pair<Type1,Type2>(value1, value2);

And the caller looks like:

Pair<Type1, Type2> temp=myMethod(parameters);
Type1 value1=temp.a;  //For code clarity: temp.a is not descriptive
Type2 value2=temp.b;
Mike Samuel
  • 118,113
  • 30
  • 216
  • 245
Kyle Lahnakoski
  • 924
  • 10
  • 16
  • 8
    Pair has class type control benefit – Hlex Aug 05 '14 at 03:39
  • 7
    IMHO, don't go this way - the declaration says too little about the expected return values. AFAIK, it is more widely preferred to create generic classes that specify how many parameters are being returned, and the type of those parameters. `Pair`, `Tuple`, `Tuple`, etc. Then a specific use shows the number and types of parameters `Pair temp = ...` or whatever. – ToolmakerSteve Jun 13 '15 at 23:03
26

You may use any of following ways:

private static final int RETURN_COUNT = 2;
private static final int VALUE_A = 0;
private static final int VALUE_B = 1;
private static final String A = "a";
private static final String B = "b";

1) Using Array

private static String[] methodWithArrayResult() {
    //...
    return new String[]{"valueA", "valueB"};
}

private static void usingArrayResultTest() {
    String[] result = methodWithArrayResult();
    System.out.println();
    System.out.println("A = " + result[VALUE_A]);
    System.out.println("B = " + result[VALUE_B]);
}

2) Using ArrayList

private static List<String> methodWithListResult() {
    //...
    return Arrays.asList("valueA", "valueB");
}

private static void usingListResultTest() {
    List<String> result = methodWithListResult();
    System.out.println();
    System.out.println("A = " + result.get(VALUE_A));
    System.out.println("B = " + result.get(VALUE_B));
}

3) Using HashMap

private static Map<String, String> methodWithMapResult() {
    Map<String, String> result = new HashMap<>(RETURN_COUNT);
    result.put(A, "valueA");
    result.put(B, "valueB");
    //...
    return result;
}

private static void usingMapResultTest() {
    Map<String, String> result = methodWithMapResult();
    System.out.println();
    System.out.println("A = " + result.get(A));
    System.out.println("B = " + result.get(B));
}

4) Using your custom container class

private static class MyContainer<M,N> {
    private final M first;
    private final N second;

    public MyContainer(M first, N second) {
        this.first = first;
        this.second = second;
    }

    public M getFirst() {
        return first;
    }

    public N getSecond() {
        return second;
    }

    // + hashcode, equals, toString if need
}

private static MyContainer<String, String> methodWithContainerResult() {
    //...
    return new MyContainer("valueA", "valueB");
}

private static void usingContainerResultTest() {
    MyContainer<String, String> result = methodWithContainerResult();
    System.out.println();
    System.out.println("A = " + result.getFirst());
    System.out.println("B = " + result.getSecond());
}

5) Using AbstractMap.simpleEntry

private static AbstractMap.SimpleEntry<String, String> methodWithAbstractMapSimpleEntryResult() {
    //...
    return new AbstractMap.SimpleEntry<>("valueA", "valueB");
}

private static void usingAbstractMapSimpleResultTest() {
    AbstractMap.SimpleEntry<String, String> result = methodWithAbstractMapSimpleEntryResult();
    System.out.println();
    System.out.println("A = " + result.getKey());
    System.out.println("B = " + result.getValue());
}

6) Using Pair of Apache Commons

private static Pair<String, String> methodWithPairResult() {
    //...
    return new ImmutablePair<>("valueA", "valueB");
}

private static void usingPairResultTest() {
    Pair<String, String> result = methodWithPairResult();
    System.out.println();
    System.out.println("A = " + result.getKey());
    System.out.println("B = " + result.getValue());
}
Slava Vedenin
  • 58,326
  • 13
  • 40
  • 59
18

I almost always end up defining n-Tuple classes when I code in Java. For instance:

public class Tuple2<T1,T2> {
  private T1 f1;
  private T2 f2;
  public Tuple2(T1 f1, T2 f2) {
    this.f1 = f1; this.f2 = f2;
  }
  public T1 getF1() {return f1;}
  public T2 getF2() {return f2;}
}

I know it's a bit ugly, but it works, and you just have to define your tuple types once. Tuples are something Java really lacks.

EDIT: David Hanak's example is more elegant, as it avoids defining getters and still keeps the object immutable.

Ulrik Rasmussen
  • 1,098
  • 1
  • 8
  • 25
9

Apache Commons has tuple and triple for this:

  • ImmutablePair<L,R> An immutable pair consisting of two Object elements.
  • ImmutableTriple<L,M,R> An immutable triple consisting of three Object elements.
  • MutablePair<L,R> A mutable pair consisting of two Object elements.
  • MutableTriple<L,M,R> A mutable triple consisting of three Object elements.
  • Pair<L,R> A pair consisting of two elements.
  • Triple<L,M,R> A triple consisting of three elements.

Source: https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/tuple/package-summary.html

jacob
  • 2,762
  • 1
  • 20
  • 49
9

Before Java 5, I would kind of agree that the Map solution isn't ideal. It wouldn't give you compile time type checking so can cause issues at runtime. However, with Java 5, we have Generic Types.

So your method could look like this:

public Map<String, MyType> doStuff();

MyType of course being the type of object you are returning.

Basically I think that returning a Map is the right solution in this case because that's exactly what you want to return - a mapping of a string to an object.

kipz
  • 365
  • 2
  • 8
  • This won't work if any of the names collide. A list could contain duplicates, but a Map can't (contain duplicate keys). – tvanfosson Jan 19 '09 at 14:08
  • Of course. I was making assumptions based on the question - perhaps unduely :) – kipz Jan 19 '09 at 14:11
  • Although your assumption holds true in this case, I am walking into the domain of premature optimization (which I should not do). – Jagmal Jan 19 '09 at 14:16
6

While in your case, the comment may be a good way to go, in Android, you can use Pair . Simply

return new Pair<>(yourList, yourCommaSeparatedValues);
ElectroBuddha
  • 630
  • 10
  • 20
serv-inc
  • 35,772
  • 9
  • 166
  • 188
6

Alternatively, in situations where I want to return a number of things from a method I will sometimes use a callback mechanism instead of a container. This works very well in situations where I cannot specify ahead of time just how many objects will be generated.

With your particular problem, it would look something like this:

public class ResultsConsumer implements ResultsGenerator.ResultsCallback
{
    public void handleResult( String name, Object value )
    {
        ... 
    }
}

public class ResultsGenerator
{
    public interface ResultsCallback
    {
        void handleResult( String aName, Object aValue );
    }

    public void generateResults( ResultsGenerator.ResultsCallback aCallback )
    {
        Object value = null;
        String name = null;

        ...

        aCallback.handleResult( name, value );
    }
}
  • sorry for commenting on a very old answer of yours, but how do callbacks go regarding Garbage collection? I certainly don't have a good understanding of memory management in `java`, if you have object `A` call object `B.getResult()` and `B.getResult()` calls `A.finishResult()` as a `callback`, does the object `B` get garbage collected or does it stay around till A finishes?? probably a stupid question but its a fundamental confusion i have! – wired00 Jun 05 '12 at 06:07
5

Use of following Entry object Example :

public Entry<A,B> methodname(arg)
{
.......

return new AbstractMap.simpleEntry<A,B>(instanceOfA,instanceOfB);
}
THelper
  • 15,333
  • 6
  • 64
  • 104
5

Regarding the issue about multiple return values in general I usually use a small helper class that wraps a single return value and is passed as parameter to the method:

public class ReturnParameter<T> {
    private T value;

    public ReturnParameter() { this.value = null; }
    public ReturnParameter(T initialValue) { this.value = initialValue; }

    public void set(T value) { this.value = value; }
    public T get() { return this.value; }
}

(for primitive datatypes I use minor variations to directly store the value)

A method that wants to return multiple values would then be declared as follows:

public void methodThatReturnsTwoValues(ReturnParameter<ClassA> nameForFirstValueToReturn, ReturnParameter<ClassB> nameForSecondValueToReturn) {
    //...
    nameForFirstValueToReturn.set("...");
    nameForSecondValueToReturn.set("...");
    //...
}

Maybe the major drawback is that the caller has to prepare the return objects in advance in case he wants to use them (and the method should check for null pointers)

ReturnParameter<ClassA> nameForFirstValue = new ReturnParameter<ClassA>();
ReturnParameter<ClassB> nameForSecondValue = new ReturnParameter<ClassB>();
methodThatReturnsTwoValues(nameForFirstValue, nameForSecondValue);

Advantages (in comparison to other solutions proposed):

  • You do not have to create a special class declaration for individual methods and its return types
  • The parameters get a name and therefore are easier to differentiate when looking at the method signature
  • Type safety for each parameter
koma
  • 1,237
  • 10
  • 9
  • Thanks for a solution that gives names and type safety to each returned value, without requiring a class declaration per set of returned value types. – ToolmakerSteve Nov 20 '14 at 21:06
3

All possible solutions will be a kludge (like container objects, your HashMap idea, “multiple return values” as realized via arrays). I recommend regenerating the comma-separated list from the returned List. The code will end up being a lot cleaner.

Bombe
  • 81,643
  • 20
  • 123
  • 127
  • I agree with you on this but if I do so, I will end up looping twice (I am actually creating the elements of list one-by-one in the existing method). – Jagmal Jan 19 '09 at 14:04
  • 1
    @Jagmal: you might loop twice, but it doesn't matter most of the time (see gizmos answer). – Joachim Sauer Jan 19 '09 at 14:06
  • 1
    Yeah, don’t try to optimize your code unless you really have to. gizmo is very right about that. – Bombe Jan 19 '09 at 14:11
3

Keep it simple and create a class for multiple result situation. This example accepts an ArrayList and a message text from a databasehelper getInfo.

Where you call the routine that returns multiple values you code:

multResult res = mydb.getInfo(); 

In the routine getInfo you code:

ArrayList<String> list= new ArrayList<String>();
add values to the list...
return new multResult("the message", list);

and define a class multResult with:

public class multResult {
    public String message; // or create a getter if you don't like public
    public ArrayList<String> list;
    multResult(String m, ArrayList<String> l){
        message = m;
        list= l;
}

}

Martin
  • 1,430
  • 10
  • 19
2

Can do some thing like a tuple in dynamic language (Python)

public class Tuple {
private Object[] multiReturns;

private Tuple(Object... multiReturns) {
    this.multiReturns = multiReturns;
}

public static Tuple _t(Object... multiReturns){
    return new Tuple(multiReturns);
}

public <T> T at(int index, Class<T> someClass) {
    return someClass.cast(multiReturns[index]);
}
}

and use like this

public Tuple returnMultiValues(){
   return Tuple._t(new ArrayList(),new HashMap())
}


Tuple t = returnMultiValues();
ArrayList list = t.at(0,ArrayList.class);
Deva
  • 105
  • 1
  • 7
2

I followed a similar approach than the described in the other answers with a few tweaks based on the requirement I had, basically I created the following classes(Just in case, everything is Java):

public class Pair<L, R> {
    final L left;
    final R right;

    public Pair(L left, R right) {
        this.left = left;
        this.right = right;
    }

    public <T> T get(Class<T> param) {
        return (T) (param == this.left.getClass() ? this.left : this.right);
    }

    public static <L, R> Pair<L, R> of(L left, R right) {
        return new Pair<L, R>(left, right);
    }
}

Then, my requirement was simple, in the repository Class that reaches the DB, for the Get Methods than retrieve data from the DB, I need to check if it failed or succeed, then, if succeed, I needed to play with the returning list, if failed, stop the execution and notify the error.

So, for example, my methods are like this:

public Pair<ResultMessage, List<Customer>> getCustomers() {
    List<Customer> list = new ArrayList<Customer>();
    try {
    /*
    * Do some work to get the list of Customers from the DB
    * */
    } catch (SQLException e) {
        return Pair.of(
                       new ResultMessage(e.getErrorCode(), e.getMessage()), // Left 
                       null);  // Right
    }
    return Pair.of(
                   new ResultMessage(0, "SUCCESS"), // Left 
                   list); // Right
}

Where ResultMessage is just a class with two fields (code/message) and Customer is any class with a bunch of fields that comes from the DB.

Then, to check the result I just do this:

void doSomething(){
    Pair<ResultMessage, List<Customer>> customerResult = _repository.getCustomers();
    if (customerResult.get(ResultMessage.class).getCode() == 0) {
        List<Customer> listOfCustomers = customerResult.get(List.class);
        System.out.println("do SOMETHING with the list ;) ");
    }else {
        System.out.println("Raised Error... do nothing!");
    }
}
halfer
  • 19,824
  • 17
  • 99
  • 186
Marco Vargas
  • 1,232
  • 13
  • 31
2

As I see it there are really three choices here and the solution depends on the context. You can choose to implement the construction of the name in the method that produces the list. This is the choice you've chosen, but I don't think it is the best one. You are creating a coupling in the producer method to the consuming method that doesn't need to exist. Other callers may not need the extra information and you would be calculating extra information for these callers.

Alternatively, you could have the calling method calculate the name. If there is only one caller that needs this information, you can stop there. You have no extra dependencies and while there is a little extra calculation involved, you've avoided making your construction method too specific. This is a good trade-off.

Lastly, you could have the list itself be responsible for creating the name. This is the route I would go if the calculation needs to be done by more than one caller. I think this puts the responsibility for the creation of the names with the class that is most closely related to the objects themselves.

In the latter case, my solution would be to create a specialized List class that returns a comma-separated string of the names of objects that it contains. Make the class smart enough that it constructs the name string on the fly as objects are added and removed from it. Then return an instance of this list and call the name generation method as needed. Although it may be almost as efficient (and simpler) to simply delay calculation of the names until the first time the method is called and store it then (lazy loading). If you add/remove an object, you need only remove the calculated value and have it get recalculated on the next call.

tvanfosson
  • 524,688
  • 99
  • 697
  • 795
1

This is not exactly answering the question, but since every of the solution given here has some drawbacks, I suggest to try to refactor your code a little bit so you need to return only one value.

Case one.

You need something inside as well as outside of your method. Why not calculate it outside and pass it to the method?

Instead of:

[thingA, thingB] = createThings(...);  // just a conceptual syntax of method returning two values, not valid in Java

Try:

thingA = createThingA(...);
thingB = createThingB(thingA, ...);

This should cover most of your needs, since in most situations one value is created before the other and you can split creating them in two methods. The drawback is that method createThingsB has an extra parameter comparing to createThings, and possibly you are passing exactly the same list of parameters twice to different methods.


Case two.

Most obvious solution ever and a simplified version of case one. It's not always possible, but maybe both of the values can be created independently of each other?

Instead of:

[thingA, thingB] = createThings(...);  // see above

Try:

thingA = createThingA(...);
thingB = createThingB(...);

To make it more useful, these two methods can share some common logic:

public ThingA createThingA(...) {
    doCommonThings(); // common logic
    // create thing A
}
public ThingB createThingB(...) {
    doCommonThings(); // common logic
    // create thing B
}
X. Wo Satuk
  • 923
  • 1
  • 11
  • 18
1

In C++ (STL) there is a pair class for bundling two objects. In Java Generics a pair class isn't available, although there is some demand for it. You could easily implement it yourself though.

I agree however with some other answers that if you need to return two or more objects from a method, it would be better to encapsulate them in a class.

kgiannakakis
  • 103,016
  • 27
  • 158
  • 194
1

Why not create a WhateverFunctionResult object that contains your results, and the logic required to parse these results, iterate over then etc. It seems to me that either:

  1. These results objects are intimately tied together/related and belong together, or:
  2. they are unrelated, in which case your function isn't well defined in terms of what it's trying to do (i.e. doing two different things)

I see this sort of issue crop up again and again. Don't be afraid to create your own container/result classes that contain the data and the associated functionality to handle this. If you simply pass the stuff around in a HashMap or similar, then your clients have to pull this map apart and grok the contents each time they want to use the results.

Brian Agnew
  • 268,207
  • 37
  • 334
  • 440
  • Because its a PITA to have to define a class whenever you need to return multiple values, merely because the language lacks this commonly useful feature ;) But seriously, what you suggest is very often worth doing. – ToolmakerSteve Nov 20 '14 at 21:19
1

You can utilize a HashMap<String, Object> as follows

public HashMap<String, Object> yourMethod()
{

   .... different logic here 

  HashMap<String, Object> returnHashMap = new HashMap<String, Object>();
  returnHashMap.put("objectA", objectAValue);
  returnHashMap.put("myString", myStringValue);
  returnHashMap.put("myBoolean", myBooleanValue);

  return returnHashMap;
}

Then when calling the method in a different scope, you can cast each object back to its initial type:

// call the method
HashMap<String, Object> resultMap = yourMethod();
                
// fetch the results and cast them
ObjectA objectA = (ObjectA) resultMap.get("objectA");
String myString = (String) resultMap.get("myString");
Boolean myBoolean = (Boolean) resultMap.get("myBoolean");
KAD
  • 10,972
  • 4
  • 31
  • 73
1
public class MultipleReturnValues {

    public MultipleReturnValues() {
    }

    public static void functionWithSeveralReturnValues(final String[] returnValues) {
        returnValues[0] = "return value 1";
        returnValues[1] = "return value 2";
    }

    public static void main(String[] args) {
        String[] returnValues = new String[2];
        functionWithSeveralReturnValues(returnValues);
        System.out.println("returnValues[0] = " + returnValues[0]);
        System.out.println("returnValues[1] = " + returnValues[1]);
    }

}
tsug303
  • 29
  • 1
0

Pass a list to your method and populate it, then return the String with the names, like this:

public String buildList(List<?> list) {
    list.add(1);
    list.add(2);
    list.add(3);
    return "something,something,something,dark side";
}

Then call it like this:

List<?> values = new ArrayList<?>();
String names = buildList(values);
Triqui
  • 9
  • 1
0

I noticed there is no no-custom class, n-length, no-cast, type-safe answers yet to returning multiple values.

Here is my go:

import java.util.Objects;

public final class NTuple<V, T extends  NTuple<?, ?>> {
    private final V value;
    private final T next;

    private NTuple(V value, T next) {
        this.value = value;
        this.next = next;
    }

    public static <V> NTuple<V, ?> of(V value) {
        return new NTuple<>(value, null);
    }

    public static <V,  T extends  NTuple<?, ?>> NTuple<V, T> of(V value, T next) {
        return new NTuple<>(value, next);
    }

    public V value() {
        return value;
    }

    public T next() {
        return next;
    }

    public static <V> V unpack0(NTuple<V, ?> tuple) {
        return Objects.requireNonNull(tuple, "0").value();
    }

    public static <V, T extends NTuple<V, ?>> V unpack1(NTuple<?, T> tuple) {
        NTuple<?, T> tuple0 = Objects.requireNonNull(tuple, "0");
        NTuple<V, ?> tuple1 = Objects.requireNonNull(tuple0.next(), "1");
        return tuple1.value();
    }

    public static <V, T extends NTuple<?, NTuple<V, ?>>> V unpack2(NTuple<?, T> tuple) {
        NTuple<?, T> tuple0 = Objects.requireNonNull(tuple, "0");
        NTuple<?, NTuple<V, ?>> tuple1 = Objects.requireNonNull(tuple0.next(), "1");
        NTuple<V, ?> tuple2 = Objects.requireNonNull(tuple1.next(), "2");
        return tuple2.value();
    }
}

Sample use:

public static void main(String[] args) {
    // pre-java 10 without lombok - use lombok's var or java 10's var if you can
    NTuple<String, NTuple<Integer, NTuple<Integer, ?>>> multiple = wordCount("hello world");
    String original = NTuple.unpack0(multiple);
    Integer wordCount = NTuple.unpack1(multiple);
    Integer characterCount  = NTuple.unpack2(multiple);

    System.out.println(original + ": " + wordCount + " words " + characterCount + " chars");
}

private static NTuple<String, NTuple<Integer, NTuple<Integer, ?>>> wordCount(String s) {
    int nWords = s.split(" ").length;
    int nChars = s.length();
    return NTuple.of(s, NTuple.of(nWords, NTuple.of(nChars)));
}

Pros:

  • no-custom container class - no need to write a class just for a return type
  • n-length - can handle any number of return values
  • no-cast - no need to cast from Object
  • type-safe - the types are checked via Java's generics

Cons:

  • inefficient for large numbers of return values
    • according to my experience with python's multiple return values, this should not happen in practice
  • heavy type declarations
    • can be alleviated by lombok/Java 10 var
bernard paulus
  • 1,644
  • 1
  • 21
  • 33
-4

In C, you would do it by passing pointers to placeholders for the results as arguments:

void getShoeAndWaistSizes(int *shoeSize, int *waistSize) {
    *shoeSize = 36;
    *waistSize = 45;
}
...
int shoeSize, waistSize;
getShoeAndWaistSize(&shoeSize, &waistSize);
int i = shoeSize + waistSize;

Let's try something similar, in Java.

void getShoeAndWaistSizes(List<Integer> shoeSize, List<Integer> waistSize) {
    shoeSize.add(36);
    waistSize.add(45);
}
...
List<Integer> shoeSize = new List<>();
List<Integer> waistSize = new List<>();
getShoeAndWaistSizes(shoeSize, waistSize);
int i = shoeSize.get(0) + waistSize.get(0);
Adrian Panasiuk
  • 7,249
  • 5
  • 33
  • 54
  • 1
    However in an OO language, it is generally considered preferable to do what several people already suggested four years before this answer: group the two related values into one object (pair, tuple, or custom class definition), and then have a list of those objects. Doing so avoids the need to pass around multiple lists. This becomes especially important if need to pass such a pair (one element of each of your lists) to other methods, for further processing. – ToolmakerSteve Nov 20 '14 at 21:10
  • @ToolmakerSteve To clarify: the lists are intended to have exactly one element each and are only a means to implement an analogue to pass-by-pointer. They are not intended to collect several results, or even be used further than a couple lines after the method call. – Adrian Panasiuk Nov 22 '14 at 03:57
-5

PASS A HASH INTO THE METHOD AND POPULATE IT......

public void buildResponse(String data, Map response);

l_39217_l
  • 2,090
  • 1
  • 13
  • 15