Não diria que é uma "boa prática" ou "má prática" usar uma classe cheia de métodos e propriedades estática, mas diria que isso é um malmau uso do recurso.
Vamos pensar, realmente é necessário criar uma classe, chegacheia de métodos estáticos, só pra fazer dela um "repositório de funções"?
Eu amo usar o Framework Laravel, mas não preciso concordar com tudo que está lá. Apesar de usar essa classe em algumas partes dos sistemas que desenvolvi, percebo que estruturar uma classe todotoda estática apenas para trabalhar com arrays
é fazer malmau uso dos método estáticos.
Se você observar bem, poderia ser feitofeita uma função para cada método dessa classe, ao invés de criar uma classe só pra isso.
O que eu percebo no final de contas é que muita gente faz coisas do tipo (não só encher uma classe de métodos estáticoestáticos, mas outras coisas) porque não tem ideia do que está fazendo.
Lembrando que os método estáticos não se limitalimitam a fazer o que está sendo feito acima, mas no meu ponto de vista, numa estrutura OOP faz mais sentido você usar um método estático para auxiliar nas operações internas da própria classe do que encher uma classe dele só pra ter uma "organização".