Program ankieta

Cześć, piszę sobie taką prostą ankietę w C#, gdzie na podstawie zebranych danych program przydzieli nam zawód, do którego pasujemy.
Jest 5 zawodów, pytania wczytuję z pliku. Jest 10 pytań, po 2 pytania do każdego zawodu, odpowiedzi tak lub nie.

Ja zrobiłem to tak na piechotę, ponieważ porobiłem tam ify i jeśli odpowiedź na 1 i 2 pytanie była tak, to dodaje 2 punkty do kucharza, jeśli odpowiedź na 3 i 4 pytanie była tak i nie, to dodaje 1 punkt do nauczyciela itd.
Na końcu program sprawdza, który z zawodów ma więcej punktów i wyświetla go na ekranie.
Niestety ten sposób jest bardzo słaby, tak ręcznie zrobiony, ale nie mam pomysłu jak można by to było zrobić lepiej...

Może macie jakieś ciekawe pomysły? Będę wdzięczny

Kod programu:

static void Main(string[] args)
        {
            int kucharz = 0;
            int nauczyciel = 0;
            int ogrodnik = 0;
            int strazak = 0;
            int mechanik = 0;
            int nr_pyt = 0;

            Console.SetWindowSize(60, 30);
            Console.Title = "Program";
            Console.ForegroundColor = ConsoleColor.Cyan;
            Console.BackgroundColor = ConsoleColor.DarkBlue;
            Console.Clear();
            Console.SetCursorPosition(10, 10);
            Console.WriteLine("Hello World");
            Console.WriteLine("To okno nazywa się " + Console.Title);

            string[] lines = File.ReadAllLines("Pytania.txt");

            for (int i = 0; i < 10; i++)
            {
                string odp;

                Console.WriteLine(lines[nr_pyt]);
                nr_pyt++;

                do
                {
                    odp = Console.ReadLine();
                    if (odp != "t" && odp != "T" && odp != "n" && odp!="N")
                  {
                      Console.Beep(300, 200);
                      Console.WriteLine("Twoja odpowiedź była inna niż t lub n! Spróbuj jeszcze raz:");
                  }

                } while (odp != "t" && odp != "T" && odp != "n" && odp != "N");


                if (nr_pyt == 1 || nr_pyt == 2)
                    {
                        if (odp == "t" || odp == "T") ++kucharz;
                    }

                    if (nr_pyt == 3 || nr_pyt == 4)
                    {
                        if (odp == "t" || odp == "T") ++nauczyciel;
                    }

                    if (nr_pyt == 5 || nr_pyt == 6)
                    {
                        if (odp == "t" || odp == "T") ++ogrodnik;
                    }

                    if (nr_pyt == 7 || nr_pyt == 8)
                    {
                        if (odp == "t" || odp == "T") ++strazak;
                    }

                    if (nr_pyt == 9 || nr_pyt == 10)
                    {
                        if (odp == "t" || odp == "T") ++mechanik;
                    }

                }


            if (kucharz > nauczyciel && kucharz > ogrodnik && kucharz > strazak && kucharz > mechanik) Console.WriteLine("Zostaniesz kucharzem!");
            if (nauczyciel > kucharz && nauczyciel > ogrodnik && nauczyciel > strazak && nauczyciel > mechanik) Console.WriteLine("Zostaniesz nauczycielem");
            if (ogrodnik > kucharz && ogrodnik > nauczyciel && ogrodnik > strazak && ogrodnik > mechanik) Console.WriteLine("Zostaniesz ogrodnikiem");
            if (strazak > kucharz && strazak > nauczyciel && strazak > ogrodnik && strazak > mechanik) Console.WriteLine("Zostaniesz strażakiem");
            if (mechanik > kucharz && mechanik > nauczyciel && mechanik > ogrodnik && mechanik > strazak) Console.WriteLine("Zostaniesz mechanikiem");

            Console.ReadLine();

        }

1. Używaj list, np. do przechowywania pytań
2. Jak już będziesz miał listy to użyj pętli "foreach"
3. Z zawodów możesz zrobić Enum, a potem zrobić słownik i inkrementować dany klucz

var dict = new Dictionary<Professions, int>;
dict[element.Key] += 1;

4. Maksymalną wartość znajdziesz dzięki metodzie Max(). np.

dict.Keys.Max() //wartosc klucza z maksymalna wartością
dict.Values.Max() //maksymalny klucz

5. Jak sprawdzasz T/N to odpowiedź konwertuj do małej litery, aby bez sensu porównywać małe i wielkie litery

odp.ToLower() != "n" && odp.ToLower() != "t"

Znajdź jakiś kurs/książkę od C# i w prostych programach uaktualniaj kod do nowo poznanych składki i mechanizmów

annonymouzinho napisał(a):

Cześć, piszę sobie taką prostą ankietę w C#, gdzie na podstawie zebranych danych program przydzieli nam zawód, do którego pasujemy.
Jest 5 zawodów, pytania wczytuję z pliku. Jest 10 pytań, po 2 pytania do każdego zawodu, odpowiedzi tak lub nie.

Ja zrobiłem to tak na piechotę, ponieważ porobiłem tam ify i jeśli odpowiedź na 1 i 2 pytanie była tak, to dodaje 2 punkty do kucharza, jeśli odpowiedź na 3 i 4 pytanie była tak i nie, to dodaje 1 punkt do nauczyciela itd.

