Can´t return fibonacci sequence

Question

I tried to return fibonacci sequence in java, but i have a problem with the values.

Example code.

@Override
public int fibonacci(int numero) throws RemoteException {
    int numero1 = 1;
    int numero2 = 0;
    int fibonacci = 0;
    for (int i = 0; i <= numero; i++) {
        fibonacci = numero1 + numero2;
        numero1 = numero2;
        numero2 = fibonacci;
        //return "La serie fibonnaci es:" + fibonacci;
    }
    return fibonacci;

}

Fibonacci variable return 0 but i need to return a fibonnaci sequence. ¿Any idea?

RMI has nothing to do with this question. Your remote method implementation doesn't need to be declared as throwing `Remote Exception.` — user207421, Feb 14 '16 at 02:22

score 2 · Accepted Answer · answered Feb 13 '16 at 20:01

2

Two peoposals for your problem:

Number one: Print on screen

public static void fibonacci(int numero) {
    int numero1 = 0;
    int numero2 = 1;
    int fibonacci = 0;
    for (int i = 0; i <= numero; i++) {
        fibonacci = numero1 + numero2;
        numero1 = numero2;
        numero2 = fibonacci;
        System.out.println(fibonacci);
    }
}

Number two (better in my opinion): Return list

public static List<Integer> fibonacci(int numero) {
    LinkedList<Integer> fibonacciSequence = new LinkedList<>();
    int numero1 = 0;
    int numero2 = 1;
    int fibonacci = 0;
    for (int i = 0; i <= numero; i++) {
        fibonacci = numero1 + numero2;
        numero1 = numero2;
        numero2 = fibonacci;
        fibonacciSequence.add(fibonacci);
    }
    return fibonacciSequence;
}

answered Feb 13 '16 at 20:01

exception1

1,239
8
17

The fibonacci method implements interface so need this return :$ – El Hombre Sin Nombre Feb 13 '16 at 20:19
Then use the second one – exception1 Feb 13 '16 at 21:29
One question ¿Why LinkedList and not Arraylist? – El Hombre Sin Nombre Feb 13 '16 at 22:21
Does not matter which one you take, as both implement `List` interface. You can also use ArrayList. It might even be more accurate since the length of the list is known in advance. – exception1 Feb 13 '16 at 22:26

score 0 · Answer 2 · answered Feb 13 '16 at 19:56

0

You switched numero1 and numero2 during the initialization.

This should fix it:

int numero1 = 0;
int numero2 = 1;

answered Feb 13 '16 at 19:56

TimoStaudinger

41,396
16
88
94

Right, but does not solve OP's problem. – exception1 Feb 13 '16 at 20:03

Can´t return fibonacci sequence

2 Answers2