Recursion through a linked list in Java

Question

I need to code a method that prints the contents of a linked list recursively, however my method isn't working.

 public LinkedList reverse() {
    LinkedList list = new LinkedList();
    list = reverseRecursion(list, 0);
    return list;
  }
  public LinkedList reverseRecursion(LinkedList list, int pos) {
    if (pos != size()) {
      rotate();
      list.add(get());
      pos = pos + 1;
      reverseRecursion(list, pos);
    }
    return list;
  }

  public String toString() {
    String result = "Contents of list: {";
    result = toStringRecursion(result, 0);
    return result;
  }
  private String toStringRecursion(String result, int pos) {
    if (pos != size()) {
      rotate();
      String temp = result + get().getData() + ",";
      pos = pos + 1;
      toStringRecursion(temp, pos);
    }     
    return result;
  }

the reverse function is working fine, however my toString is only showing up as the initial result "Contents of list: {", despite being passed the previous result as a parameter. If I print the result each time I pass through the toStringRecursion, I can see the desired output right before it reverts back to the inital result when the return line is reached.

xlm · Answer 1 · 2014-02-18T01:51:50.227

Among other problems with your current approach, you aren't returning the result of the recursive calls to your list:

  private String toStringRecursion(String result, int pos) {
    if (pos != size()) {
      rotate();
      String temp = result + get().getData() + ",";
      pos = pos + 1;

      // Recurse upon the appended String
      return toStringRecursion(temp, pos);
    }     
    // Stop recursing
    return "}";
  }

There's actually much simpler way of approaching this but without seeing your full class definition I can't provide further meaningful feedback without giving a lengthy explanation on linked lists as recursive data structures.

score 0 · Accepted Answer · edited Feb 18 '14 at 02:13

You need to append the String to the return value. But in this case, using StringBuilder matches what you want to do. That's not good for performance because "+" creates new String instance every time it's called.

      public String toString() {
        StringBuilder sb = new StringBuilder("Contents of list: {");

        appendRecursion(sb, 0);
        return sb.toString();
      }
      private void appendRecursion(StringBuilder result, int pos) {
        if (pos != size()) {
          rotate();
          result.append(get().getData()).append(",");
          pos = pos + 1;
          appendRecursion(result, pos);
        }     
      }

Recursion through a linked list in Java

2 Answers2