Na verdade o código tem vários problemas, inclusive conceituais, mas para um exercício muito básico só de mecanismos a maioria não é um grade problema. Também tem problemas de organização e nomenclatura de nomes que não segue o padrão do C#. Não dá erro mas acostume-se fazer de uma forma melhor. Note que eu simplifiquei bastante o código, faz tempo que não precisa escrever o código de forma tão complexa. Podia ter simplificado mais. E dei nomes mais corretos para o que o método faz, assim o código fica mais legível.
Uma coisa que eu não fiz foi validar a entrada de dados, o correto é validar. Por exemplo na hora de converter para número pode dar erro, não deveria deixar isto ocorrer, então deveria usar o TryParse()
e não o Parse()
que só deve ser usado em locais onde você pode garantir que o dado está correto, o que não é o caso de uma entrada de dados.
Também não me preocupei se a classe deveria ter um construtor, talvez.
Também não entrei no mérito se devia usar um decimal
para notas, em muitos casos não precisa, mas exatidão pode ser um requisito.
O problema central é que está chamando um método que exige um parâmetro sem passar um argumento. Mas tem um outro conceito errado ali. O certo é fazer isso em cima da média já existente e decidir se a pessoa passou ou quanto falta de nota. Pelo menos é o que o código dá a entender que deve fazer.
Enfim, resolvi o problema, mas pode ensinar errado se achar que está bom para usar em aplicações reais.
using static System.Console;
public class Program {
public static void Main() {
var estudante = new Nota {
Matricula = ReadLine(),
Idade = int.Parse(ReadLine()),
Prova1 = double.Parse(ReadLine()),
Prova2 = double.Parse(ReadLine()),
Trabalho = double.Parse(ReadLine())
};
estudante.MostraDadosDeNota();
}
}
public class Nota {
public string Matricula { get; set; }
public string Nome { get; set; }
public int Idade { get; set; }
public double Prova1 { get; set; }
public double Prova2 { get; set; }
public double Trabalho { get; set; }
public double Media { get => (Prova1 * 2.5 + Prova2 * 2.5 + Trabalho * 1.5) / 6.5; }
public void MostrarMedia() => WriteLine($"Sua media foi igual a: {Media:0.00}");
public void MostrarNotaFaltante() {
if (Media >= 6.0) WriteLine("Passou");
else WriteLine($"Faltou {6.0 - Media:0.00} pontos para passar");
}
public void MostrarMelhorNota() {
double nota = Prova1;
if (Prova2 > nota) nota = Prova2;
if (Trabalho > nota) nota = Trabalho;
WriteLine($"Sua maior nota foi: {nota:0.00} pontos");
}
public void MostraDadosDeNota() {
MostrarMedia();
MostrarNotaFaltante();
MostrarMelhorNota();
}
}
Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.
public string falta(double mfinal)
não retorna umastring
. se não retorna nada, então mude paravoid
. 2- No métodopublic void print()
a chamada para o métodofalta()
não está passando o parâmetro. Enfim... o problema está na base...!!!