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

    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 já que a propriedade deixa de ser automática.

Se o campo, e não a propriedade, for necessário em outros pontos da classe, então precisa declarar fora. Mas isto 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