Tak jenom namátkou:
Metoda přijímá jako vstup string, který pak vůbec nepoužívá.
Buď tam měj property nebo metodu. Pokud tam máš obojí a nepoužívá se to na nic jiného, tak zbytečně zesložiťuješ kód.
A poměrně klasická věc: mícháš do sebe data a uživatelské rozhraní.