Блог

У меня есть два правила, которые я хочу реализовать в XML-приложении.

1 Если вес больше 10, то по названию Отдела должно быть написано: Большой

Лучше всего позволить самому Department свойству вычислить его значение (вычисляемое свойство). Это чище и удобочитаемее, так как вычисления выполняются там, где вы этого ожидаете. В противном случае в сценарии, где вычисления зависят более чем от одного свойства, вы обнаружите, что пишете дублированный вычислительный код в каждом участвующем свойстве.
Поэтому выполняйте вычисление при запросе значения, то есть при вызове вычисляемого свойства Get() , а не при изменении участвующих свойств. Если вычисления более сложные, то следование этим правилам также повысит производительность.

Также избегайте реализации такого фильтра непосредственно в Set() / Get() свойства. Переместите его в метод.

Я также предлагаю использовать оператор переключения или выражение переключения, поскольку это более читабельно, чем длинная цепочка блоков if-else, и, следовательно, проще в обслуживании. При использовании C# 9 выражение переключения может быть мощным инструментом фильтрации с хорошо читаемым синтаксисом.

Кроме того, поскольку строка, которую вы пытаетесь создать для печати, является фиксированным string представлением фактического Parcel экземпляра, было бы более уместно переопределить Parcel.ToString .

Ваш класс посылок должен выглядеть примерно так:

Посылка.cs

 public class Parcel
{
  public override string ToString() 
    => $"Name: {this.Name} - Postal code {this.PostalCode} - Weight {this.Weight} - Value {this.Value} - Department {this.Department}";

  public string Name { get; set; }
  public string PostalCode { get; set; }
  public decimal Weight { get; set; }
  public decimal Value { get; set; }
  public string Department => CreateDepartmentValue();

  private string CreateDepartmentValue()
  {
    var result = string.Empty;

    switch (this.Weight)
    {
      case decimal weightInKiloGrams when weightInKiloGrams <= 1: result = "Mail"; break;
      case decimal weightInKiloGrams when weightInKiloGrams <= 10: result = "Regular"; break;
      case decimal weightInKiloGrams when weightInKiloGrams > 10: result = "Heavy"; break;
    };


    switch (this.Value)
    {
      case decimal value when value > 1000: result  = ", Insurance"; break;
    };

    return resu<
  }
}
 

Пример Использования

 public class Program
{
  static void Main(string[] args)
  {

    XDocument xdoc = XDocument.Load($"Container.xml");
    var items = ...;

    foreach (var item in items)
    {
      // Call ToString() implicitly
      Console.WriteLine(item);
      Console.WriteLine("*********************************************************************");
    }

    Console.ReadLine();
  }
}
 

Вы можете создать посылку класса и определить отдел на основе веса и значения из вызова функции

 public class Parcel
{
    public string Name { get; set; }
    public string PostalCode { get; set; }
    public decimal Weight { get; set; }
    public decimal Value { get; set; }
    public string Department => GetDepartment();
    private string GetDepartment()
    {
        string _department = "";
        if (this.Weight <= 1)
        {
            _department = "Mail";
        }
        else if (this.Weight > 1 amp;amp; this.Weight <= 10)
        {
            _department = "Regular";
        }
        else if (this.Weight > 10)
        {
            _department = "Heavy";
        }
        else
        {
            _department = "Unknown";
        }

        if (this.Value > 1000)
        {
            _department  = ",Insurance";
        }

        return _department;
    }
}
 

Ваш XDcoument будет выглядеть так, как показано ниже

 XDocument xdoc = XDocument.Load($"XMLFile1.xml");

var items = xdoc.Descendants("Parcel")
                .Select(xelem => new Parcel
                {
                    Name = xelem.Element("Sender").Element("Name").Value,
                    PostalCode = xelem.Element("Sender").Element("Address").Element("PostalCode").Value,
                    Weight = Convert.ToDecimal(xelem.Element("Weight").Value),
                    Value = Convert.ToDecimal(xelem.Element("Value").Value)
                });

foreach (var item in items)
{
    Console.WriteLine($"{ item.Name} - { item.PostalCode} - { item.Weight} - { item.Value} - { item.Department}");
}
 

Выход

 Klaas - 2402AE - 0.02 - 0.0 - Mail
ykken groot B.V. - 2497GA - 2.0 - 0.0 - Regular
seti - 2497GA - 100.0 - 2000.0 - Heavy,Insurance
Aad - 2353HS - 11 - 500 - Heavy
 

Почему бы не расширить класс посылок, чтобы иметь свойство «DepartmentSize», которое обрабатывает размер:

 class Parcel
{
     public string DepartmentSize 
     {
         var parts = new List<string>();
         if(_Weight> 10)
             parts.Append("Big");
         if(_Value > 1000) 
             parts.Append("Large");
         return string.Join(",", parts);
     }
 }
 

Затем вы можете вывести размер следующим образом:

 Console.WriteLine($"... Department: {item.DepartmentSize}");