Não está declarando propriedade duas vezes, etá declarando um campo e uma propriedade que usa este campo. Em algo simples assim não é necessário fazer desta forma:

    public int Years { get; }

O campo é automaticamente declarado internamente (não fica disponível para seu código, nem dá para saber o nome que usado salvo com reflexão, o que não faz sentido fazer.

Quando precisa fazer algum algoritmo dentro da propriedade aí precisa declarar o campo explicitamente.

Se o campo mesmo for necessário em outros pontos da classe, então precisa declarar fora. Mas isso raramente é necessário.

Nem sempre precisamos de propriedades. Tem até uma escola que é contra elas já que ela não tem uma função definida. Eu acho academicismo.

Tem casos que um [campo público pode ser útil ou até melhor][1], quando se sabe o que está fazendo.


  [1]: https://pt.stackoverflow.com/q/133924/101