Timeline for Stacks: Balanced Brackets Hacker Rank
Current License: CC BY-SA 3.0
10 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Feb 16, 2018 at 14:48 | comment | added | Xiaoy312 | @HenrikHansen Good catch. I can't believe I made a such mistake. | |
| Feb 16, 2018 at 14:41 | history | edited | Xiaoy312 | CC BY-SA 3.0 |
Fixed issue with trailing brackets
|
| Feb 16, 2018 at 12:34 | comment | added | user73941 |
@Xiaoy312: OK, about the non-bracket chars. Another problem is that your solution can't handle a pattern like "([]())(" correctly, because you don't check for an empty stack at the end of the IsBalanced() {...} method.
|
|
| Feb 15, 2018 at 18:09 | history | edited | Xiaoy312 | CC BY-SA 3.0 |
code fix
|
| Feb 15, 2018 at 15:02 | comment | added | Xiaoy312 | @HenrikHansen It is there so avoid checking for non-bracket characters in the for-loop, and validation for the sake of validation. Now, that I look at it, it does seem ill-fitted to consider presence of other characters as exception. | |
| Feb 15, 2018 at 8:39 | vote | accept | Gilad | ||
| Feb 15, 2018 at 8:39 | comment | added | Gilad | @Xiaoy312 Thank you so much for a great review | |
| Feb 15, 2018 at 4:18 | comment | added | user73941 | Why do you only allow brackets in the expressions? | |
| Feb 14, 2018 at 23:28 | history | edited | t3chb0t | CC BY-SA 3.0 |
formatting corrections
|
| Feb 14, 2018 at 22:57 | history | answered | Xiaoy312 | CC BY-SA 3.0 |