Se você não usarousar o value
de alguma forma istoisso não é um setter.
Na verdade o código original está cheio de problemas conceituais.
Nem vou comentar o nome do namespace SergioS_301011342_LIB
, ele só me dá medo :)
Me parece que o construtor está sendo usado de forma equivocada. Em primeiro lugar o que está ali deveriam ser propriedades inicializadas, mas o mais provável é que deveria realmente ter um construtor com os dados obrigatórios do objeto. Desta forma o objeto nem faz sentido existir porque ele pode ser criado de forma inválida. Parece que vale para todos os construtores.
E está permitindo inicialização de coisas que são muito fáceis de cometer erros, deveria ter um outro controle sobre a inicialização de quase tudo, especialmente estes textos soltos, isto não pode estar certo.
Parece que os getters e setters existem porque foi seguida receita de bolo. Já se questionou se precisa mesmo? Se deve ser assim? Pesquise aqui no site, tem muita informação sobre o abuso deles.
Pegar OrderItens
parece vazamento de abstração.
Idealmente deveria usar só as propriedades auto implementadas. Veja mais abaixo links sobre o assunto.
A forma como está construindo o texto é bastante inadequada. Veja O que significa o sinal de arroba "@" no C#? e O que significa o simbolo "$" antes de uma string?, e principalmente Por que devo usar a classe StringBuilder ao invés da classe String?.
Aí você tem que se perguntar, o pedido quando criado pode ter sua data mudada? Se não puder, deve fazer algo assim:
public DateTime OrderDateDeli { get; } = DateTime.Today;
Se puder ser mudado, isto pode ser feito de forma indiscriminada? Provavelmente não, aí seria algo assim:
public DateTime OrderDateDeli {
get => orderDateDeli;
set {
//faz aqui o tratamento necessário com o value para poder usar corretamente
}
}
E aí agora você precisa fazer a inicialização no construtor, não tem jeito, o facilitador só existe quando a propriedade é auto implementada. É só fazer orderDateDeli = DateTime.Today;
no construtor.
Somente se não tiver restrição que pode fazer assim (mas sem restrição parece errado):
public DateTime OrderDateDeli { get; set; } = DateTime.Today;
Coloquei no GitHub para referência futura.
Desconfie de alguém pedir para não usar a auto implementada quando ela é a adequada, isto nãoisso ensina nada. Usar uma implementada onde precisa ensina corretamente a mesma coisa que fazer manual o que pode ser auto implementada.
Veja mais:
- Métodos e propriedades em C# - vantagens e desvantagens
- Propriedade Vs variáveis
- Como se utilizar propriedade expandida no C#
- Campo público X propriedade
- Sintaxe de inicialização de propriedade estranha pra mim
- Como funcionam as propriedades no C#?
- Nova funcionalidade do C# 6 "Auto-property initializers" é apenas um facilitador?