Bizarre JavaScript Comparisons

Question

Bizarre JavaScript Comparisons

I encountered an issue with the comparison process. The expected result should be 11 since the cost of the product at index 11 is lower than the cost of the product at index 18. However, the computed result turns out to be 18.

    var scores = [60, 50, 60, 58, 54, 54, 58, 50, 52, 54, 48, 69, 34, 55, 51, 52, 44, 51, 69, 64, 66, 55, 52, 61, 46, 31, 57, 52, 44, 18,
    41, 53, 55, 61, 51, 44];
    var costs = [.25, .27, .25, .25, .25, .25, .33, .31, .25, .29, .27, .22, .31, .25, .25, .33, .21, .25, .25, .25, .28, .25, .24, .22, .20, .25, .30, .25, .24, .25,
        .25, .25, .27, .25, .26, .29];
//TOTAL, HIGHEST, INDEX OF THE HIGHEST
function findMaxIndex(array,arrayCost){
    var maximum=Math.max(...array);
    var arrayIndex=[];
    for(var i=0;i<array.length;i++){
        if(array[i]==maximum){
            arrayIndex.push(i)
        }
    }

   var minimum=arrayCost[arrayIndex[0]];//0.22
   for(var i=0;i<arrayIndex.length;i++){
       if(arrayCost[arrayIndex[i]]<=minimum){//0.25>=0.22
           minimum=arrayIndex[i];
       }
   }
    return minimum;
}

javascript

Answer 1

Answer №1

Upon entering the second if condition, you update the value of minimum to 11 by assigning it the value of arrayIndex[i]. Consequently, in the subsequent loop iterations, the minimum will remain at 11, causing arrayCost[arrayIndex[i]] to be less than 11. To address this issue, consider creating a new variable to store the index value.

function findMinimum(array, arrayCost) {
  var maximumValue = Math.max(...array);
  var arrayIndex = [];
  
  for (var i = 0; i < array.length; i++) {
    if (array[i] == maximumValue) {
      arrayIndex.push(i);
    }
  }

  var resultIndex = arrayIndex[0];
  var currentMinCost = arrayCost[arrayIndex[0]]; //0.22
  
  for (var i = 0; i < arrayIndex.length; i++) {
    if (arrayCost[arrayIndex[i]] <= currentMinCost) {
      currentMinCost = arrayCost[arrayIndex[i]];
      resultIndex = arrayIndex[i];
    }
  }
  
  return resultIndex;
}

Answer 2

Upon entering the second if condition, you update the value of minimum to 11 by assigning it the value of arrayIndex[i]. Consequently, in the subsequent loop iterations, the minimum will remain at 11, causing arrayCost[arrayIndex[i]] to be less than 11. To address this issue, consider creating a new variable to store the index value.

function findMinimum(array, arrayCost) {
  var maximumValue = Math.max(...array);
  var arrayIndex = [];
  
  for (var i = 0; i < array.length; i++) {
    if (array[i] == maximumValue) {
      arrayIndex.push(i);
    }
  }

  var resultIndex = arrayIndex[0];
  var currentMinCost = arrayCost[arrayIndex[0]]; //0.22
  
  for (var i = 0; i < arrayIndex.length; i++) {
    if (arrayCost[arrayIndex[i]] <= currentMinCost) {
      currentMinCost = arrayCost[arrayIndex[i]];
      resultIndex = arrayIndex[i];
    }
  }
  
  return resultIndex;
}

Answer 3

Answer №2

The bug you are facing is visible in the comments, however, I suggest reworking it to optimize with a single loop:

var scores = [60, 50, 60, 58, 54, 54, 58, 50, 52, 54, 48, 69, 34, 55, 51, 52, 44, 51, 69, 64, 66, 55, 52, 61, 46, 31, 57, 52, 44, 18,
    41, 53, 55, 61, 51, 44];
    var costs = [.25, .27, .25, .25, .25, .25, .33, .31, .25, .29, .27, .22, .31, .25, .25, .33, .21, .25, .25, .25, .28, .25, .24, .22, .20, .25, .30, .25, .24, .25,
        .25, .25, .27, .25, .26, .29];


function index(array, arrayCost) {
  var maxScore = Math.max(...array);
  var minCostIndex = null;
  for (var i = 0; i < arrayCost.length; i++) {
    if (array[i] != maxScore) {
      continue;
    }
    if (minCostIndex == null || arrayCost[i] < arrayCost[minCostIndex]) {
      minCostIndex = i;
    }
  }

  return minCostIndex;
}

console.log("res", index(scores, costs));

To enhance efficiency further, you can even remove the Math.max function call:

