0

Colegas.

Estou trazendo do banco as notas dos alunos que estão da seguinte forma:

C,D,A,C,A,E,A,B,C,C,B,D,A,C,D,D,B,B,B,A

Porém preciso comparar com as notas do gabarito que vem de um XML e graças a ajuda dos colegas, cheguei ao seguinte código:

   $contar = 0;
    $notasAlunos = explode(',',$aluno->avaliacao_respostas);
    foreach($xml->avaliacao->disciplina as $disciplina) {
           echo $disciplina->attributes()->nome.":<br>";
           foreach ($disciplina as $questao) {
                    foreach($notasAlunos as $notaAluno){ // Aqui trago as notas dos alunos
                            if($questao->resposta == $notaAluno){ // aqui verifico se as questões do gabarito é igual a nota do aluno
                               $conta = $contar++;                       
                           }                
                    }   
           }
    }
echo $conta;

O único problema é que as quando uso dessa forma, em vez de me trazer 17 pontos, está me trazendo 91.

17
  • $conta = $contar++; isso é redundância.
    – user28595
    21/04/2016 às 13:04
  • Olá Diego... certo.... qual a melhor forma?
    – user24136
    21/04/2016 às 13:05
  • 1
    @DiegoF nao é redundância, é um bug sério, pois fica imprevisível o resultado (depende de implementação e pode mudar).
    – Square Obama
    21/04/2016 às 13:11
  • 1
    @DiegoF o ++ incrementa a variável DEPOIS de fazer a atribuição (igual). O problema neste caso é que dependendo de como a linguagem processa isso, o valor pode ou não ser incrementado. Contar com comportamento implícito por si só já é péssimo. Nesses casos, com ambiguidade, pior ainda. E mesmo que você teste e funcione, numa mudança de versão de linguagem pode mudar o comportamento.
    – Square Obama
    21/04/2016 às 13:22
  • 1
    @DiegoF mas nao tá errado o que vc disse, só alertei pra um perigo maior. Em principio, o "efeito" é apenas redundante mesmo. O problema é que não tem como garantir que o comportamento seja o esperado sempre. (a não ser que em algum lugar da documentação do PHP explicitasse o que acontece nestes casos, aí faria parte do design da linguagem, aí seria apenas "retrabalho" desnecessário mesmo)
    – Square Obama
    21/04/2016 às 13:30

1 Resposta 1

2

Tente desta forma:

$contar = 0;
$notasAlunos = explode(',',$aluno->avaliacao_respostas);
foreach($xml->avaliacao->disciplina as $disciplina) {
    echo $disciplina->attributes()->nome.":<br>";
    foreach ($disciplina as $key => $questao) {
        if($questao->resposta == $notaAluno[$key]){
            $contar++;                       
        }                  
    }
    echo $contar;
    $contar = 0;
}

Assim, você exibe a nota da disciplina assim que o laço responsável por contar os acertos terminar, e em seguida, zera a variavel para que ela não some valores de acertos de outras disciplinas.

9
  • Olá Diego. Fiz dessa forma, porém continua me retornando 91. Estranho mesmo. Quando coloquei o print dentro do laço, ele me retorna: 4 8 13 18 23 24 29 34 39 43 48 52 57 62 67 71 76 81 86 91 O XML retorna as respostas corretas na mesma ordem da resposta dos alunos
    – user24136
    21/04/2016 às 14:25
  • @Jose.Marcos acontece que sem ver como seu xml está, fica dificil melhorar o código. Adicione um trecho do xml com as respostas certas.
    – user28595
    21/04/2016 às 14:28
  • Estou achando que a linha foreach($notasAlunos as $notaAluno){ está dando esse problema. Existe algum meio de pegar as notas do aluno sem o foreach?
    – user24136
    21/04/2016 às 14:35
  • @Jose.Marcos é garantido que o numero de respostas em $notasAlunos é sempre o total de questoes de todas as disciplinas do xml?
    – user28595
    21/04/2016 às 14:37
  • 1
    @Jose.Marcos veja se dessa forma exibe corretamente
    – user28595
    21/04/2016 às 15:11

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