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; }
Coloquei no GitHub para referência futura.
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, quando se sabe o que está fazendo.