To jest bez sensu, że pytania masz w pliku, a ich interpretacja znajduje się w kodzie. Nie możesz dodać nowych pytań, zawodów ani zasad bez zmiany kodu, co daje ten plik?

            Console.SetWindowSize(60, 30);
            Console.Title = "Program";
            Console.ForegroundColor = ConsoleColor.Cyan;
            Console.BackgroundColor = ConsoleColor.DarkBlue;
            Console.Clear();
            Console.SetCursorPosition(10, 10);
            Console.WriteLine("Hello World");
            Console.WriteLine("To okno nazywa się " + Console.Title);

To powinno się znaleźć w jakiejś oddzielnej funkcji.

(odp != "t" && odp != "T" && odp != "n" && odp!="N")

To również. Niech funkcja przyjmuje string i zwraca true jeśli to prawidłowa wartość napisu.

                        if (odp == "t" || odp == "T") 

I to też. Unikniesz powielania tych samych warunków wielokrotnie.

++kucharz;

Czemu preinkrementacja? Uczysz się z jakiejś starożytnej księgi do C++?

Co do meritum - ja bym umieścił wszystkie potrzebne dane i zasady w pliku, np. w ten sposób:

nauczyciel;strażak;kucharz
Czy lubisz bawić się sikawką?|0;1;0
Czy lubisz dzieci?|1;0;0
Czy lubisz kroić warzywa?|0;0;1

Masz tu wszystkie dane potrzebne do ułożenia takiego testu, liczba zawodów i pytań może być dowolna, a do tego możesz nadawać nawet różne wagi pytaniom (nie wszędzie musi być 0 i 1, mogą być inne wartości). Teraz wystarczy tak napisać program, aby wczytał listę zawodów oraz pytań, następnie w pętli zadawał pytania i w zależności od odpowiedzi dodawał punkty zawodom.

somekind, Twój pomysł podoba mi się najbardziej. Jednak mam pewien problem z wczytywaniem tych danych. Z zawodami sobie bez problemu poradziłem:

StreamReader sr = new StreamReader("nowy.txt");

            string linia = sr.ReadLine();
            string[] zawody = linia.Split(';');

Jednak oddzielanie tych pytań od punktów jakoś mi nie wychodzi :/ Próbowałem zrobić podobnie, ale coś się sypie

while(sr.EndOfStream==false)
            {
                string linia2 = sr.ReadLine();
                string[] pytania = linia2.Split('|');
                string linia3 = sr.ReadLine();
                string[] punkty = linia3.Split(';');
            }

ReadLine czyta całą linię, wewnątrz pętli czytasz dwie linie, jedną splitujesz po |, a drugą po ;, a przecież trzeba jedną linię posplitować najpierw po |, a potem drugi element tak powstałej tablicy po ;.

I w ogóle po co ten StreamReader? Z File.ReadAllLines było łatwiej.

Czyli najlepiej by było najpierw wczytać wszystkie linie z pliku, a potem rozdzielać te dane i wrzucać do konkretnych tablic, tak?

No ja bym tak zrobił, bo myślę, że tak jest łatwiej. Ten plik i tak będzie malutki, więc nie ma po co bawić się w jakiś sekwencyjny odczyt, można go wczytać na raz.

Ok, dziękuję za pomoc. Jeszcze mam jedno pytanie, bo jak tworzę tablicę w pętli, to potem jak chcę wypisać pytania, to podkreśla mi pytania na czerwono i jest taki komunikat: "The name 'pytania' does not exist in the current context". Jak wywalę to z pętli to jest ok, ale pętla jest mi tutaj potrzebna. Wiesz może czemu tak się dzieje?

for (int n = 1; n < 11; n++)
            {
                string[] pytania = linia2[n].Split('|');
            }

for (int i = 0; i < 10; i++)
            {
                Console.WriteLine(pytania[i]);
                .
                .
             }

string[] pytania = new string[10]; // utworzenie 10 elementowej tablicy
for (int n = 0; n < 10; n++)
{
    pytania[n] = linia2[n].Split('|');
}
for (int i = 0; i < 10; i++)
{
    Console.WriteLine(pytania[i]);
}

Wygoogluj sobie takie hasła jak scope of variables, zakres zmiennych czy coś w tym stylu.

Tylko, że teraz podkreśla mi ten fragment:

linia2[n].Split('|');

bo próbuję konwertować string[] na string :/

Przepraszam, powyższa odpowiedź jest błędna jeżeli chodzi o kod. Jeżeli chcesz mieć zapisane pytania w zmiennej to możesz użyć tzw. jagged array. Działa to na zasadzie, że masz zew i wewnętrzną tablice. Wewnętrzna tablica może mieć różny rozmiar elementów.

string[][] pytania = new string[10][];

		
for (int n = 0; n < 10; n++)
{
	pytania[n] = linia2[n].Split('|');
}
for (int i = 0; i < pytania.Length; i++)
{
	string[] wew = pytania[i];
	for(int j = 0; j<wew.Length; j++)
	Console.WriteLine(wew[j]);
}```

Ok, dziękuję wszystkim za pomoc. Temat można zamknąć, pozdrawiam