Per me va bene, quello che ha usato è un modo diverso di scrivere una singleton, senza inserire un riferimento d'istanza, che è utile in quei casi in cui sia necessario creare un'istanza e richiamare metodi non statici.
Io le scrivo in entrambi i modi a seconda di quello che devo fare, in questo caso trovo che vada bene scriverla così.

[edit] Nello stesso post che hai linkato ci sono una serie di spunti tra i commenti che dibattono su quale sia il metodo migliore di scrivere una singleton o una classe a metodi statici, il che dimostra quanto siano molteplici i modi di vedere e gestire questo tipo di classi.