6 kyu
AD2070: Help Lorimar troubleshoot his robots-Search and Disable
146 of 236user8476848
Loading description...
Data Structures
Mathematics
Algorithms
Data Science
View
This comment has been reported as {{ abuseKindText }}.
Show
This comment has been hidden. You can view it now .
This comment can not be viewed.
- |
- Reply
- Edit
- View Solution
- Expand 1 Reply Expand {{ comments?.length }} replies
- Collapse
- Spoiler
- Remove
- Remove comment & replies
- Report
{{ fetchSolutionsError }}
-
-
Your rendered github-flavored markdown will appear here.
-
Label this discussion...
-
No Label
Keep the comment unlabeled if none of the below applies.
-
Issue
Use the issue label when reporting problems with the kata.
Be sure to explain the problem clearly and include the steps to reproduce. -
Suggestion
Use the suggestion label if you have feedback on how this kata can be improved.
-
Question
Use the question label if you have questions and/or need help solving the kata.
Don't forget to mention the language you're using, and mark as having spoiler if you include your solution.
-
No Label
- Cancel
Commenting is not allowed on this discussion
You cannot view this solution
There is no solution to show
Please sign in or sign up to leave a comment.
The description is bad. Instead of writing a useless story, explain the requirements in a much clearer way.
Nice kata of the series.
python new test framework is required. updated in this fork
In all languages, there should be tests whereby numbers go beyond 10K to invalidate solutions like this
Why the second log '2031 ...') (Count = 51) is 'no match continue' ? Why doesn't it match the pattern ??? Is it not the same as '1031 ...') (Count = 51) ? == 'match disable bot'
You forget to consider this
The count of the integer in the log is > 3
What does that mean? I count 51 times the integer 2031 in the log. 51 > 3 o_O
But
2031
is not a prime number ~~ ^_^(EDIT: I give the wrong reasoning about above reply, sorry ~~)
hehe yeah I forgot my prime check
This comment has been hidden.
This comment has been hidden.
Thank you for awesome kata, enjoyed it very much :)
Thank you Gabbek!
Finally, solved it. Although this is a process full of doubts ;-)
Nice work and solution. Now it's ready for approval...
Done :]
thank you!!
Well, i tried my solution again and... it didn't pass ! By reading the discussions, I found what was the trouble : this is the condition about "the integer must be Prime and appear more than 3 times in order to be counted". You described it, but you didn't implement this in the basic test cases, so it may occur than this case does not show up in the random tests (my code is the proof of that... :/ )
Hello again, I am not sure about this one maybe you can double check? The last basic test is the example from the description which does indeed test the condition you are describing(if I understand what you are describing correctly).
This comment has been hidden.
Fantastic! Thanks for the feedback and testing. I have implemented both basic tests. Your other solution is now invalid since it slipped past the randoms.
Hi !! ;-)
At the end of the description, you should correct :
Return "match disable BOT"
withReturn "match disable bot"
. This way, it will be coherent with test cases.++ (trying your last kata... ;) )
description updated... thank you! I am glad you are working on the other Kata... Thanks!
Hi again, the issue is resolved now, but i think there is a new one:
You can see, there are 90 hits (output of my counting, i looked into the output list too), so it's greater than 50 which means "match disable bot"?!
This comment has been hidden.
Can't see your hidden comment, if it is a comment for me;-)... Seems to me you solved it, so i'm wrong with my issue - i will look again into the description... Update: After posting my message i can read your hidden comment... So it's still an issue...
Hi the requirement is the integer must be Prime and appear more than 3 times in order to be counted. In the list Zozo uploaded there were 252 Primes but only one appears more than 3 times '1223' so this return is correct. I dont think it is an issue.
Ah ok, so thanks for your info, i misunderstood this single rule, your message is much clearer;-). Will try it soon again (and mark it as resolved now)...
Thank you Zozo and Smile I will try to make it a little clearer on the description. Very much appreciate the feedback.
I added an example on the description page please let me know if this makes it easier to understand. Thanks again for your help.
Hi, my code works now, so your message here was clear to me. I just read your new description after solving the kata, so the example is clear and it surely helps;-)! For me it would be ok to change one rule to "The count of EACH integer in the log is > 3", but the example probably is better... So thanks for your work:-)!
Thank you Smile!
I ranked it as 6kyu - maybe it's more 7kyu, but some steps are necessary to solve it (and some testing:-)), so hope it's ok for you;-)...
That is what I was thinking as well. Thanks again.
Same as smile67 :
search_disable_test
should be rewied !Hi, two solutions, don't know how...;-)? Seems to me your "submit- testcases" are wrong:
is_prime is not a function of my code. Later on same message with "Counter is not defined" appears... it's not in my code too;-). Haven't tested more, so please have a look;-).
Very strange it published and passed appears some of the code was wiped out on an edit. Will you please try again?
It's resolved thanks.