16

Tive que fazer uma verificação para concatenar conteúdo a uma variável, porém acho que o serviço ficou um tanto "porco" e queria ver se alguém poderia me ajudar a simplificar essa verificação

if($IdUserOnline2){
    $IdsInteracoes .= implode(',',$IdUserOnline2);
    if($IdUserComum2){
        $IdsInteracoes .= ','.implode(',',$IdUserComum2);
        if($IdUserNovo2){
            $IdsInteracoes .= ','.implode(',',$IdUserNovo2);
        }
    }elseif($IdUserNovo2){
        $IdsInteracoes .= ','.implode(',',$IdUserNovo2);
    }
}elseif($IdUserComum2){
    $IdsInteracoes .= implode(',',$IdUserComum2);
    if($IdUserNovo2){
        $IdsInteracoes .= ','.implode(',',$IdUserNovo2);
    }
}elseif($IdUserNovo2){
    $IdsInteracoes .= implode(',',$IdUserNovo2);
}
4
  • 3
    Nossa cara, você poderia explicar o que você queria fazer quando fez isto? desenvolver uma nova lógica com certeza vai ser melhor do que tentar corrigir esta(acredite) seria ótimo :D você pode editar sua pergunta com a intenção original? Commented 21/03/2014 às 14:22
  • A intenção ja esta explicita na própria pergunta, tenho que adicionar varios arrays a 1 só, porém, em dado momento preciso adicionar um prefixo contendo uma "," para separar os arrays e não bugar, após juntar os 3 terei uma string com todos eles e passo isso para um bind_param do mysqli. Commented 24/03/2014 às 12:18
  • Humm, Porque não adicionar todos os valores em um único Array e depois criar um laço de repetição para adicionar uma , em cada valor? as , sao por causa do sql né? Commented 24/03/2014 às 12:22
  • eu poderia fazer isso com array merge, mas se uma delas voltar vazia da erro na função, gerando uma cadeia de erros, a verificação tem que ser individual e a ação tambem... Commented 24/03/2014 às 12:24

2 Respostas 2

17

Caso as chaves do seu array não se repitam entre eles você pode fazer o seguinte:

$todosArrays = $IdUserOnline2 + $IdUserComum2 + $IdUserNovo2;
$IdsInteracoes = implode(',',$todosArrays);

Você pode também usar a função array_merge caso as chaves do array se repitam

$todosArrays = array_merge($IdUserOnline2, $IdUserComum2, $IdUserNovo2);
$IdsInteracoes = implode(',',$todosArrays);

Lembrando que, no caso do array_merge, verifique se as variáveis anteriores são arrays, evitando os erros:

$IdUserOnline2 = isset($IdUserOnline2) ? $IdUserOnline2 : array();
$IdUserComum2 = isset($IdUserComum2) ? $IdUserOnline2 : array();
$IdUserNovo2 = isset($IdUserNovo2) ? $IdUserOnline2 : array();

// É possível usar um ternário reduzido, porém é preciso inicializar as variáveis antes
// $IdUserOnline2 = $IdUserOnline2 ?: array();
// $IdUserComum2 = $IdUserComum2 ?: array();
// $IdUserNovo2 = $IdUserNovo2 ?: array();

$todosArrays = array_merge($IdUserOnline2, $IdUserComum2, $IdUserNovo2);
$IdsInteracoes = implode(',',$todosArrays);

A verificação das condições utilizam um operador ternário.

A lógica é simples: Verifica se a variável existe, se ela não existir, inicializa ela como um array em branco. Exemplo completo.


PHP 7+

Foi implementado um novo operador no PHP 7 chamado Null Coalesce Operator, indicado para situações onde é preciso definir um valor padrão caso a expressão anterior retorne null.

Com ele, mesmo que a variável não esteja definida, o PHP não irá gerar um Notice, funcionando da mesma forma que o || do Javascript.

Assim poderíamos simplificar o código acima para:

$IdUserOnline2 = $IdUserOnline2 ?? [];
$IdUserComum2 = $IdUserComum2 ?? [];
$IdUserNovo2 = $IdUserNovo2 ?? [];

$todosArrays = array_merge($IdUserOnline2, $IdUserComum2, $IdUserNovo2);
$IdsInteracoes = implode(',',$todosArrays);
13
  • 2
    $IdsInteracoes=implode(',',array_merge($IdUserOnline2,$IdUserComum2,$IdUserNovo2)); Bem mais simples assim Commented 20/03/2014 às 19:11
  • Não tinha reparado ainda, mas se uma das variaveis vier vazia da erro nessas funções, então me geraria uma nova hierarquia de verificações que no caso não simplifica o if Commented 21/03/2014 às 11:24
  • o problema é que eu fiz isso e mesmo assim ele me retornou erro no array_merge, pois de uma forma ou de outra ela iria pro array_merge vazia, verificando ou não Commented 21/03/2014 às 13:47
  • Nesse caso não, pois o ternário define a variável como um array vazio caso ela não esteja definida.
    – gmsantos
    Commented 21/03/2014 às 14:43
  • 2
    Gostei dessa modificação no PHP 7. Facilita bastante a vida Commented 29/06/2015 às 20:12
11

Há uma "hierarquia" entre "online", "comum" e "novo", por isso essa forma como você escreveu também é como eu escreveria... Mas no seu caso particular, vejo que o comportamento se repete seja dentro seja fora de cada if. Ou seja, basta substituir tudo por apenas 3 ifs:

$prefixo = '';
if($IdUserOnline2){
    $IdsInteracoes .= implode(',',$IdUserOnline2);
    $prefixo = ',';
}

if($IdUserComum2){
    $IdsInteracoes .= $prefixo . implode(',',$IdUserComum2);
    $prefixo = ',';
}

if($IdUserNovo2){
    $IdsInteracoes .= $prefixo . implode(',',$IdUserNovo2);
}
1
  • @gmsantos PHP dá suporte a "ou" tal como em JavaScript? (ex.: null || [] vira []) Ou só se aplica a booleanos? Se for como no primeiro caso, você pode simplificar sua resposta usando essa forma simples dentro do próprio array_merge (usar o operador ternário não compensa nesse caso - ia ficar uma linha muito grande...). Nesse caso sua resposta de fato seria melhor que a minha (na minha opinião, tal como está a minha é mais clara/legível).
    – mgibsonbr
    Commented 21/03/2014 às 12:09

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .