Master

[xonatialee]

No description provided.

[awdriggs]

So all your logic looks sounds, but the way you have the functions written they will only run once. The power of functions is that they are reusable!

I'm going to use your first one as an example...copied below. Then rewrite as a reusable function.

//finds max
var numbers = [16, 25, 4, 13, 57, 88, 102, 45];

var max = numbers[0];
for(var i = 1; i < numbers.length; i++) 
{
    if(max > numbers[i]){
        max = max; 
        i++; 
}
    if(numbers[i] > max){
        max = numbers[i];
        i++;
    }
}
var printMax = function(numbers, max){
    return printMax + " max is " + numbers;
}
console.log(printMax(max));

now look at your same code, just refactored.

//finds max


var findMax = function(numArray){
  var max = numbers[0];

  for(var i = 1; i < numbers.length; i++) {
    if(max > numbers[i]){
        max = max; 
        i++; 
  } else if(numbers[i] > max){
        max = numbers[i];
        i++;
    }
  }// end for loop

  return max; //this will send the max back anywhere that findMax is executed
}

var numbers = [16, 25, 4, 13, 57, 88, 102, 45];

console.log(findMax(numbers)); //here, we'll call the function we wrote with the numbers array

In this way, we can use findMax over and over again without writing any more logic!