Nem todos os números mágicos são iguais.
Eu acho que nesse caso, essa constante é boa. O problema com os números mágicos é quando eles são mágicos, ou seja, não está claro qual é a sua origem, porque o valor é o que é, ou se o valor está correto ou não.
Ocultar 1024 atrás de BYTES_PER_KBYTE também significa que você não vê instantaneamente se está correto ou não.
Eu esperaria que qualquer um soubesse imediatamente porque o valor é 1024. Por outro lado, se você estivesse convertendo bytes em megabytes, eu definiria a constante BYTES_PER_MBYTE ou similar porque a constante 1.048.576 não é tão óbvia que a sua 1024 ^ 2, ou que é mesmo correto.
O mesmo vale para valores que são ditados por requisitos ou padrões, que são usados apenas em um lugar. Acho que colocar o direito constante no lugar com um comentário para a fonte relevante é mais fácil de lidar do que defini-lo em outro lugar e ter que perseguir ambas as partes, por exemplo:
// Value must be less than 3.5 volts according to spec blah.
SomeTest = DataSample < 3.50
Acho melhor que
SomeTest = DataSample < SOME_THRESHOLD_VALUE
Somente quando SOME_THRESHOLD_VALUE
é usado em vários lugares é que a compensação vale a pena definir uma constante, na minha opinião.