Olá Wellington, tudo bem com você?
As duas maneiras são válidas, a única diferença é que na segunda maneira você está separando em mais uma variável a parte do texto, então normalmente o que fazemos é pensar em como será estruturado o código
Por exemplo, se eu quero sempre pegar o valor textual, a primeira maneira é mais simplificada dado que eu não utilizarei as variáveis que guardam os elementos, então podemos dizer que são "desnecessárias" :)
Agora por exemplo, se além de pegar o texto eu quero criar um evento de clique, ai a segunda maneira é mais eficiente pois eu já tenho separado o elemento e não preciso procurar novamente na DOM :)
Conseguiu Compreender? Então vai mais do que iremos fazer no código!
E normalmente no segundo método podemos indicar que se trata de um elemento html
para não precisar criar variáveis com nome parecido, então poderiamos trocar salada
por:
var saladaTr = pratoDoDia.querySelector(".salada");
var saladaHtml = pratoDoDia.querySelector(".salada");
E ai poderíamos deixar salada
para ser o valor do .textContent
Abraços e Bons Estudos!