Diskuze – Lekce 7 - Upomínač narozenin v C# .NET WPF - Logická vrstva

Díky za odpověď. Kontroloval jsem tu Arénu a opravdu tam jsou všechny třídy internal, tak tam asi bude nějaká spojitost s vlastnostmi, jak píše David, jelikož atributy v Aréně fungovaly dobře..

Nalezena chyba: v kodu ZbyvaDni je zbytečně vytvořená instance: 'DateTime dnes = DateTime.Today;'

Původní:

public int ZbyvaDni
{
        get
        {
                DateTime dnes = DateTime.Today;
                DateTime dalsiNarozeniny = Narozeniny.AddYears(Vek + 1);

                TimeSpan rozdil = dalsiNarozeniny - DateTime.Today;

                return Convert.ToInt32(rozdil.TotalDays);
        }
}

Opravené:

public int ZbyvaDni
{
        get
        {
                DateTime dalsiNarozeniny = Narozeniny.AddYears(Vek + 1);

                TimeSpan rozdil = dalsiNarozeniny - DateTime.Today;

                return Convert.ToInt32(rozdil.TotalDays);
        }
}

Zbytečně vytvořená sice je, ale že by to byla nějaká ukrutná chyba, se mi nezdá...
stačí jen opravit toto

TimeSpan rozdil = dalsiNarozeniny - DateTime.Today;

na toto

TimeSpan rozdil = dalsiNarozeniny - dnes;

Ahoj, dnes když jsem napsal ObservableCollec­tion ve Visual Studiu, tak to tento typ kolekce neznalo. Pokud by jste měli podobný problém stačí nahoře napsat. using System.Collec­tions.ObjectMo­del.

Díky, měl jsem podobný problém, ušetřilo mi to nějaký čas.

Stačí tam tu "kolekci" napsat a po najetí myši na červeně podtržené slovo se objeví možnosti řešení, mezi kterými je i přidání potřebných usingů...

zápis vlastnosti DnesniDatum

public DateTime DnesniDatum
{
        get
        {
                return DateTime.Now;
        }
}

sa dá skrátene zapísať aj takto:

public DateTime DnesniDatum { get; } = DateTime.Now;

To len pre zaujímavosť.
Originálny zápis je síce rozsiahlejší, ale zato edukatívnejší

To je samozřejmě pravda, ale tento článek vznikl v době, kdy takový zápis ještě nebyl možný (myslím, že je to možné až od verze C# 6) a hlavně bych řekl, že pro začátečníka je ten "delší" zápis lépe srozumitelný...

Tyhle kódy bohužel nedělají totéž. Když pustíš tenhle kód:

 class A
    {
        public DateTime X { get; } = DateTime.Now;

        public DateTime Y
        {
            get { return DateTime.Now; }
        }
    }

// ....

A a = new A();

Console.WriteLine(a.X == a.X);
Console.WriteLine(a.Y == a.Y);

tak (i přes upozornění od Rideru, který tvrdí že se dají oba výrazy nahradit za true...) dostaneš na výstupu True a potom False.

Důvodem je to že vlastnost X se neinicializuje při vytvoření instance A, tudíž pro všechny dotazy má stejnou hodnotu. Ale zato vlastnost Y se při každém dotazu na ní koukne co v sobě zrovna má vlastnost DateTime.Now. Y by se dal zkrátit na:

public DateTime Y => DateTime.Now;

což jsi možná měl na mysli...

Ďakujem za upozornenie.
Máš pravdu. Príkazy, ktoré som považoval za totožné, majú rozdielný význam.
Lepšie je to vidieť, keď namiesto tvojho porovnania:

A a = new A();

Console.WriteLine(a.X == a.X);
Console.WriteLine(a.Y == a.Y);

ich 2x vypíšeš a medzi výpismi uspíš vlákno:

A a = new A();

Console.WriteLine(a.X);
Console.WriteLine(a.Y);

Thread.Sleep(3000);

Console.WriteLine(a.X);
Console.WriteLine(a.Y);

hodnota X sa nezmení a hodnota Y sa vyhodnocuje pri každom zavolaní