trjhtr

If you were only permitted to complete at most one transaction (i.e., buy one and sell one share of the stock), design an algorithm to find the maximum profit.

Example one:
Input: [7,1,5,3,6,4]
Output: 5
Explanation: Buy on day 2 (price = 1) and sell on day 5 (price = 6), profit = 6-1 = 5. Not 7-1 = 6, as selling price needs to be larger than buying price.

Example two:
Input: [7,6,4,3,1]
Output: 0
Explanation: In this case, no transaction is done, i.e. max profit = 0.

My solution:

walked the array grabbing the initial price and continued to swap it with the smallest value. Between the current smallest value < and only the proceeding larger peaks. With the solution being the largest chasms (difference) between the lowest peak and highest summit in value within an interval. TimeComplexity is O(n) space complexity is O(1) only using one object.

const perfectPrice = (prices, peak = 
 buy: null,
 buy_day: null,
 sell_day: null,
 sell: null,
 profit: 0,
 profitable: 
) => 


console.log("(test 1):", perfectPrice([7, 1, 5, 3, 6, 4]))
console.log("(test 2):", perfectPrice([7, 6, 4, 3, 1]))
console.log("(test 3):", perfectPrice([7, 1, 5, 3, 6, 4]))
console.log("(test 4):", perfectPrice([1, 2]))
console.log("(test 5):", perfectPrice([7, 1, 5, 3, 6, 4]))
console.log("(test 6):", perfectPrice([2, 9, 5, 3, 1, 4]))

.as-console-wrapper 
 max-height: 100% !important;
 top: 0;

Readability

• The way you've named your variables makes the code more difficult to read for example peak just doesn't make sense to me. Another consideration would be to rename buy_day to cheapest_day
  


• Don't overcomplicate things: I found the return statement at the end of your forEach confusing

General syntax improvement that would make the code easier to read:

• Use camelCase over snake_case


• Use dot notation when accessing an objects properties where possible

Remove unnecessary/unused code

• You have an if/elseif that does the same thing this can be simplified using an or statement if (this || that)
  


• 
  peak.sell and peak.sell_day are never used, same for all in your prices.forEach
  

Misc

There may be some reasoning you had which I'm missing but I did't see the point in accepting the second argument peak.

Rewrite

const perfectPrice = (prices) => ;


console.log("(test 1):", perfectPrice([7, 1, 5, 3, 6, 4]))
console.log("(test 2):", perfectPrice([7, 6, 4, 3, 1]))
console.log("(test 3):", perfectPrice([7, 1, 5, 3, 6, 4]))
console.log("(test 4):", perfectPrice([1, 2]))
console.log("(test 5):", perfectPrice([7, 1, 5, 3, 6, 4]))
console.log("(test 6):", perfectPrice([2, 9, 5, 3, 1, 4]))