Porque a cor verde está repetindo duas vezes?

Carlos Henrique, se você alterar a maneira de escrever a função, vai funcionar sem repetir cores:

    function mudaCor() {  

    cores[corAtual] ;  
    corAtual++; 

    if (corAtual>=cores.length) {
        corAtual = 0;

}

Não consegui encontrar uma explicação teórica, mas lembro de ter visto em algum momento que existe um laço de repetição que tem a característica de executar a instrução uma vez mais depois que atinge a condição estabelecida como parâmetro. Talvez o motivo esteja relacionado a isso. Vamos ver se aparecem outras sugestões. Bons estudos!

Leila, pelo que parece o problema está com o if else, já que quando se faz 2 if funciona normalmente.Apesar de o if else somente funcionar se deixássemos o código assim:

   function mudaCor() {  
   corAtual++; 
   if (corAtual<cores.length) {
    cores[corAtual] ;  

} else {corAtual = 0;

}
   return false;
}

tela.oncontextmenu = mudaCor;

Vamos ver se alguém sabe uma explição teórica para isso. Obrigado pela sua sugestão.

Eu fiz um teste colocando "- 1" depois do length e resolveu o problema de repetir cor:

function mudaCor() {  
     if (corAtual<cores.length - 1) {
      cores[corAtual] ;  
      corAtual++; 
} else {corAtual = 0;

}
   return false;
}

Mas, continuo sem saber explicar o por quê...

É verdade, nesse caso resolveria oproblema de repetir cor ,porém colocando esse trecho de código que você botou agora, somente fica azul e verde. No caso, teria que tirar o -1 do cores.length, já que ao invés de contar 3 cores do array 'cores', conta somente os 2 primeiros.

Mais especificamente :

cores.length = 3

passaria para

cores.length =2

, ou seja , do jeito que vc colocou corAtual<2 , e não corAtual<3, sendo que na situação que eu coloquei teriam 3 cores, portanto a cor vermelha ( a última do array, teria o índice 2).

Leila, caso algum de nós dois não ache a solução, eu acho que seria uma boa ideia postarmos essa dúvida de novo no fórum, para chamar a atenção do pessoal do scubateam, ou de outra pessoa que saiba porque isso acontece.

isso acontece porque o corAtual++; esta dentro do IF, mas ele precisa ficar fora, no caso em cima, assim;

function mudaCor() {
        corAtual++; 
       if (corAtual < cores.length) {
        corAtual[cores] ;         
    }   else {corAtual = 0;

    }
       return false;
    }

Primeiro a gente troca ao clicar depois ele checa se ainda é menor. Se não ele conta um índice a mais e continua sendo vermelho

Quando tem dois ifs, como no caso abaixo, parece que raciocínio é parecido, quanto a parte de trocar e depois conferir. Acho que agora ficou mais claro! Obrigado Lucas! Deixo aqui o trecho do código:

function mudaCor() {
        corAtual++; 
       if (corAtual < cores.length) {
        corAtual[cores] ;         
    }   if (corAtual>=cores.length) {

    corAtual = 0;
    }
       return false;
    }