var scores = [60, 50, 60, 58, 54, 54, 58, 50, 52, 54, 48, 69, 34, 55, 51, 52, 44, 51, 69, 64, 66, 55, 52, 61, 46, 31, 57, 52, 44, 18,
    41, 53, 55, 61, 51, 44];
    var costs = [.25, .27, .25, .25, .25, .25, .33, .31, .25, .29, .27, .22, .31, .25, .25, .33, .21, .25, .25, .25, .28, .25, .24, .22, .20, .25, .30, .25, .24, .25,
        .25, .25, .27, .25, .26, .29];


function index(array, arrayCost) {
  var maxScore = array[0];
  var minCostIndex = 0;
  
  // start iteration at 1, as 0 is already considered
  for (var i = 1; i < arrayCost.length; i++) {
    if (array[i] > maxScore) {
      maxScore = array[i];
      minCostIndex = i;
      continue
    }

    if (array[i] == maxScore && arrayCost[i] < arrayCost[minCostIndex]) {
      minCostIndex = i;
    }
  }

  return minCostIndex;
}

console.log("res", index(scores, costs));

Answer 4

The bug you are facing is visible in the comments, however, I suggest reworking it to optimize with a single loop:

var scores = [60, 50, 60, 58, 54, 54, 58, 50, 52, 54, 48, 69, 34, 55, 51, 52, 44, 51, 69, 64, 66, 55, 52, 61, 46, 31, 57, 52, 44, 18,
    41, 53, 55, 61, 51, 44];
    var costs = [.25, .27, .25, .25, .25, .25, .33, .31, .25, .29, .27, .22, .31, .25, .25, .33, .21, .25, .25, .25, .28, .25, .24, .22, .20, .25, .30, .25, .24, .25,
        .25, .25, .27, .25, .26, .29];


function index(array, arrayCost) {
  var maxScore = Math.max(...array);
  var minCostIndex = null;
  for (var i = 0; i < arrayCost.length; i++) {
    if (array[i] != maxScore) {
      continue;
    }
    if (minCostIndex == null || arrayCost[i] < arrayCost[minCostIndex]) {
      minCostIndex = i;
    }
  }

  return minCostIndex;
}

console.log("res", index(scores, costs));

To enhance efficiency further, you can even remove the Math.max function call:

var scores = [60, 50, 60, 58, 54, 54, 58, 50, 52, 54, 48, 69, 34, 55, 51, 52, 44, 51, 69, 64, 66, 55, 52, 61, 46, 31, 57, 52, 44, 18,
    41, 53, 55, 61, 51, 44];
    var costs = [.25, .27, .25, .25, .25, .25, .33, .31, .25, .29, .27, .22, .31, .25, .25, .33, .21, .25, .25, .25, .28, .25, .24, .22, .20, .25, .30, .25, .24, .25,
        .25, .25, .27, .25, .26, .29];


function index(array, arrayCost) {
  var maxScore = array[0];
  var minCostIndex = 0;
  
  // start iteration at 1, as 0 is already considered
  for (var i = 1; i < arrayCost.length; i++) {
    if (array[i] > maxScore) {
      maxScore = array[i];
      minCostIndex = i;
      continue
    }

    if (array[i] == maxScore && arrayCost[i] < arrayCost[minCostIndex]) {
      minCostIndex = i;
    }
  }

  return minCostIndex;
}

console.log("res", index(scores, costs));

Bizarre JavaScript Comparisons

Answer №1

Answer №2

Similar questions

Using jQuery's append function will retrieve external source files within script tags, however, it will not officially render them in the DOM

Manipulate your table data with jQuery: add, update, delete, and display information with ease

The modification of HTML styles

Mobile phone web development using HTML5

Creating a file logging system with console.log() in hapi.js

React Hook Form: Reset function triggers changes across all controllers on the page

Tips for updating an element in an array by using another element from a separate array

Transferring information to a subordinate view

Reduce the identification number within a JSON array following the removal of an item

Create a random word from a single string within the data in Nuxt.js

Simultaneously activate the 'onClick' and 'onClientClick' events on an ASP button using JavaScript

utilizing the .on method for dynamically inserted elements

Mocha Chai: Dive into an array of elements, discovering only fragments of the desired object

Creating a dynamic dropdown menu that changes based on the selection from another dropdown menu

The NodeJS module 'request' is producing symbols instead of expected HTML content

Is there a way to determine the model name programmatically within a Sails.js lifecycle callback?

Conceal the parent div from its sibling within the same parent container

Is there a method to modify the arrangement in which 3 specific HTML elements are displayed (i.e., their hierarchy in relation to one another)?

Discovering the presence of a NAN value within a JSON string

Include web browsing history