Desempenho
Não sou um expert nesse assunto, mas posso garantir que o desempenho nesse caso não faça tanta diferença ao ponto de valer a penas escolher um ou outro por conta disso.
Segurança - Entendendo o comportamento de um e outro
A única segurança que está em risco é a sua. Digo isso pois é necessário entender o comportamento das referências nesse caso.
Como foi respondido nessa pergunta existe uma pequena variação, em relação ao que se é esperado de um foreach normal, em relação à variável passada por referência.
Resumindo: O último valor fica sempre referenciado, podendo te causar problemas se você sobrescrever o valor de $value
.
$array = [1, 2, 3];
foreach ($array as $key => &$value) {
$value = $value . 'R$';
}
$value = null;
Isso geraria:
['1R$', '2R$', null]
Veja um exemplo no IDEONE
Por isso depois de usar um foreach
com referência, sempre é bom chamar unset
na variável referenciada (depois do loop).
Não vou falar muito sobre isso aqui, pois a pergunta linkada tem bastante sobre o assunto.
Nesse caso, não estou dizendo pra você não fazer assim (eu já fiz algumas poucas vezes). Estou apenas alertando que é melhor você saber tudo sobre o assunto, para o que "você não sabe" te cause problemas.
Mas, observando que podem surgir um "novo comportamento" em relação ao primeiro foreach
- e ao mesmo tempo, isso também pode afetar a legibilidade caso esqueça de tratar a variável da referência devidamente-, eu poderia pensar na sua segunda opção como sendo a melhor.
$array = [1, 2, 3];
foreach ($array as $key => $value) {
$array[$key] = $value . 'R$';
}
Porém é preciso saber se é necessário e se é sábio alterar o valor original do seu array
. Se você tem certeza que não vai mais usar o array
"original", então você pode fazer isso sem medo de ser feliz :p
A desvantagem de referência com foreach
Lembrando que também você tem uma desvantagem ao usar foreach
com referência: Você só pode fazer loop com o array
guardado em variáveis. Expressões arbitrárias não podem ser usadas com referência em algumas versões do PHP anteriores ao 5.5.
Exemplo:
// Certo:
$array = [];
foreach ([1, 2, 3, 4] as $key => $value) {
$array[$key] = $value . 'R$';
}
// Errado: Isso vai gerar um erro caso use PHP 5.4
foreach ([1, 2, 3, 4] as $key => &$value) {
$value = $value . 'R$';
}
Veja um pequeno teste
Tire a sua própria conclusão
Não quero te dar uma opinião sobre o assunto, quero que você apenas entenda que fazer um foreach com referência e fazer uma reatribuição com as chaves do foreach
pode parecer que é a mesma coisa, mas tecnicamente não é. Referências podem ser vilãs se não forem usadas com sabedoria. Da mesma forma sobrescrever um valor de um array
pode não ser uma boa ideia se você quer manter os valores originais.
Estou te mostrando alguns pontos, pois essa coisa de "boas práticas" na verdade é bem pessoal. Sempre é melhor você saber o que está fazendo e saber usar na hora certa.
if
sem chaves. Uma das respostas com certeza seria relacionada a boas práticas e diria para sempre utilizar chaves noif
.