Skip to main content
5 de 5
adicionou 304 caracteres ao conteúdo
rray
  • 67,7mil
  • 21
  • 116
  • 285

No lugar de strpos() pode usar strstr() para pegar a última parte da string e comparar o valor do loop com array $tipos com ajuda de in_array().

$tipos = array('SUPORTE', 'MANUTENÇÃO', 'DESENVOLVIMENTO');

foreach ($json_data->ConsultarNfseResposta->ListaNfse->CompNfse as $data) {
    echo '<br>Número da nota: ' .$data->Nfse->InfNfse->Numero;
    echo '<br>Valor da nota: ' .$data->Nfse->InfNfse->Servico->Valores->ValorServicos;
    echo '<br>CNPJ do cliente: ' .$data->Nfse->InfNfse->TomadorServico->IdentificacaoTomador->CpfCnpj->Cnpj;
    echo '<br>Discriminação do serviço: ' .$data->Nfse->InfNfse->Servico->Discriminacao;
    echo '<br>';
    
    $tipoAtual = strstr($data->Nfse->InfNfse->Servico->Discriminacao, ' ');
    
    if(in_array($tipoAtual, $tipos)){
        printf('<b>String encontrada: </b> %s <br>', $data->Nfse->InfNfse->Servico->Discriminacao);
    }
}

Explicação:

Utilizar strpost() sem tomar os devidos cuidados é uma tremenda furada, pois ele retorna a posição em que a string passada foi encontrada. Isso significa que se ela for a zero seu código gera um falso positivo. Veja esse exemplo clássico:

$pos = strpos('X-salada', 'X');

if($pos == true){
    echo 'tem';
}else{
    echo 'não tem';
}

A saída é não tem. Normalmente (em linguagens com tipagem forte) não se compara booleanos com true ou false, simplemente se omite a comparação passa-se a variável/expressão direto, então usar o operador == não é o correto, nesse caso use o !== ou === pois ele compara o valor e o tipo.

$pos = strpos('X-salada', 'X');

if($pos !== false){
    echo 'tem';
}else{
    echo 'não tem';
}

Os três ifs dentro do forech fazem a mesma coisa imprimem algo o que muda é a comparação um || (OU) já é o suficiente para elimina-los.

Duplicação de código:

if ($posSuporte == true) {
    $servSuporte = $data->Nfse->InfNfse->Servico->Discriminacao;
    echo '<b>String encontrada: </b>' .$servSuporte;
    echo '<br>';
} 

if ($posManutencao == true) {
    $servManutencao = $data->Nfse->InfNfse->Servico->Discriminacao;
    echo '<b>String encontrada: </b>' .$servManutencao;
    echo '<br>';
} 

if ($posDesenvolvimento == true) {
    $servDesenvolvimento = $data->Nfse->InfNfse->Servico->Discriminacao;
    echo '<b>String encontrada: </b>' .$servDesenvolvimento;
    echo '<br>';
}

Pode virar algo simples como:

if($data->Nfse->InfNfse->Servico->Discriminacao == 'TAXA: SUPORTE' ||
   $data->Nfse->InfNfse->Servico->Discriminacao == 'TAXA: MANUTENÇÃO' ||
   $data->Nfse->InfNfse->Servico->Discriminacao == 'TAXA: DESENVOLVIMENTO')
{
    echo '<b>String encontrada: </b>' . $data->Nfse->InfNfse->Servico->Discriminacao .'<br>';
}   

Relaciondas:

Sinal idêntico “===” é usado apenas em PHP? Por quê?

rray
  • 67,7mil
  • 21
  • 116
  • 285