[java]è efficente?(codice banale)

[Dim11]

Ciao a tutti, ho da poco iniziato a studiare un po' Java dato che sono diplomato come perito informatico e sono a digiuno dalla programmazione da qualche anno..(secondo voi ne vale la pena per avere possibilità di lavoro?).
Ho due domande stupide su questo piccolo codice che ho fatto prima:

-come mai se ometto l'ELSE l'IF non funziona nel modo corretto? non dovrebbe essere facoltativo se è presente solo un'istruzione 'alternativa' all'if?
-a vederlo così il codice mi sembra ridondante e che si potrebbe scrivere in modi più efficenti..se si, come?
Tnx

import javax.swing.*;
public class es8
{
  public static void main (String argv[])
  {
  	int n;
  	int j;
  	int m;
  	int k=Integer.parseInt(JOptionPane.showInputDialog(null,"inserisci un valore:"));

  	for(int i=k;i>=2;i--)
  	{
  	   System.out.print("\r"+i+" -> ");
  	   if(i%2==0)	
  	     n=i/2;
  	   else 
  	     n=((3*i)+1); 
  	   System.out.print("f("+i+")="+n);
	
  	   for(j=n;j>1;)
  	  {
  	     System.out.print(" -> ");
  	     if(j%2==0)	
  	       m=j/2;
  	     else  
  	       m=((3*j)+1); 
  	     System.out.print("f("+j+")="+m);
  	     j=m;
  	  }	
  	  j=n;	
  	}
  }
}

[banryu79]

Io l'ho fatto così:

import java.util.*;
import javax.swing.JOptionPane;

/**
 * Computing Collatz sequences 
 * see http://www.hwupgrade.it/forum/showthread.php?t=2554545
 *
 * CollatzComputer
 * - launcher. Accepts a "gui" arguments to launch the GUI version, otherwise
 * - the Consolle version is used.
 * 
 * CollatzGuiApp
 * - the GUI app. Use javax.swing.JOptionPane for input/output.
 * 
 * CollatzConsolleApp
 * - the consolle app. Use java.io.System for input/output.
 * 
 * CollatzSeq
 * - Istances of this class rappresent Collatz Sequences.
 * 
 */
public class CollatzComputer {
    
    public static void main(String[] args) {
        if (args.length > 0 && "gui".equalsIgnoreCase(args[0])) {
            CollatzGuiApp.run();
        } else {
            CollatzConsolleApp.run();
        }
    }
}


class CollatzGuiApp {

    static void run() {
        String inputMsg = "Please, insert an integer N bigger than zero.";
        String errorMsg = "Invalid input!";
        int n;
        
        while (true) {
            try {
                n = Integer.parseInt(JOptionPane.showInputDialog(null, inputMsg));
            } catch (NumberFormatException ex) {
                JOptionPane.showMessageDialog(null, errorMsg);
                continue;
            }
            if (n > 0) break;
        };
        
        StringBuilder buffer = new StringBuilder();
        for (int i=n; i>0; i--) {
            CollatzSeq sequence = CollatzSeq.forInteger(i);
            buffer.append(sequence).append("\n");
        }
        
        JOptionPane.showMessageDialog(null, buffer);
    }
    
}

class CollatzConsolleApp {

    static void run() {     
        System.out.println("CollatzConsolleApp > Welcome.");
        
        Scanner scanner = new Scanner(System.in);
        scanner.useDelimiter("\n");
        String inputMsg = "Please, insert an integer N bigger than zero.";
        String errorMsg = "Invalid input!";
        int n;
        
        while (true) {
            System.out.println(inputMsg);
            try {
                n = Integer.parseInt(scanner.next());
            } catch (NumberFormatException ex) {
                System.out.println(errorMsg);
                continue;
            }
            if (n > 0) break;
        };
        
        for (int i=n; i>0; i--) {
            CollatzSeq sequence = CollatzSeq.forInteger(i);
            System.out.println(sequence);
        }
        
        System.out.println("End.");
    }
    
}

/**
 * A Collatz sequence.
 */
final class CollatzSeq implements Iterable<Integer> {
    
    /**
     * Compute the Collatz sequence for the integer n
     * @param n the first integer in the Collatz sequence to compute
     * @return the Collatz sequence starting from n to 1 inclusive
     */
    static CollatzSeq forInteger(int n) {
        return new CollatzSeq(n);
    }
    
    // compute the Collatz sequence [n..1] - iterative method
    private static List<Integer> iterativeSequence(int n) {
        List<Integer> list = new ArrayList<>();
        
        while (n > 1) {
            list.add(n);
            if (n % 2 == 0) {
                n = n/2;
            } else {
                n = 3*n + 1;
            }
        }
        
        list.add(1);
        
        return list;
    }
    
    // compute the Collatz sequence [n..1] - trampoline
    private static List<Integer> recursiveSequence(int n) {
        List<Integer> list = new ArrayList<>();
        recursiveSequence0(n, list);
        return list;
    }
    
    // compute the Collatz sequence [n..1] - recursive method with accumulator
    private static void recursiveSequence0(int n, List<Integer> acc) {
        acc.add(n);
        
        if (n == 1) return;
        
        if (n % 2 == 0) {
            recursiveSequence0(n/2, acc);
        } else {
            recursiveSequence0(3*n+1, acc);
        }
    }
    
    // For testing purposes
    public static void main(String[] args) {
        CollatzSeq.test101(1);
        CollatzSeq.test101(4);
        CollatzSeq.test101(10);
        CollatzSeq.test101(79);
        CollatzSeq.test101(101);
    }
    
    private static void test101(int n) {
        CollatzSeq sequence = CollatzSeq.forInteger(n);
        System.out.println(sequence);
        System.out.println(sequence.count() + " elements");
        System.out.println("");
    }
    
    private final List<Integer> sequence;
    
    /**
     * Private 'ctor. 
     * Use static method CollatzSeq.forInteger to create instances.
     */
    private CollatzSeq(int n) {
        //List<Integer> list = recursiveSequence(n);
        List<Integer> list = iterativeSequence(n);
        sequence = Collections.unmodifiableList(list);
    }
    
    /**
     * @return the number of integers in this CollatzSeq
     */
    public int count() {
        return sequence.size();
    }
    
    @Override
    public String toString() {
        StringBuilder buffer = new StringBuilder().append("[");
        
        Iterator<Integer> it = sequence.iterator();
        while (it.hasNext()) {
            buffer.append(it.next());
            if (it.hasNext()) buffer.append(" -> ");
        }
        
        return buffer.append("]").toString();
    }
    
    @Override 
    public Iterator<Integer> iterator() {
        return sequence.iterator();
    }
}