Javascript: potete verificare se questo codice che ho scritto è corretto?

😈╰•ALBERTO•╯😈

Diciamo che il codice soddisfa la mia richiesta, però sono sempre pessimista, penso sempre che forse il codice si dovrebbe scrivere in un altro modo rispetto a come l'ho scritto io, ormai sono così scoraggiato dal mio tutor che quando devo far vedere un esercizio mi viene l'attacco di panico. Quindi prima vorrei sapere se il codice è giusto, altrimenti lo modifico

L'esercizio spiega:
Determinare qual è il valore "true o "false" che è più "frequente" all'interno dell'array e stampare uno dei seguenti valori:

true se il seguente valore è true
false se il valore più frequente è false
null se la lunghezza dell'array è 0 o vuota o contiene la stessa quantità (io l'ho fatta solo vuota).

let array_uno = [true , true , false , false , true];
let array_due = [false , true , false];
let array_tre = [];
let vero = 0;
let falso = 0;
let i = 0;

/*RISULTATO: TRUE*/

for(i = 0; i < array_uno.length; i++) {


if(array_uno[i]){

vero++;

}

}


if(vero > falso){

console.log(true);

} else{console.log(false);}

/*RISULTATO: FALSE*/

for(i = 0; i < array_due.length; i++){

if(array_due[i]){

vero++;


}

}
if(vero > falso){

console.log(false);

}else{console.log(true);}

/*RISULTATO: NULL*/


for(i = 0; i < array_tre; i++){


if(array_tre[i]){

vero++

}
}
if(array_tre.length ===0 ){

console.log(null);

}else{console.log("Array_tre è piena");}
</script>

Mi da tutti i risultati che dell'esercizio.

Not an engineer

😈╰•ALBERTO•╯😈 Il codice sembra corretto quindi no problem, da notare che potevi anche scriverlo in maniera più corta senza ripetere tutto 3 volte con una funzione che può essere chiamata con diversi array.
La funzione controlla esplicitamente anche il caso in cui l'array sia vuoto.

function findMostFrequentValue(arr) {
  let trueCount = 0;
  let falseCount = 0;

  for (let i = 0; i < arr.length; i++) {
    if (arr[i]) {
      trueCount++;
    } else {
      falseCount++;
    }
  }

  if (trueCount > falseCount) {
    return true;
  } else if (falseCount > trueCount) {
    return false;
  } else {
    return null;
  }
}

console.log(findMostFrequentValue([true, true, false, false, true])); // true
console.log(findMostFrequentValue([false, true, false])); // false
console.log(findMostFrequentValue([])); // null

😈╰•ALBERTO•╯😈

Not an engineer non li posso usare perché ancora non ci siamo arrivati, altrimenti il tutor si mangia mezz'ora della lezione per lamentarsi che ho usato una cosa che non dovevo, quindi per adesso cerco di fare quello che posso con le poche armi che ho.

Giulio_M

😈╰•ALBERTO•╯😈 mah, in pratica non puoi usare le funzioni. Quindi prendi ciò che ha scritto Not an engineer che direi che è perfetto per la risoluzione di questo problema (analogo a quanto ti avevo scritto io nel caso JavaScript: "pizze vs patatine") e semplicemente, al posto della funzione, crei una variabile di appoggio (anziché dire "return true" ad esempio scrivi x=true). Modifichi questo e di fatto non ti serve la funzione. Poi se la variabile che hai creato è "x" (che prende i valori "true", "false", "null" a seconda dei casi), semplicemente console.log(x) ti mostra il risultato.

Giulio_M

😈╰•ALBERTO•╯😈 giusto per completezza, secondo quanto ho detto prima (Giulio_M), riporto tutto il codice (quindi creando una variabile "x" anziché usare le funzioni).

let V=[true, true, false, false, true]
let trueCount = 0;
let falseCount = 0;
let x;
  for (let i = 0; i < V.length; i++) {
    if (V[i]) {
      trueCount++;
    } else {
      falseCount++;
    }
  }

  if (trueCount > falseCount) {
    x='true';
  } else if (falseCount > trueCount) {
    x='false';
  } else {
    x='null';
  }
console.log(x)

In questo caso, dato che abbiamo let V=[true, true, false, false, true] come valore di console.log(x) viene stampato "true". Se provi a modificare ciò che è contenuto nell'array, vedi infatti che viene stampato rispettivamente "false" o "null".

Considera che questo codice da per scontato che il contenuto dell'array sia corretto ovvero che contenga solametne "true" oppure "false"; altrimenti basterebbe aggiungere una condizione ulteriore più restrittiva anziché semplicemente if-else nel ciclo (nel senso che se V[ i ] è diverso sia da "true" che "false", non incrementa nessun contatore quindi anziché if-else mettere due if separati, questa è solamente una sottigliezza).