public static void Control() { Console.WriteLine("Wcisnij 1 by obliczyć silnię lub E by wyjść"); sterowanie = Console.ReadLine().ToString(); if (sterowanie == "1") { Silnia.EterValues(); Silnia.GetLoopValue(Silnia.liczba); Console.WriteLine(Silnia.wynik); Silnia.GetRecursiveValue(Silnia.liczba); Console.WriteLine(Silnia.wynik); Sterowanie.Control(); } else if (sterowanie.ToLower() == "e") { } else { Console.WriteLine("Wybierz tylko jedną z opcji"); Sterowanie.Control(); } { } }
//TODO Trochę tu trzeba posprzątać. To jest prosta aplikacja ale mimo wszystko standardy trzeba zachowywać i tak. //TODO Stworz klasę Silnia ktora będzie miała 2 metody. GetRecursiveValue i GetLoopValue, ktore to będą liczyć tą silnie. //TODO Lub też patrząc bardziej obiektowo. Interfejs IFactorial z metodą CountFactorial i dwie implementacje tego interfejsu LoopFactorial i RecursiveFactorial. //TODO W metodzie main skoro już jesteśmy w konsolowej aplikacji powinno być sterowanie, tworzenie potrzebnych obiektów. //TODO Kod powinien być samodokumentujący się. Popraw nazwy zmiennych i metod - method2/liczba to nie są dobre nazwy. Już lepiej byłoby PoliczSilnięRekurencyjnie i wynikCzesciowy //TODO Formatuj kod - jak wyżej CTRL + E + C sformatuje ci ładnie kod. //TODO Wprowadz te poprawki co pisalem i sprawdze jeszcze raz :) private static void Main(string[] args) { Sterowanie.Control(); }