Skip to content

completed mock 10 - #459

Open
meetrpatels30 wants to merge 1 commit into
super30admin:masterfrom
meetrpatels30:master
Open

completed mock 10#459
meetrpatels30 wants to merge 1 commit into
super30admin:masterfrom
meetrpatels30:master

Conversation

@meetrpatels30

Copy link
Copy Markdown

No description provided.

@super30admin

Copy link
Copy Markdown
Owner
  • Strengths:
    • The solution is optimal in both time and space complexity.
    • The logic is straightforward and easy to understand.
    • The code is clean and well-commented.
  • Areas for Improvement:
    • While the solution is correct and efficient, it might be helpful to mention why this approach works (i.e., because you can capture all upward movements in the price series).
    • The variable n is not strictly necessary since you can directly use prices.length in the loop condition, but this is a minor point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants