Esse codigo é muito gambiarra?

Thihup
(usa Manjaro Linux)

Enviado em 12/10/2014 - 20:36h

Ola pessoal , eu decidi tentar fazer uma coisa (nao posso falar agr , depois vcs verao),e usei esse codigo , mas acho que ele está muito gambiarra.

Alguem ae que manja dos Javascript pode alterar-lo e deixar-lo menos gambiarra ?

var nota1 = 0,nota2 = 0,nota3 = 0,nota4 = 0;

function testaValor(){
	
	if(nota1 < 0 || nota1 > 10 || isNaN(nota1)){
	
		alert("Nota 1 " + materia + " Invalida");
	
		return "Erro";
	
	}
	
	if(nota2 < 0 || nota2 > 10 || isNaN(nota2)){
	
		alert("Nota 2 " + materia + " Invalida");
	
		return "Erro";
	
	}
	
	
	if(nota3 < 0 || nota3 > 10 || isNaN(nota3)){
	
		alert("Nota 3 " + materia + " Invalida");
	
		return "Erro";
	
	}
	
	if(nota4 < 0 || nota4 > 10 || isNaN(nota4)){
	
		alert("Nota 4 " + materia + " Invalida");
	
		return "Erro";
	
	}		
}


function pegaP() {
	
	materia="Portugues";
    
    nota1 = document.getElementById('notaPort1').value;
    
    if(testaValor() == "Erro"){return;}
    
    nota2 = document.getElementById('notaPort2').value;
    
    if(testaValor() == "Erro"){return;}
    
    nota3 = document.getElementById('notaPort3').value;
    
    if(testaValor() == "Erro"){return;}
    
    nota4 = document.getElementById('notaPort4').value;
    
    if(testaValor() == "Erro"){return;}
    
    var media = parseFloat(nota1) + parseFloat(nota2) + parseFloat(nota3) + parseFloat(nota4);
    
    media /= 4;
    
    document.getElementById("MediaP").innerHTML = media; 
}

function pegaM() {
    
    materia="Matematica";
    
    nota1 = document.getElementById('notaMat1').value;
    
    if(testaValor() == "Erro"){return;}
    
    nota2 = document.getElementById('notaMat2').value;
    
    if(testaValor() == "Erro"){return;}
    
    nota3 = document.getElementById('notaMat3').value;
    
    if(testaValor() == "Erro"){return;}
    
    nota4 = document.getElementById('notaMat4').value;
    
    if(testaValor() == "Erro"){return;}
    
    var media = parseFloat(nota1) + parseFloat(nota2) + parseFloat(nota3) + parseFloat(nota4);
    
    media /= 4;
    
    document.getElementById("MediaM").innerHTML = media; 
}

function pegaC() {
    
    materia="Ciencias";
    
    nota1 = document.getElementById('notaCie1').value;
    
    if(testaValor() == "Erro"){return;}
    
    nota2 = document.getElementById('notaCie2').value;
    
    if(testaValor() == "Erro"){return;}
    
    nota3 = document.getElementById('notaCie3').value;
    
    if(testaValor() == "Erro"){return;}
    
    nota4 = document.getElementById('notaCie4').value;
    
    if(testaValor() == "Erro"){return;}
    
    var media = parseFloat(nota1) + parseFloat(nota2) + parseFloat(nota3) + parseFloat(nota4);
    
    media /= 4;
    
    document.getElementById("MediaC").innerHTML = media;   
}

 

Grato desde já

[]'s

T+

removido
(usa Nenhuma)

Enviado em 12/10/2014 - 20:53h

É só arrumar a indentação ;)

var nota1 = 0;
var nota2 = 0;
var nota3 = 0;
var nota4 = 0;

function testaValor()
{
	if(nota1 < 0 || nota1 > 10 || isNaN(nota1))
	{
		alert("Nota 1 " + materia + " Invalida");
		return "Erro";
	}

	if(nota2 < 0 || nota2 > 10 || isNaN(nota2))
	{
		alert("Nota 2 " + materia + " Invalida");
		return "Erro";
	}

	if(nota3 < 0 || nota3 > 10 || isNaN(nota3))
	{
		alert("Nota 3 " + materia + " Invalida");
		return "Erro";
	}

	if(nota4 < 0 || nota4 > 10 || isNaN(nota4))
	{
		alert("Nota 4 " + materia + " Invalida");
		return "Erro";
	}		
}

function pegaP()
{
	materia="Portugues";


	nota1 = document.getElementById('notaPort1').value;
	if(testaValor() == "Erro"){return;}

	nota2 = document.getElementById('notaPort2').value;
	if(testaValor() == "Erro"){return;}

	nota3 = document.getElementById('notaPort3').value;
	if(testaValor() == "Erro"){return;}

	nota4 = document.getElementById('notaPort4').value;
	if(testaValor() == "Erro"){return;}


	var media = parseFloat(nota1) + parseFloat(nota2) + parseFloat(nota3) + parseFloat(nota4);
	media /= 4;
	document.getElementById("MediaP").innerHTML = media; 
}

