Impostazioni codice

di il
1 risposte

Impostazioni codice

Ciao a tutti. Ho eseguito un esercizio dove si chiede di fare delle operazioni sugli array. Posto il codice per avere un vostro parere sulla indentazione del codice o formattazione e ,visto che ci sono, avere un vostro parere su come è stato sviluppato. Mi preme sapere e mi vergogno un pò, se può andar bene sviluppato così. Sono sicuro che ci sono altri modi interessantissimi di svolgere un lavoro così ma forse bisogna fare esperienza. Grazie

<html>
    <body onload="visualizza()">
        
        <input type="button" value="Somma" onclick="somma()">
        <input type="button" value="Sottrazione" onclick="sottrazione()">
        <input type="button" value="Moltiplicazione" onclick="moltiplicazione()">
        <input type="button" value="Divisione" onclick="divisione()">
        <script src="operazioni_su_array.js"></script>
        <div id="vis"></div>
        <div id="vis1"> </div>
        <div id="vis2"></div>
        <div id="vis3"></div>

    
    </body>
</html>

let arr1=[]
let arr2=[]
let arr3=[]
let operazione=["somma","addizione","moltiplicazione","divisione"]

function visualizza(){
    let vis=document.getElementById("vis") // esercizio operazioni sugli array: array
    let vis1=document.getElementById("vis1")
    let numRnd2=0
    let numRnd=0


    for(let i=0; i< 10; i++){
    numRnd=Math.floor(Math.random()*10)+1
    arr1.push(numRnd)
    vis.innerHTML= "Input a= " + "[" +arr1+ "]"
}
    for(let x=0; x < 10; x++){
        numRnd2=Math.floor(Math.random()*10)+1
        arr2.push(numRnd2)
        vis1.innerHTML="Input b= " + "[" + arr2 + "]"
    }
}   

function somma(){
    let vis2=document.getElementById("vis2")
    let vis3=document.getElementById("vis3")

    for(let y=0; y < 10;y++){
                arr3 += arr1[y] + arr2[y]+","
                vis2.innerHTML= "Operazione= " + operazione[0]
                vis3.innerHTML= "Output c= " + "[" +arr3+ "]"   
           
    }
}
  
function sottrazione(){
    let vis2=document.getElementById("vis2")
    let vis3=document.getElementById("vis3")

    for(let y=0; y < 10;y++){
                arr3 += arr1[y] - arr2[y]+","
                vis2.innerHTML= "Operazione= " + operazione[1]
                vis3.innerHTML= "Output c= " + "[" +arr3+ "]"   
           
    }
}

function moltiplicazione(){
    let vis2=document.getElementById("vis2")
    let vis3=document.getElementById("vis3")

    for(let y=0; y < 10;y++){
                arr3 += arr1[y] * arr2[y]+","
                vis2.innerHTML= "Operazione= " + operazione[2]
                vis3.innerHTML= "Output c= " + "[" +arr3+ "]"   
           
    }
}

function divisione(){
    let vis2=document.getElementById("vis2")
    let vis3=document.getElementById("vis3")

    for(let y=0; y < 10;y++){
                arr3 += (arr1[y] / arr2[y]).toFixed(2) + "," //il metodo toFixed NON è preciso
                
                vis2.innerHTML= "Operazione= " + operazione[3]
                vis3.innerHTML= "Output c= " + "[" +arr3+ "]"   
           
    }
}

1 Risposte

  • Re: Impostazioni codice

    tonyven ha scritto:


    Posto il codice per avere un vostro parere sulla indentazione del codice o formattazione e ,visto che ci sono, avere un vostro parere su come è stato sviluppato. [...]
    Al netto che la formattazione non mi sembra corretta, dato che vi sono delle graffe male indentate, qui suggerirei l'uso di una funzione specifica dell'editor (es. VS Code) che già è in grado di ri-formattare all'occorrenza una parte o tutto il codice scritta, senza doverlo fare manualmente (cosa che suppongo tu stia facendo).

    Detto questo, leggersi tutto il codice di un intero listato, capire cosa fa, decifrarlo a causa della formattazione e suggerire migliorie che potrebbero essere molteplici, diventa un lavoro abbastanza faticoso e impegnativo in termini di tempo, almeno per me.

    Io focalizzerei l'attenzione su cose specifiche e magari indicando cosa dovrebbero fare, discuterei possibili migliorie nell'ambito di parti singole o specifiche, senza dover analizzare tutto quanto il codice, partendo da un tuo dubbio o proposta.
Devi accedere o registrarti per scrivere nel forum
1 risposte