C# Błąd w programie obliczającym silnie za pomocą funkcji

Question

Witam, mam problem z tym oto kodem:

Po debugowaniu zamiast silni z "a" zwraca mi 0. Wiecie może gdzie zrobiłem błąd?

 

Kod: 

using System;

namespace funkcje3
{
    class Program
    {
        static void Main(string[] args)
        {
            int wynik = 1;

            Console.WriteLine("Podaj a: ");
            int a = Int32.Parse(Console.ReadLine());
            Silnia(ref a,ref wynik );
            Console.WriteLine(a);
            

        }
        static int Silnia(ref int a, ref int wynik)
        {
         
            for (int i = 0; i <= wynik; i++)
            {
                a *= i;
            }            
            return a;

        }

    }

}

Answer 1

Zaczynasz pętlę od zera, więc mnożysz przez zero, a cokolwiek razy zero == zero.

(btw, raczej trochę mało sensu że funkcja silnia bierze ref int - niech po prostu zwraca wynik i tyle?)

Comments

Tak jak sugerowałeś zmieniłem i  na i=1 i teraz zamiast 5! = 120 wyświetla mi 5!=5 musi być jeszcze jakiś błąd

---

Spójrz jaką zmienną używasz jako liczbę wywołań funkcji i jaką ma wartość ;)

---

Męczę się nad tym już tyle i nic nie mogę wykombinować, próbowałem już każdej możliwej wariacji poza tą jedną prawdziwą, mógłbyś mnie bardziej nakierować? Wydaje mi się że to jest problem z return, bo nie zwraca żadnej liczby po zakończeniu funkcji.

 

using System;

namespace funkcje3
{
    class Program
    {
        static void Main(string[] args)
        {
            int wynik = 1;

            Console.WriteLine("Podaj a: ");
            int a = Int32.Parse(Console.ReadLine());
            Silnia(a,wynik );
            

        }
        static int Silnia(int a, int wynik)
        {
          
            for (int i = 1; i <= a; i++)
            {
                a *= i;
            }            
            return a;

        }

    }

}

---

bo nie zwraca żadnej liczby po zakończeniu funkcji.

Zwraca, ale w ogóle z funkcji nie wyciągasz wartości którą zwróciła. Poczytaj może jeszcze raz, jak się woła funkcje które coś zwracają :)

            for (int i = 1; i <= a; i++)
            {
                a *= i;
            }          

A to jest masło maślane. Tą samą zmienną używasz jako zmienną przechowującą wynik silnii i jako liczbę wywołań pętli.

Answer 2

Dobra udało mi się to rozgryść, zostawiam tu kod jakby ktoś miał taki sam problem jak ja

 

 

using System;

namespace funkcje3
{
    class Program
    {
        static void Main(string[] args)
        {
            int wynik = 1;

            Console.WriteLine("Podaj a: ");
            int a = Int32.Parse(Console.ReadLine());
            Console.WriteLine(Silnia(a, wynik));

        }
        static int Silnia(int a, int wynik)
        {
            
          for (int i = 1; i <= a; i++)
            {

                wynik = wynik * i;
               
            }
            return wynik;   

        }
        

    }
}

Comments

OK, tylko wciąż przekombinowane :)

Nie potrzebujesz tego argumentu do funkcji `wynik`, ani zmiennej `wynik` w mainie. Wystarczy mieć zmienną wynik w funkcji liczącej silnię.