Olá Emerson, tudo bem com você?
Não tem problema em fazer dessa forma. O que acontece é que, ao guardar os índices dos registros a serem removidos em uma variável, estamos apenas organizando melhor o código e tornando-o mais legível. Isso facilita o entendimento do que está sendo feito, especialmente em códigos mais complexos.
No entanto, sua sugestão de código está correta e vai funcionar da mesma maneira. Nesse caso, você está fazendo a query e removendo os registros em uma única linha. Isso pode tornar seu código mais conciso, mas pode ser um pouco mais difícil de entender para quem está lendo.
df.drop(df.query('Valor == 0 | Condominio == 0').index, axis=0, inplace=True)
Mas ambas as abordagens estão corretas e a escolha entre uma ou outra depende do seu estilo de codificação e do contexto em que você está trabalhando.
Conte com o apoio do Fórum na sua jornada. Espero ter ajudado. Fico à disposição. Abraços e bons estudos!
Caso este post tenha lhe ajudado, por favor, marcar como solucionado ✓. Bons Estudos!