Nesse exemplo a diferença é que escreveu uma linha a mais no primeiro.
Há zero vantagens aí, e a única razão que vejo as pessoas usando assim é que não sabem o que é uma variável e para que ela serve. Claro, o gosto da pessoa pode ser outro motivo, mas aí não é razão.
Se tivesse uma expressão mais complexa eu até entenderia que se fizesse antes e guardasse em uma variável com nome significativo. Um nome que não é informativo ou informa a obviedade não aumenta a legibilidade.
Se o valor fosse usado em outros lugares aí tem o exato motivo para uma variável existir.
Fazer algo porque um dia pode precisar viola o princípio do YAGNI, e de uma certa forma do KISS. Tendo um IDE bom mudar a expressão para uma variável se precisar é a coisa mais simples que tem.
Não é para poupar código, é para simplificar o código. Colocar a declaração/definição do que vai usar mais próximo possível de onde é usado costuma dar mais legibilidade. Código longo é mais difícil de acompanhar. Fora a semântica que se existe uma variável é porque deve ser usado em vários lugares. Código é expressão, quando expressa errado passa a ideia errada e torna o código convoluto.
Dependendo da linguagem há diferença de performance e até consumo de memória.
Nunca faça nada sem saber porque está fazendo. Não vá no automático, não siga regras que outros disseram que é bom. Procure o que é bom para você, mas faça isso com a consciência de escolha.
$sql
que se você colocasse diretamente na linha$result
a dificuldade de encontrar o erro é maior. É só um exemplo.