C# инициализация структуры с неполными параметрами + формула

Question

Я уверен, что это просто, но я слишком долго смотрел на нее, и мне нужен ответ в ближайшее время. Я новичок в C#. Если я ставлю GetCommission() в структуры я получаю

ошибка CS0188: «это» объект не может быть использован, прежде чем все ее поля присваиваются

вне структуры

ошибка CS0038: не удается получить доступ к нестатический члену внешнего типа «Ex5._3.CommissionForm» с помощью вложенного типа «» Ex5._3.CommissionForm.salespersonFigures

Как это сделать? Оговорка: часть задания заключалась в том, что комиссия рассчитывается по методу. Ни один из обучающих программ, которые я нашел, не связан с назначением одного члена на основе значения другого. Он должен быть кошерным, поскольку вычисления используют только статические данные. Правильно?

// Declare class variables and constants 
private const decimal WEEKLY_BASE_SALARY = 250m; 
private const decimal WEEKLY_QUOTA = 1000m; 
private const decimal COMMISSION_RATE = .15m; 

public struct salespersonFigures 
{ 
    // Fields 
    private string salesperson; 
    private decimal weeklySales; 
    private decimal commission; 
    private decimal pay; 

    // Constructor 
    public salespersonFigures(string name, decimal sales) 
    { 
     salesperson = name; 
     weeklySales = sales; 
     commission = GetCommission(sales); // error occurs at this line 
     pay = WEEKLY_BASE_SALARY + commission; 
    } 

    // Property 

    public decimal Sales 
    { 
     get 
     { 
      return weeklySales; 
     } 
     set 
     { 
      weeklySales = value; 
     } 
    } 

    public string Name 
    { 
     get 
     { 
      return salesperson; 
     } 
     set 
     { 
      salesperson = value; 
     } 
    } 

    // Method 

    public decimal GetCommission(decimal sales) 
    { 
     if (sales > WEEKLY_QUOTA) 
      return sales * COMMISSION_RATE; 
     else return 0m; 
    } 
} 

Answer 1

Конструктор:

// Constructor 
public salespersonFigures(string name, decimal sales) 
{ 
    salesperson = name; 
    weeklySales = sales; 
    commission = GetCommission(sales); 
    pay = WEEKLY_BASE_SALARY + commission; 
} 

интерпретируется как:

// Constructor 
public salespersonFigures(string name, decimal sales) 
{ 
    this.salesperson = name; 
    this.weeklySales = sales; 
    this.commission = this.GetCommission(sales); 
    this.pay = WEEKLY_BASE_SALARY + this.commission; 
} 

Таким образом, проблемы, которые можно было бы сталкивающиеся можно решить, выполнив две вещи:

// Constructor 
public salespersonFigures(string name, decimal sales) 
{ 
    salesperson = name; 
    weeklySales = sales; 
    var tempCommission = GetCommission(sales); 
    commission = tempCommission 
    pay = WEEKLY_BASE_SALARY + tempCommission; 
} 

И сделайте GetCommissio n метод статический

Answer 2

Действительно ли вам поле для комиссионных?

private decimal commission; 

Вы можете удалить его, и у вас есть метод в структуре. И я думаю, что вам не нужен параметр продаж в методе yout GetComission.

Answer 3

Очень простой обходной путь/хак:

public salespersonFigures(string name, decimal sales) 
{ 
     salesperson = name; 
     weeklySales = sales; 
     // initialize pay and commission 
     pay = 0m; 
     commission = 0m; 
     commission = GetCommission(sales); 
     pay = WEEKLY_BASE SALARY + commission; 
} 

И вам не нужна else часть статьи в getCommission, просто сделать это:

public decimal GetCommission(decimal sales) 
{ 
    if (sales > WEEKLY_QUOTA) 
     return sales*COMMISSION_RATE; 
    return 0m; 
} 

Еще одна вещь, почему вы используете decimal? Вы, вероятно, следует использовать double «с или float» S вместо этого, потому что все больше полагаются на методы float х и doubles-х и преобразования между может быть дорогостоящим

Answer 4

Почему бы не просто сделать GetCommission() в собственность: Комиссия.

Там нет необходимости хранить комиссию в качестве поля, он должен быть запрос к SalespersonFigures объекта