2

Решал, казалось бы, простую задачу, даже погуглил потом правильный код... Но в консоль выводится массив заданной величины, который заполнен единицами. Подскажите, в чем ошибка?

public class Test {

    public static void main(String[] args) {

      prMass(fibMass(20));

    }

    public static int[] fibMass(int x)
    {
        int m[]=new int[x];
        for(int i=0;i<x;i++)
        {
            if(m[i]<2)
                m[i]=1;
            else
                m[i]=m[i-1]+m[i-2];
        }

        return m;
    }
    public static void prMass(int[] x)
    {   int z=x.length;
        for(int i=0;i<z;i++)
        {
            System.out.print(x[i]+" ");
        }
        System.out.println();
    }
}

4 ответа 4

5
for(int i=0;i<x;i++) {
    if(m[i]<2)
         m[i]=1;
    else
         m[i]=m[i-1]+m[i-2];
}

Вы для каждого элемента массива, если его значение меньше двух, задаете значение равное 1. И переходите к следующему элементу.

Поскольку изначально массив инициализирован нулями, вполне ожидаемо, что вы получаете массив единиц.

2
  • 1
    стоило бы дописать что нужно сменить проверку элемента на проверку индекса :-)
    – Grundy
    13 авг 2016 в 10:28
  • Спасибо,даже не пойму как это можно было не заметить..
    – Artyom
    15 авг 2016 в 7:54
2

Вот еще одно решение:

     public class Fibonacci {
      public static int result=0;
      public static int count=0;
      public static int a=1, b=1;
      public static int GenerateFibonacci(int number) {
        while (count<=number){
          System.out.print(a+" ");
          b=b+a;
          a=b-a;
          count++;
        }
        return result;
      }
      public static void main(String[] args) {
        GenerateFibonacci(5);
      }
    }
0
for(int i=0;i<x;i++) {
    if(i<2)
       m[i]=1;
    else
       m[i]=m[i-1]+m[i-2];
}
2
  • @Mihanik71, как ни странно, но В опубликованном сообщении содержится решение вопроса
    – Grundy
    13 авг 2016 в 22:36
  • @user217504 было бы хорошо если бы Вы указали в чём именно проблема
    – Mihanik71
    13 авг 2016 в 22:46
0

Вариант с использование stream:

    int count = 100;
    Stream
            .generate(new Supplier<BigDecimal>() {
                BigDecimal prevSecondNumber = BigDecimal.ZERO;
                BigDecimal prevFirstNumber = BigDecimal.ZERO;

                @Override
                public BigDecimal get() {
                    BigDecimal result = prevFirstNumber.add(prevSecondNumber);
                    if (Objects.equals(result, BigDecimal.ZERO)) result = BigDecimal.ONE;

                    prevSecondNumber = prevFirstNumber;
                    prevFirstNumber = result;

                    return result;
                }
            })
            .limit(count)
            .forEach(System.out::println);

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.