Rozwiązany: [c++] Ciąg Fibonacciego iteracyjnie

[Hyronimaru]

Witam. Już na samym wstępie zaznaczam że jestem zupełnym nowicjuszem w programowaniu, ale w ramach praktyki i samodoskonalenia postanowiłem napisać program w C++ wyświetlający wskazaną liczbę w ciągu Fibonacciego. Jednak jakiejkolwiek wartości bym nie wpisał zawsze w konsoli otrzymuję 1. Proszę Was o wskazanie błędu (błędów) i ewentualnej ich korekcie.

#include <iostream>
using namespace std;
int fibonacci (int k) {
    if(k=0) return 0;
    if(k=1) return 1;
    if(k=2) return 1;
    else;
    fibonacci(k-1)+fibonacci(k-2);
    }
int main () {
    int k;
    cout << "ktory wyraz ciagu wygenerowac: " << endl;
    cin >> k;
    cout << fibonacci;
}

PS. Macie jakieś pomysły co do tworzenia prostych programów w celu opanowania języka? Uczę się z książki "Thinking in C++" Bruce'a Eckela, ale dopiero ją zacząłem. Z góry dzięki

[Jakobeks]

#include <iostream>
using namespace std;
int fibonacci (int k) {
    if(k=0) return 0;
    if(k=1) return 1;
    if(k=2) return 1;
    else; //tutaj kończysz else (średnik zaraz po słowie kluczowym)
    fibonacci(k-1)+fibonacci(k-2); //tutaj wywołujesz tylko funkcję, nic nie zwracasz
    }
int main () {
    int k;
    cout << "ktory wyraz ciagu wygenerowac: " << endl;
    cin >> k;
    cout << fibonacci;
}

[Hyronimaru]

Okej, ale po usunięciu średnika po else nadal dostaję ciągle 1. Jak zmienić kod żeby było dobrze?

[[Ekspert] Hakken]

1. Zmienne porównuje się podwójnym znakiem równości (==)

Pojedyńczy znak równości (=) przypisuje wartość.

2. Funkcja fibbonacci przyjmuje argument (k), którego nie podajesz wywołując ją.

3. Funkcja fibbonacci nic nie zwraca, a powinna.

A tak swoją drogą: to jest fibbonacci metodą rekurencyjną, w iteracyjnej masz wszystko w pętli.

[Hyronimaru]

Kyrie, fakt, mój błąd. I'm ashamed. Czyli powinienem zastosować pętlę 'for'?

[[Ekspert] Hakken]

To zależy, czy chcesz napisać rekurencyjnie, ale pomyliłeś się w nazwie tematu, czy też chcesz pisać iteracyjnie, ale przez przypadek piszesz rekurencyjnie

Jeśli ma być iteracyjnie, tak tak, powinienieś użyć pętli (np. for).

[Hyronimaru]

Chcę napisać iteracyjnie, co prawda znalazlem w potężnym narzędziu jakim jest wyszukiwarka kod pisany rekurencyjnie, mianowicie

int fibonacci(int n) {
    if(n == 0)  return 0;
    if(n == 1)  return 1;

    return fibonacci(n-1)+fibonacci(n-2);
}

Jednak, kolejny dowód mego lamerstwa, dostawałem błąd "undefined reference to 'WinMain@16'". Odpuściłem więc i zacząłem pisać od zera z zamiarem napiasnia iteracyjnie

[[Ekspert] Hakken]

Kod który znalazłeś jest poprawny, referencja do winmain musi być gdzieś indziej.

[Hyronimaru]

Po wielu próbach, po hektolitrach przelanej krwi i łez, udało mi się napisać poprawny kod. Dziękuję wszystkim za pomoc, temat do zamknięcia.

[[Ekspert] Hakken]

Problem rozwiązany, więc temat zamykam.

W razie potrzeby otwarcia tematu, proszę o kontakt przez PW.