O design está ruim, está misturando coisas. Não sei se entendi o que realmente deseja. Eu vou mudar para uma forma que está correta. Ainda não ficará tão correto porque as classes misturam coisas que deveriam estar em lugares separados. Entendo que seja só um exercício artificial, mas aprender errado com ele não vai ajudar.
A moto não deveria ter conhecimento sobre a pessoa que a detém. Se por acaso isto for necessário, então o design teria que ser mudado mais ainda. E teria que ter uma boa justificativa. Então fazendo que a moto manipule só dados da moto e o cliente só dados do cliente (que tem como parte integrante a pessoa), fica tudo certo.
Tem que entender o problema como ele realmente é. Querer criar uma situação artificial vai criar problemas mais cedo ou mais tarde. Claro que em um exercício não vai criar problemas, afinal ele não receberá manutenção, não terá requisitos alterados. Mas se fizer errado no exercício, aprenderá fazer errado depois.
No exemplo atual não existe motivo para ter um método virtual. Não é o caso de usar polimorfismo aqui.
Talvez fosse o caso de usar um construtor e não este método Add()
. Principalmente se tirar a lógica de entrada e saída de dados que não deveria estar junto dos dados. Pra consertar as classes teria que refazer de forma tão radical que se tornaria outra coisa completamente diferente.
Também é muito estranho ter um método Add()
em duas classes e um AddMoto()
na outra, não faz sentido.
Tem outros problemas no código. Não vou tentar resolver tudo.
using static System.Console;
public class Program {
public static void Main() => new Cliente().Add();
}
public class Pessoa {
public string Nome { get; set; }
public int Idade { get; set; }
public virtual void Add() {
Nome = ReadLine();
Idade = int.Parse(ReadLine()); //não é assim que faz, só pra simplificar
}
}
public class Cliente : Pessoa {
public int Codigo { get; set; }
public override void Add() {
base.Add();
Codigo = int.Parse(ReadLine());
var moto = new Moto();
moto.AddMoto();
moto.ExibirDados();
ExibirDados();
}
public void ExibirDados() => WriteLine("Titular: " + Nome);
}
public class Moto {
public string Cor { get; set; }
public void AddMoto() => Cor = ReadLine();
public void ExibirDados() => WriteLine("Cor: " + Cor);
}
Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.
Eu fico imaginando porque uma Pessoa
e não um Cliente
pode ser dono de uma Moto
neste caso. Mesmo que isto seja possível, então Cliente
é uma coisa e Pessoa
é outra. Aí o Cliente
pode ser até composto por uma Pessoa
, mas não ser a Pessoa
. Ou seja, o design estaria ainda mais errado. Considerando esta hipótese e considerando que os dados da pessoa e moto podem ser nulos (outro código vai cuidar disto) fiz um código melhor mas ainda com problemas. Algo assim:
using static System.Console;
public class Program {
public static void Main() {
var pessoa = new Pessoa();
var moto = new Moto(pessoa);
var cliente = new Cliente(moto);
cliente.ExibirDados();
cliente.Moto.ExibirDados();
cliente.Pessoa.ExibirDados();
}
}
public class Pessoa {
public string Nome { get; set; }
public int Idade { get; set; }
public Pessoa() {
Write("Nome: ");
Nome = ReadLine();
Write("Idade: ");
Idade = int.Parse(ReadLine()); //não é assim que faz, só pra simplificar
}
public void ExibirDados() => WriteLine($"Nome: {Nome}");
}
public class Cliente {
public int Codigo { get; set; }
public Pessoa Pessoa { get; set; }
public Moto Moto { get; set; }
public Cliente() {
Write("Código do cliente: ");
Codigo = int.Parse(ReadLine());
}
public Cliente(Pessoa pessoa) : this() => Pessoa = pessoa;
public Cliente(Moto moto) : this(moto.Pessoa) => Moto = moto;
public void ExibirDados() => WriteLine($"Código: {Codigo}");
}
public class Moto {
public string Cor { get; set; }
public Pessoa Pessoa { get; set; }
public Moto() {
Write("Cor: ");
Cor = ReadLine();
}
public Moto(Pessoa pessoa) : this() => Pessoa = pessoa;
public void ExibirDados() => WriteLine($"Cor: {Cor}");
}
Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.
Note que esta referência cruzada do Cliente
que tem uma Pessoa
X Moto
que também tem uma Pessoa
não me agrada. Mas aí vou ter que ir mais longe.
Não sou fã do termo atributo aí, prefiro campo.
Pessoa.Nome
Moto
deve ter uma propriedade chamadaPessoa
? Não consigo imaginar uma explicação plausível. Então o problema parece estar aí. Se deixar de ter isto, começa fazer mais sentido. Ainda nem tanto. Misturar a lógica de entrada e saída de dados está torando tudo muito confuso. Tente explicar melhor o que realmente deseja.Add()
.