the encryption dict should contain tuples as values, not lists
the encryption dict should be imported explicitely
in the tests (sample at least): the solution should imported explicitely
no fixed tests in the full test suite
2 random tests aren't enough => build a loop and do like 50-100 of them
the tests shouldn't log anything to the console
The last three points must be applied to all languages.
General:
...well... Sorry to say that, but the description asbolutely doesn't make any sense to me.
"the encryption dict is the card". But a card isn't a dict. => what is encoded and how?
"Pushing zone replaced with array "A" and "B"." -> wut? where? what zone A? what zone B?
"To read the symbol, you need to replace the corresponding letters and/or numbers with an empty string " " (aka hole)." -> WUT!? read where? from what?
"If the whole character row is in its place, it is a space in the string." -> WHERE!?
"pushing zone" -> not explained
"digit punching" -> not explained
The description is describing stuff, but nothing is self explanatory. Especially when there are two methods involved (encryption+decryption) while it's never said when/where those are explained (and I bet they are mixed in the description)
I finally understood (I believe) what I'm supposed to do, but in this case, I think one part of the description is wrong: If the whole character row is in its place, it is a space in the string. isn't that rather a column instead of a row?
btw:
the content of the encryption dict (preloaded) isn't explained
when I look at the version in the sample tests, I have no clue what it means, since nothing has been explained.
As you can see, the description needs a massive rethinking (but not necesseraly a lot of rewriting, I guess. Just a push here and there, with a proper structure might do)
Right. Now ok.
I can see it just fine. Maybe your ISP or firewall is blocking the domain name ?
image is broken in the description.
This is the most elegant solution. It should be on top.
Hi,
In python:
The last three points must be applied to all languages.
General:
The description is describing stuff, but nothing is self explanatory. Especially when there are two methods involved (encryption+decryption) while it's never said when/where those are explained (and I bet they are mixed in the description)
I finally understood (I believe) what I'm supposed to do, but in this case, I think one part of the description is wrong:
If the whole character row is in its place, it is a space in the string.
isn't that rather a column instead of a row?btw:
As you can see, the description needs a massive rethinking (but not necesseraly a lot of rewriting, I guess. Just a push here and there, with a proper structure might do)
Cheers
I don't see a change. Did you re-publish?
Fixed
Python should use snake_case naming convention, not camelCase.
Fixed
Preloaded
encryption
is typed as[String : [String]]
, when should be[Character : [Character]]
.Similar, all
card
s should be[[Character]]
instead of[[String]]
.No random tests
Duplicate of existing LCM and GCD kata.