function pegaM()
{
	materia="Matematica";


	nota1 = document.getElementById('notaMat1').value;
	if(testaValor() == "Erro"){return;}

	nota2 = document.getElementById('notaMat2').value;
	if(testaValor() == "Erro"){return;}

	nota3 = document.getElementById('notaMat3').value;
	if(testaValor() == "Erro"){return;}

	nota4 = document.getElementById('notaMat4').value;
	if(testaValor() == "Erro"){return;}

	var media = parseFloat(nota1) + parseFloat(nota2) + parseFloat(nota3) + parseFloat(nota4);
	media /= 4;
	document.getElementById("MediaM").innerHTML = media; 
}

function pegaC()
{
	materia="Ciencias";


	nota1 = document.getElementById('notaCie1').value;
	if(testaValor() == "Erro"){return;}

	nota2 = document.getElementById('notaCie2').value;
	if(testaValor() == "Erro"){return;}

	nota3 = document.getElementById('notaCie3').value;
	if(testaValor() == "Erro"){return;}

	nota4 = document.getElementById('notaCie4').value;
	if(testaValor() == "Erro"){return;}

	var media = parseFloat(nota1) + parseFloat(nota2) + parseFloat(nota3) + parseFloat(nota4);
	media /= 4;
	document.getElementById("MediaC").innerHTML = media;   
}
 

Thihup
(usa Manjaro Linux)

Enviado em 12/10/2014 - 21:12h

Fora a identação , teria algum metodo de fazer esse codigo + simplificado?

Quero fazer um tipo um boletim , e colocar uma funcao p/ cada uma coisa ficaria muito grande .Teria solução?

[]'s

T+

Thihup
(usa Manjaro Linux)

Enviado em 12/10/2014 - 21:43h

#SAMUEL_ESTRAGA_PRAZER

eduardoweiland
(usa Fedora)

Enviado em 12/10/2014 - 21:58h

Um jeito de organizar melhor esse código seria utilizar um vetor para armazenar as notas, e você só precisaria percorrê-lo em vez de testar várias vezes a mesma coisa com variáveis diferentes. E também, nas funções pegaP(), pegaM() e pegaC() você chama a função para testar o valor das notas quatro vezes (uma para cada valor lido dos inputs), e a função valida todos os quatro valores todas as vezes. Poderia primeiro pegar todos os valores e só no final testar.

E outra ideia, tente evitar o uso de variáveis globais, é muito fácil de se perder no código e não dá pra saber quando o valor dela é alterado. Tente usar parâmetros para as funções.

Seu código com essas melhorias ficaria parecido com isso:

function testaValor(notas, materia) {
    for (var i = 0; i < notas.length; ++i) {
        if (notas[i] < 0 || notas[i] > 10 || isNaN(notas[i])) {
            alert("Nota " + (i + 1) + " " + materia + " Invalida");
            return false;
        }
    }
    return true;
}

function calculaMedia(notas) {
    var total = 0;

    for (var i = 0; i < notas.length; ++i) {
        total += parseFloat(notas[i]);
    }

    return (total / notas.length);
}
 
function pegaP() {
    var materia = "Portugues";
    var notas   = [];

    notas.push(document.getElementById('notaPort1').value);
    notas.push(document.getElementById('notaPort2').value);
    notas.push(document.getElementById('notaPort3').value);
    notas.push(document.getElementById('notaPort4').value);

    if (!testaValor(notas, materia)) {
        return;
    }

    var media = calculaMedia(notas);
    document.getElementById("MediaP").innerHTML = media;
}
 
function pegaM() {
    var materia = "Matematica";
    var notas   = [];

    notas.push(document.getElementById('notaMat1').value);
    notas.push(document.getElementById('notaMat2').value);
    notas.push(document.getElementById('notaMat3').value);
    notas.push(document.getElementById('notaMat4').value);

    if (!testaValor(notas, materia)) {
        return;
    }

    var media = calculaMedia(notas);
    document.getElementById("MediaM").innerHTML = media;
}
 
function pegaC() {
    var materia = "Ciencias";
    var notas   = [];

    notas.push(document.getElementById('notaCie1').value);
    notas.push(document.getElementById('notaCie2').value);
    notas.push(document.getElementById('notaCie3').value);
    notas.push(document.getElementById('notaCie4').value);

    if (!testaValor(notas, materia)) {
        return;
    }

    var media = calculaMedia(notas);
    document.getElementById("MediaC").innerHTML = media;
}
 

Thihup
(usa Manjaro Linux)

Enviado em 12/10/2014 - 21:59h

????
Eu disse que vc estragou o meu prazer te ter feito o "Erro".

Eu ainda quero saber como posso melhorar esse codigo , seria bom se tivesse structs.

[]'s

T+