Tem vários problemas no código, alguns apenas de estilo. O maior é que está tentando converter algo que não é garantido que pode ser convertido e isto quebrará a aplicação se a digitação for errada. O certo é verificar antes de usar. EU apesar encerrei a aplicação para simplificar, mas você pode emitir uma mensagem de erro e pedir novamente a digitação até que se entrado um valor válido. Em código real as pessoas criam códigos que fazem isso em cada item e colocam em função para uso fácil.
Note que mudei várias coisas no código, como a forma de imprimir, os nomes das coisas, e o uso do foreach
no lugar do for
. Ele deve ser a preferência, até que só o for
resolva.
Você estava mandando imprimir a lista. Ela não tem um meio de impressão automático dela que sai tudo como imagina. Tem infinitas formas de imprimir um lista, você tem que montar isso item por item. E mesmo dentro do item também tem muitas formas de imprimir cada membro do item da lista, então tem que lidar com isso. Até a lista estava pegando cada item, mas não cada membro.
using static System.Console;
using System.Collections.Generic;
public class Program {
public static void Main() {
Write("Quantos funcionários serão cadastrados? ");
if (!int.TryParse(ReadLine(), out var quantidade)) return;
var lista = new List<Funcionario>();
for (int i = 0; i < quantidade; i++) {
WriteLine($"Dados do {(i + 1)}º funcionário: ");
Write("CPF: ");
if (!int.TryParse(ReadLine(), out var cpf)) return;
Write("Nome: ");
var nome = ReadLine();
Write("Salário: ");
if (!double.TryParse(ReadLine(), out var salario)) return;
lista.Add(new Funcionario(cpf, nome, salario));
WriteLine();
}
foreach (var item in lista) WriteLine($"CPF: {item.Cpf} - Nome: {item.Nome} - Salário: {item.Salario}");
}
}
public class Funcionario {
public int Cpf;
public string Nome;
public double Salario { get; private set; }
public Funcionario (int cpf, string nome, double salario) {
Cpf = cpf;
Nome = nome;
Salario = salario;
}
}