Pomoc z kodem

Question

Witam
Potrzebował bym w pomocy gdyż dopiero się uczę i do końca nie ogarniam tego języka. Mam do wykonania polecenie jak na zdjęciu jednak mój program nie działa. Dopiero zaczynam swoją przygode w środowisku C# więc licze na wyrozumiałość.

Kod:

using System;
public class Uczen {
    public string Imie {set; get;}
    public string Nazwisko { set; get; }
    public int Pesel { set; get; }
    public Uczen() { }
    public Uczen(string imie, string nazwisko, int pesel)
        {
            Imie = imie;
            Nazwisko = nazwisko;
            Pesel = pesel;
        }
    Uczen u1 = new Uczen("Adam", "Kowalski", 05 285 295 351);
    static void Wypisz()
    {
        Console.WriteLine("Imie: " + Imie + "Nazwisko: " + Nazwisko + "Pesel: " + pesel);
    }
}

public class HelloWorld{
static void Main()
  {
    Console.WriteLine("Koniec programu");
  }
}

Answer 1

Jest nie tak źle, ale z kilkoma problemami:

static void Wypisz()

metoda Wypisz ma sens dla konkretnej instancji ucznia, więc nie powinna być statyczna.

Uczen u1 = new Uczen("Adam", "Kowalski", 05 285 295 351);

Kod wykonywalny musi zawsze się znaleźć w _jakiejś_ metodzie - tutaj miejscem gdzie chcesz stworzyć ucznia i go wypisać jest wnętrze Main()a. No i musisz też kiedyś wywołać `u1.Wypisz()` :)

public int Pesel { set; get; }

Pesel nie jest liczbą, bo może się zaczynać od zer (i mają one znaczenie). (a nawet gdyby był liczbą, to nie intem, bo int jest za mały na liczbę 11-cyfrową) Znacznie prościej będzie tutaj stringa użyć (lub tablicę 11stu małych liczb, ale to bardziej skomplikowane)

Answer 2

Najważniejsze: programować po angielsku. Tutaj to zapewne zadanie od nauczyciela, więc... po prostu pomińmy to tym razem.

Pole dla numeru PESEL uczyniłbym ciągiem znaków: łatwiej z walidacją z wyrażeniami regularnymi, a poza tym maksymalny zakres typu Int32 to 2^31, czyli liczba 10-cyfrowa, a PESEL ma ich 11:

private string _Pesel { get; set; }

public string Pesel
{
    get => _Pesel;

    set
    {
        Regex regex = new(@"^\d{11}$");
        if (!regex.IsMatch(value))
        {
            throw new ArgumentException("Wprowadzony numer PESEL jest nieprawidłowy.");
        }
        else
        {
            _Pesel = value;
        }
    }
}

Konstruktor z parametrami można skrócić:

public Uczen(string imie, string nazwisko, string pesel)
{
    (Imie, Nazwisko, Pesel) = (imie, nazwisko, pesel);
}

Dodaj metodę publiczną void Wypisz() która wypisuje dane ucznia

Tutaj to – według zadania – musi zostać, ale znacznie lepiej byłoby po prostu przeciążyć metodę ToString, a dopiero z logiki głównej ją wywołać podczas wypisywania:

public override string ToString()
{
    return $"Imie: {Imie}{Environment.NewLine}Nazwisko: {Nazwisko}{Environment.NewLine}PESEL: {Pesel}";
}

W przypadku wypisywania, warto użyć interpolacji, która tworzy jeden ciąg znaków (w .NET ciąg znaków jest niezmienny, a to oznacza, że przy każdej konkatenacji ciągów tworzony jest nowy, a nie dołączany do istniejącego):

public void Wypisz()
{
    Console.WriteLine($"Imie: {Imie}{Environment.NewLine}Nazwisko: {Nazwisko}{Environment.NewLine}PESEL: {Pesel}";);
}

Zamiennie można skorzystać z metody String.Format albo klasy StringBuilder, ale powyższa interpolacja zdecydowanie spełnia swoją rolę w tym przypadku IMHO jest wygodniejszym podejściem do łączenia ciągów znaków.

Tworzenie obiektów pozostaw funkcji głównej programu (a teraz właściwie to plikowi Program.cs). Należy pamiętać, że z racji walidacji, warto byłoby skorzystać z konstrukcji try ... catch, jeżeli numer PESEL będzie niepoprawny:

[...]
Uczen uczen = default;
try
{
    uczen = new(imie, nazwisko, pesel);
}
catch (ArgumentException ex)
{
    Console.WriteLine($"Wystąpił problem z utworzeniem obiektu przechowującego informacje o uczniu:{Environment.NewLine}{ex.Message}");
}

---

EDIT #1: Poprawiono znak cudzysłowia dla wycinka logiki głównej.
EDIT #2: Zmodyfikowano informację o konkatenacji, zgodnie z dokumentacją metody String.Concat.

Comments

Zdecydowanie potężnie przekombinowałeś jak dla osoby która prawdopodobnie pisze swoją pierwszą klasę :)

(Imie, Nazwisko, Pesel) = (imie, nazwisko, pesel);

Nie jestem świadom by ktokolwiek tak idiomatycznie pisał.

(w .NET ciąg znaków jest niezmienny, a to oznacza, że przy każdej konkatenacji ciągów tworzony jest nowy, a nie dołączany do istniejącego):

W praktyce kompilator optymalizuje łączenie stringów `a + b + c + d + c + b + a` do jednego String.Concat. Dokumentacja też o tym wspomina:

Even if you use several + operators in one statement, the string content is copied only once

Więc mimo że interpolacja w ogólności jest fajna, to tutaj aż tak wielkiej różnicy nie robi.