Думаю нет ничего страшного в том, что я оставил бы такой код и не пытался бы его оптимизировать, функция работает и она не особо длинная; но все же на будущее я хотел бы узнать от более опытных программистов как я мог бы оптимизировать её, чтобы было, возможно, меньше ненужного кода?
Итак, суть функции - спарсить входную строчку с ходом, записанным с помощью шахматной нотации, в строчку, понятную обычному человеку (например - "Ng1-f3" => "Конь g1 на f3" или "Pa7-a8Q" => "Пешка a7 на a8 и превращается в ферзя").
Код:
public static string MoveParser(string move)
{
if (move == "0-0") return "Короткая рокировка";
if (move == "0-0-0") return "Длинная рокировка";
Dictionary<char, string> figures = new Dictionary<char, string>
{
{ 'P', "Пешка"},
{ 'R', "Ладья"},
{ 'N', "Конь"},
{ 'B', "Слон"},
{ 'Q', "Ферзь"},
{ 'K', "Король"}
};
var figure = figures[move[0]];
var startCell = move[1].ToString() + move[2].ToString();
var endCell = move[4].ToString() + move[5].ToString();
var pawnToFigure = "";
if (move.Length == 7)
pawnToFigure = " и превращается в " + figures[move[6]];
return figure + startCell + " на " + endCell + pawnToFigure;
}