Конвертировать строку в объект?

Question

[juliion]

Есть строка, которая содержит информацию про транзакцию.
(Пример "John, Doe, “Lviv, Kleparivska 35, 4”, 500.0, 2022-27-01, 1234567, Water").
Нужно конвертировать эту строку в объект Transaction.

Код класса Transaction:

public class Transaction
    {
        public string Name { get; set; }
        public string City { get; set; }
        public decimal Payment { get; set; }
        public DateTime Date { get; set; }
        public long AccountNumber { get; set; }
        public string Service { get; set; }
    }

Вот мое решение:

public enum FieldsIndices
    {
        nameInd = 0,
        surnameInd = 1,
        cityInd = 2,
        payentInd = 6,
        dateInd = 7,
        accountNumInd = 8,
        serviceInd = 9
    }
    public class TransactionConverter
    {
        public Transaction StrToTransaction(string strTransaction)
        {
            string[] fields = strTransaction.Split(new char[] { ' ', ',', '“', '”' }, StringSplitOptions.RemoveEmptyEntries);
            Transaction transaction = new Transaction()
            {
                Name = fields[(int)FieldsIndices.nameInd] + " " + fields[(int)FieldsIndices.surnameInd],
                City = fields[(int)FieldsIndices.cityInd],
                Payment = decimal.Parse(fields[(int)FieldsIndices.payentInd], CultureInfo.InvariantCulture),
                Date = DateTime.ParseExact(fields[(int)FieldsIndices.dateInd], "yyyy-mm-dd", CultureInfo.InvariantCulture),
                AccountNumber = long.Parse(fields[(int)FieldsIndices.accountNumInd]),
                Service = fields[(int)FieldsIndices.serviceInd]
            };
            return transaction;
        }
    }

Вопрос: Как это можно сделать проще или оптимизирование? Код получился запутанным.

Comments

[Василий Банников]

1. Можно хотябы вместо enum использовать нормальные константы
2. Вместо Split надо использовать что-то другое, чтобы нормально обрабатывать кавычки

[Василий Банников]

juliion, а в файле точно именно такие, "перевёрнутые" кавычки используются?

[juliion]

Василий Банников, Да

[Василий Банников]

juliion, ужас какой.

[Popou]

Почему бы просто не использовать сериализацию?

[freeExec]

Нельзя просто так взять и изменить формат входных данных.

[juliion]

Строка по заданию считывается из txt файла.

[Popou]

juliion, а понятно

[Василий Банников]

Это лучше в комментарии

Answer 1

[Алексей]

У вас же формат строки файла CSV.
Пробуйте общедоступные библиотеки для этого. Например вот такую:
https://joshclose.github.io/CsvHelper/getting-started/

Answer 2

[freeExec]

Можете ещё создать список с названием полей по порядку, и заполнять через рефлекшн. Но походу это не то, что вам нужно, раз вы в своём коде заплутали.

Answer 3

[Nik Faraday]

Используйте сериализацию JSON

Answer 4

[Никита Гусев]

можно сделать более оптимизированно, если не аллоцировать для полей новые строки, а использовать IReadOnlySpan. Вот минимальный пример:

var input = "John Stockman|[phone]";
var record = new Record(input);

Console.WriteLine($"Name: {record.Name} Phone: {record.Phone}");

class Record
{
    private string line;
    private int delimeterIndex;
    public Record(string line)
    {
        this.line = line;
        delimeterIndex = line.IndexOf('|');
    }
    public ReadOnlySpan<char> Name => line.AsSpan(0, delimeterIndex);
    public ReadOnlySpan<char> Phone => line.AsSpan(delimeterIndex + 1, line.Length - 1 - delimeterIndex);
    public override string ToString() => line;
}