Prośba o sprawdzenie poprawności mojego kodu

Question

Cześć, napisałem kod do pewnego zadania i prosiłbym Was o sprawdzenie, czy o to chodzi :)

Dodam, że kod się kompiluje i działa.

Treść zadania:

Napisać program odwracający kolejność łańcuchów tekstu podawanych z wejścia. Program ma zapamiętywać wprowadzane
dane wykorzystując stos implementowany jako listę. Po wprowadzeniu łańcucha pustego ma zostać wyświetlona
odpowiedź. Obsłużyć wyjątki.
Do przechowywania danych na liście należy wykorzystać strukturę:

class Element
{
string Klucz;
public Element next;
public Element(string Klucz)
{
this.Klucz = Klucz;
}
}

Kod:

using System;
using System.Collections.Generic;
using System.Text;

namespace Zad._20
{
    class Element
    {
        public String wartosc;
        public Element poprzedni;

        public Element(String klucz) {
            this.wartosc = klucz;
        }
    }
}

using System;
using System.Collections.Generic;
using System.Text;

namespace Zad._20
{
    class Stos
    {
        private Element wierzcholek;

        public Stos()  {
            wierzcholek = null;
        }

        public void Push(Element e)  {
            e.poprzedni = wierzcholek;
            wierzcholek = e;
        }

        public void Pop()  {
            Element tmp = wierzcholek;
            wierzcholek = wierzcholek.poprzedni;
        }

        public void Print()    {
            while(wierzcholek != null) {
                Console.WriteLine(wierzcholek.wartosc);
                wierzcholek = wierzcholek.poprzedni;
            }  
        }
    }
}

 

Comments

Gdzie wywołujesz Push i Pop? 

public Stos()  {
            wierzcholek = null;
        }

Lepiej zrobić konstruktor pusty:

public Stos()  {
            
        }

kompilator i tak przypisze wartości domyśle.

Pętla while ma stały warunek, więc będzie wykonywać się bez przerwy.

---

W mainie wywołuję.

Pętla nie wykonuje się bez przerwy, w konsoli mi wszystko dobrze wyświetla.

---

To może tak wstaw cały kod? Nie widzę abyś np.

Po wprowadzeniu łańcucha pustego ma zostać wyświetlona
odpowiedź. 

albo:

 Obsłużyć wyjątki.

Tak więc, bez pełnego kodu ciężko coś powiedzieć.