Skip to main content
adicionou 138 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

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:

Se você não usaro value de alguma forma isto 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;

Desconfie de alguém pedir para não usar a auto implementada quando ela é a adequada, isto não ensina nada. Usar uma implementada onde precisa ensina corretamente a mesma coisa que fazer manual o que pode ser auto implementada.

Veja mais:

Se você não usar o value de alguma forma isso 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, isso ensina nada. Usar uma implementada onde precisa ensina corretamente a mesma coisa que fazer manual o que pode ser auto implementada.

Veja mais:

adicionou 92 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Estou respondendo porque a resposta aceita está errada, pelo menos a primeira parte. Ela manda fazer algo que não se deve fazer com um setter. Se você não está usando ousaro value de alguma forma isto 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 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 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;

Desconfie de alguém pedir para não usar a auto implementada quando ela é a adequada, isto não ensina nada. Usar uma implementada onde precisa ensina corretamente a mesma coisa que fazer manual o que pode ser auto implementada.

Veja mais:

Estou respondendo porque a resposta aceita está errada, pelo menos a primeira parte. Ela manda fazer algo que não se deve fazer com um setter. Se você não está usando o value de alguma forma isto 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;

Somente se não tiver restrição que pode fazer assim (mas sem restrição parece errado):

public DateTime OrderDateDeli { get; set; } = DateTime.Today;

Veja mais:

Se você não usaro value de alguma forma isto 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;

Desconfie de alguém pedir para não usar a auto implementada quando ela é a adequada, isto não ensina nada. Usar uma implementada onde precisa ensina corretamente a mesma coisa que fazer manual o que pode ser auto implementada.

Veja mais:

adicionou 4 caracteres ao conteúdo
Fonte Link
Pedro Gaspar
  • 3,4mil
  • 1
  • 16
  • 36

Estou respondendo porque a resposta aceita está errada, pelo menos a primeira parte. Ela manda fazer algo que não se deve fazer com um setter. Se você não está usando o value de alguma forma isto não é um setter.

Na verdade o código original está cheio de problemas conceituais.

Nem vou cometarcomentar 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 deveriadeveriam 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 stterssetters 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-implementadas. Veja mais abaixo links sobre o assunto.

A forma como está construindo o texto é bastante inadequadoinadequada. 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-implementada. É só fazer orderDateDeli = DateTime.Today;

Somente se não tiver restrição que pode fazer assim (mas sem restrição parece errado):

public DateTime OrderDateDeli { get; set; } = DateTime.Today;

Veja mais:

Estou respondendo porque a resposta aceita está errada, pelo menos a primeira parte. Ela manda fazer algo que não se deve fazer com um setter. Se você não está usando o value de alguma forma isto não é um setter.

Na verdade o código original está cheio de problemas conceituais.

Nem vou cometar 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 deveria 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 stters 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 inadequado. 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;

Somente se não tiver restrição que pode fazer assim (mas sem restrição parece errado):

public DateTime OrderDateDeli { get; set; } = DateTime.Today;

Veja mais:

Estou respondendo porque a resposta aceita está errada, pelo menos a primeira parte. Ela manda fazer algo que não se deve fazer com um setter. Se você não está usando o value de alguma forma isto 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;

Somente se não tiver restrição que pode fazer assim (mas sem restrição parece errado):

public DateTime OrderDateDeli { get; set; } = DateTime.Today;

Veja mais:

Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando