O conceito depende da classe, então só dá para afirmar com um caso concreto. O exemplo quis só demonstrar, mas justamente por ser artificial impede de dar uma resposta definitiva. Esse caso parece que não deve fazer isso, mas porque sequer faz sentido ter esse método como estático (até poderia, não sei o caso concreto, então estou especulando isso com base no meu conhecimento prévio sobre o assunto).
Do ponto de vista técnico tudo que dá para fazer é correto, desde que faça sentido naquele momento.
Se quer saber de boas práticas, cai de novo no primeiro parágrafo, só é boa prática se conhecer o caso concreto. O que é bom para uma coisa pode não ser para outra, e se a pessoa tenta universalizar ela acaba cometendo erros tentando acertar já que não dá para aplicar a mesma ação para todos os casos.
Se você quer saber se costuma dar algum problema? Normalmente não, mas pode criar dificuldades.
Algumas pessoas trabalham com algumas teorias. Algumas dizem que nem deveria existir método estático para começo de conversa. Eu acho que não. Veja mais em Qual a função de um método estático?. Se você quer seguir estritamente a orientação a objeto, como alguns pregam (não sei qual o objetivo de fazer isso), nem crie este tipo de método.
Precisa entender para que ele serve para saber usar bem. Se é o que precisa então está ok, se faz sentido o método existir para o tipo e não para o objeto, ele deve ser estático.
Ter um código utilitário que serve para outro método delegar a execução para ele e assim evitar duplicação de código é talvez o maior motivo de existir um método estático.
Em alguns casos deveria usar um método privado e não estático para isso, depende da necessidade. Como Python não tem métodos privados de verdade isso é feito só por convenção colocando um __
antes do nome.
O que de fato faz o método ser estático em Python é não usar um parâmetro self
. O decorador permite que este método seja chamado de fora da classe usando o nome da classe para acessar, nem sempre quer isso, pode ser seu caso ou não.