Adding __delitem__ to dict#215
Uh oh!
There was an error while loading. Please reload this page.
Conversation
Codecov ReportBase: 74.35% // Head: 74.38% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@## main #215 +/- ## ========================================== + Coverage 74.35% 74.38% +0.03% ========================================== Files 76 76 Lines 12617 12625 +8 ========================================== + Hits 9381 9391 +10 + Misses 2563 2562 -1 + Partials 673 672 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
sbinet left a comment
sbinet left a comment There was a problem hiding this comment.
thanks for the PR.
I have 2 comments.
see below.
py/dict.go Outdated
| str, ok:=key.(String) | ||
| ifok{ | ||
| _, ok:=d[string(str)] | ||
| ifok{ | ||
| delete(d, string(str)) | ||
| returnNone, nil | ||
| } | ||
| } | ||
| returnnil, ExceptionNewf(KeyError, "%v", key) |
There was a problem hiding this comment.
| str, ok:=key.(String) | |
| ifok{ | |
| _, ok:=d[string(str)] | |
| ifok{ | |
| delete(d, string(str)) | |
| returnNone, nil | |
| } | |
| } | |
| returnnil, ExceptionNewf(KeyError, "%v", key) | |
| str, ok:=key.(String) | |
| if!ok{ | |
| returnnil, ExceptionNewf(KeyError, "%v", key) | |
| } | |
| _, ok=d[string(str)] | |
| if!ok{ | |
| returnnil, ExceptionNewf(KeyError, "%v", key) | |
| } | |
| delete(d, string(str)) | |
| returnNone, nil |
yes, it's longer, but only in the vertical dimension (however, it's less indented)
| dela["hello"] | ||
| assertnota.__contains__('hello') | ||
| asserta.__contains__('hi') | ||
There was a problem hiding this comment.
perhaps add a couple of cases that exercize the 2 ExceptionNewf branches ?
2 participants
Enables the 'del' command to delete items from a dict