April 8, 2015, 7:10 a.m. |
+1 |
Experience for voted comment psystyle: 1 on suic: Hi, that if is redundant: return sorted(first_word.lower()) != sorted(second_word.lower()) |
April 8, 2015, 7:10 a.m. |
+5 |
Experience for appreciated comment psystyle: 1 on suic: Hi, that if is redundant: return sorted(first_word.lower()) != sorted(second_word.lower()) |
April 8, 2015, 6:20 a.m. |
+5 |
Experience for appreciated comment hiya-m: 1 on suic: Hi, 1. there's also 'str.isalnum()`. 2. I would move the len(data) < 10 to the beginning, once data ... |
April 8, 2015, 6:20 a.m. |
+1 |
Experience for voted comment hiya-m: 1 on suic: Hi, 1. there's also 'str.isalnum()`. 2. I would move the len(data) < 10 to the beginning, once data ... |
April 8, 2015, 6:20 a.m. |
+1 |
Experience for voted comment hiya-m: 1 on suic: Hi, you can write: return filter(text.lower().__contains__, string.lowercase) == string.lowercase |
April 8, 2015, 6:20 a.m. |
+5 |
Experience for appreciated comment hiya-m: 1 on suic: Hi, you can write: return filter(text.lower().__contains__, string.lowercase) == string.lowercase |
April 8, 2015, 6:20 a.m. |
+5 |
Experience for appreciated comment hiya-m: 1 on suic: Hi, you can use `str.__contains__`: return len(filter(text.lower().__contains__, words)) # or sum() ... |
April 8, 2015, 6:20 a.m. |
+1 |
Experience for voted comment hiya-m: 1 on suic: Hi, you can use `str.__contains__`: return len(filter(text.lower().__contains__, words)) # or sum() ... |
April 8, 2015, 6:20 a.m. |
+5 |
Experience for appreciated comment hiya-m: 1 on suic: Hi, that `lambda` is redundant: return reduce(int.__mul__, map(int, str(number).replace("0", "1"))) |
April 8, 2015, 6:20 a.m. |
+1 |
Experience for voted comment hiya-m: 1 on suic: Hi, that `lambda` is redundant: return reduce(int.__mul__, map(int, str(number).replace("0", "1"))) |
April 8, 2015, 3:16 a.m. |
+5 |
Experience for appreciated comment hiya-m: 1 on suic: Hi, 1. You don't need enumerate as you don't use the `i` variable. 2. Why not list comprehension?: r... |
April 8, 2015, 3:16 a.m. |
+1 |
Experience for voted comment hiya-m: 1 on suic: Hi, 1. You don't need enumerate as you don't use the `i` variable. 2. Why not list comprehension?: r... |
April 8, 2015, 3:13 a.m. |
+5 |
Experience for appreciated comment hiya-m: 1 on suic: Hi, this map + lambda is overkill: return ",".join(phrases).replace("right", "left") |
April 8, 2015, 3:13 a.m. |
+1 |
Experience for voted comment hiya-m: 1 on suic: Hi, this map + lambda is overkill: return ",".join(phrases).replace("right", "left") |
April 7, 2015, 10:02 p.m. |
+1 |
Experience for voted comment lllz: 1 on suic: Hi, I just wanted to know, what had happened with MLB. I'm waiting for that cooler stuff. One commen... |
April 7, 2015, 8:53 p.m. |
+1 |
Experience for voted comment lllz: 1 on suic: Hi, we can ask @guido why `lambda` is `lambda` and not `function`. I think the most important part o... |
April 7, 2015, 2:55 p.m. |
+5 |
Experience for solved mission Fizz Buzz |
April 7, 2015, 2:46 p.m. |
+2 |
Experience for voted solution AndreyDanin: 1 on One-liner-suic |
April 7, 2015, 2:05 p.m. |
+2 |
Experience for voted solution Zazawie: 1 on Revisited KvaNTy's Straight and primitive-suic |
April 7, 2015, 10:33 a.m. |
+8 |
Experience for solved mission Count Inversions |
April 7, 2015, 7:08 a.m. |
+1 |
Experience for voted comment eugen.stegnij: 1 on suic: Hi, you can look for example at [this](http://anandology.com/python-practice-book/working-with-data.... |
April 7, 2015, 7:08 a.m. |
+5 |
Experience for appreciated comment eugen.stegnij: 1 on suic: Hi, you can look for example at [this](http://anandology.com/python-practice-book/working-with-data.... |
April 6, 2015, 3:08 p.m. |
+5 |
Experience for appreciated comment ntran93_64e7962746ad4a749b65c7: 1 on suic: Hi, first of all: You have to __return__ your result and __not print__. Two tips: 1. Don't use _sum_... |
April 6, 2015, 3:08 p.m. |
+1 |
Experience for voted comment ntran93_64e7962746ad4a749b65c7: 1 on suic: Hi, first of all: You have to __return__ your result and __not print__. Two tips: 1. Don't use _sum_... |
April 6, 2015, 8:28 a.m. |
+1 |
Experience for voted comment OMFGNuts: 1 on suic: Hi, sorry, but DIY, I told you everything you need to change. Next time you can ask, if somebody can... |
April 6, 2015, 3:17 a.m. |
+1 |
Experience for voted comment jodyd4964: 1 on suic: Hi, from top to bottom left to right: # The first if checks if n_row are n_col are valid indices. # ... |
April 5, 2015, 4:03 p.m. |
+1 |
Experience for voted comment eugen.stegnij: 1 on suic: Hi, forget about myList and myList1 outside the checkio function. These are _global variables_. (loo... |
April 5, 2015, 2:44 p.m. |
+1 |
Experience for voted solution plane: 1 on One-liner variation-suic |
April 5, 2015, 12:23 p.m. |
+5 |
Experience for appreciated comment yarmak_vladislav: 1 on suic: Hi, there's three time the same pattern on line 7, try to replace this: [acos((a**2 + c**2 - b**2)/(... |
April 5, 2015, 12:23 p.m. |
+2 |
Experience for voted comment yarmak_vladislav: 1 on suic: Hi, there's three time the same pattern on line 7, try to replace this: [acos((a**2 + c**2 - b**2)/(... |
April 4, 2015, 9:53 p.m. |
+5 |
Experience for appreciated comment iken: 1 on suic: Hi, the if...else is redundant. |
April 4, 2015, 9:53 p.m. |
+2 |
Experience for voted comment iken: 1 on suic: Hi, the if...else is redundant. |
April 4, 2015, 6:06 p.m. |
+5 |
Experience for appreciated comment IgorSmirnovTW: 1 on suic: Hi, I suppose, you run that code with Python3.3: # / is true division in Python 3 but integer divisi... |
April 4, 2015, 6:06 p.m. |
+1 |
Experience for voted comment IgorSmirnovTW: 1 on suic: Hi, I suppose, you run that code with Python3.3: # / is true division in Python 3 but integer divisi... |
April 4, 2015, 1:26 p.m. |
+1 |
Experience for voted comment Bharat_Kalluri: 1 on suic: Hi, there are basically three problems: 1. Line 16 checks if there are at least __2__ digits, upper ... |
April 4, 2015, 1:26 p.m. |
+5 |
Experience for appreciated comment Bharat_Kalluri: 1 on suic: Hi, there are basically three problems: 1. Line 16 checks if there are at least __2__ digits, upper ... |
April 4, 2015, 8:11 a.m. |
+5 |
Experience for appreciated comment studiosi: 1 on suic: [RTFD](http://en.wikipedia.org/wiki/RTFM) where D is Description :D: > Output: The most frequent let... |
April 4, 2015, 8:11 a.m. |
+1 |
Experience for voted comment studiosi: 1 on suic: [RTFD](http://en.wikipedia.org/wiki/RTFM) where D is Description :D: > Output: The most frequent let... |
April 3, 2015, 10:41 p.m. |
+5 |
Experience for appreciated comment DanielHendrycks: 1 on suic: Hi, in task description there is: > string of words separated by commas in __alphabetic order__ (emp... |
April 3, 2015, 10:41 p.m. |
+1 |
Experience for voted comment DanielHendrycks: 1 on suic: Hi, in task description there is: > string of words separated by commas in __alphabetic order__ (emp... |
April 3, 2015, 7:39 p.m. |
+1 |
Experience for voted comment hanpari: 1 on suic: Hi, `@` e. g. `@hanpari` is @hanpari. |
April 3, 2015, 1:15 p.m. |
+1 |
Experience for voted comment OMFGNuts: 1 on suic: Hi, 1. You have to __return__ your solution and __not print__. Your code returns a value only in `el... |
April 3, 2015, 10:16 a.m. |
+5 |
Experience for appreciated comment alexandr.martseniuk: 1 on suic: Hi, your code does not handle the case of empty list. Empty list does not have last element :) Try: ... |
April 3, 2015, 10:16 a.m. |
+1 |
Experience for voted comment alexandr.martseniuk: 1 on suic: Hi, your code does not handle the case of empty list. Empty list does not have last element :) Try: ... |
April 3, 2015, 9:06 a.m. |
+1 |
Experience for voted comment bellot.hamza: 1 on suic: Hi, your code does not handle the case of empty list. Empty list does not have last element :) Try: ... |
April 3, 2015, 9:06 a.m. |
+5 |
Experience for appreciated comment bellot.hamza: 1 on suic: Hi, your code does not handle the case of empty list. Empty list does not have last element :) Try: ... |
April 2, 2015, 6:33 p.m. |
+1 |
Experience for voted comment Overkane_7f86a49b2e264eaa89e21: 1 on suic: Hi, if you know about extended slices why don't you use them? Don't use _sum_ as variable name _sum(... |
April 2, 2015, 6:33 p.m. |
+5 |
Experience for appreciated comment Overkane_7f86a49b2e264eaa89e21: 1 on suic: Hi, if you know about extended slices why don't you use them? Don't use _sum_ as variable name _sum(... |
April 2, 2015, 10:25 a.m. |
+5 |
Experience for appreciated comment Vesagran: 1 on suic: Hi, 1. _result_ variable is redundant also [] in sorted. 2. Lines 5-7 are quite repetitive. You coul... |
April 2, 2015, 10:25 a.m. |
+2 |
Experience for voted comment Vesagran: 1 on suic: Hi, 1. _result_ variable is redundant also [] in sorted. 2. Lines 5-7 are quite repetitive. You coul... |
April 1, 2015, 12:19 a.m. |
+1 |
Experience for voted comment ammark1111: 1 on suic: Hi, I suppose you use Python 3.3 and you see this: > TypeError: list indices must be integers, not f... |
April 1, 2015, 12:19 a.m. |
+5 |
Experience for appreciated comment ammark1111: 1 on suic: Hi, I suppose you use Python 3.3 and you see this: > TypeError: list indices must be integers, not f... |
March 31, 2015, 7:43 p.m. |
+30 |
Experience for solved mission Texas Referee |
March 31, 2015, 1:05 p.m. |
+1 |
Experience for voted comment Qandak: 1 on suic: Hi, you don't the last if...else: return i >= 3 |
March 31, 2015, 1:05 p.m. |
+5 |
Experience for appreciated comment Qandak: 1 on suic: Hi, you don't the last if...else: return i >= 3 |
March 31, 2015, 12:51 p.m. |
+5 |
Experience for appreciated comment Qandak: 1 on suic: The important fact is, that you realized it :) |
March 31, 2015, 12:51 p.m. |
+1 |
Experience for voted comment Qandak: 1 on suic: The important fact is, that you realized it :) |
March 31, 2015, 12:20 p.m. |
+1 |
Experience for voted comment MMM_AAA_NNN: 1 on suic: Hi, a tip: indent your comments. |
March 31, 2015, 12:20 p.m. |
+5 |
Experience for appreciated comment MMM_AAA_NNN: 1 on suic: Hi, a tip: indent your comments. |
March 31, 2015, 12:19 p.m. |
+5 |
Experience for appreciated comment MMM_AAA_NNN: 1 on suic: Hi, straightforward... hmmm... but not pythonic: Look at extended slices and _sum()_ built-in functi... |
March 31, 2015, 12:19 p.m. |
+1 |
Experience for voted comment MMM_AAA_NNN: 1 on suic: Hi, straightforward... hmmm... but not pythonic: Look at extended slices and _sum()_ built-in functi... |
March 30, 2015, 10:42 p.m. |
+1 |
Experience for voted comment Peter.White: 1 on suic: If you want to follow somebody, you have to go to his/her profile first, there is a _Follow_ button ... |
March 30, 2015, 10:42 p.m. |
+5 |
Experience for appreciated comment Peter.White: 1 on suic: If you want to follow somebody, you have to go to his/her profile first, there is a _Follow_ button ... |
March 30, 2015, 10:24 p.m. |
+1 |
Experience for voted comment Peter.White: 1 on suic: You can try to disassemble it, with [dis](https://docs.python.org/2.7/library/dis.html?highlight=dis... |
March 30, 2015, 8:59 p.m. |
+5 |
Experience for appreciated comment EvanRavenelle: 1 on suic: Hi, 1. lines 9, 11 are redundant. _filter()_ and Counter() can do the job for you. 2. You can replac... |
March 30, 2015, 8:59 p.m. |
+1 |
Experience for voted comment EvanRavenelle: 1 on suic: Hi, 1. lines 9, 11 are redundant. _filter()_ and Counter() can do the job for you. 2. You can replac... |
March 30, 2015, 8:56 p.m. |
+5 |
Experience for appreciated comment EvanRavenelle: 1 on suic: Hi, you don't need the if: return len(data) >= 10 and all(...) You can also extract the pattern you'... |
March 30, 2015, 8:56 p.m. |
+1 |
Experience for voted comment EvanRavenelle: 1 on suic: Hi, you don't need the if: return len(data) >= 10 and all(...) You can also extract the pattern you'... |
March 30, 2015, 8:08 p.m. |
+1 |
Experience for voted comment Peter.White: 1 on suic: What about len(set(text.lower)) == 26? |
March 30, 2015, 12:50 p.m. |
+1 |
Experience for voted comment mcsean: 1 on suic: Hi, 1. You can have nested generator expressions/list comprehensions and there's the _all()_ built-i... |
March 30, 2015, 12:50 p.m. |
+5 |
Experience for appreciated comment mcsean: 1 on suic: Hi, 1. You can have nested generator expressions/list comprehensions and there's the _all()_ built-i... |
March 30, 2015, 12:06 p.m. |
+1 |
Experience for voted comment yutkin: 1 on suic: Hi, the == [] is redundant. You can write _not array_ instead, or: return array[-1] * sum([array[i] ... |
March 30, 2015, 12:06 p.m. |
+5 |
Experience for appreciated comment yutkin: 1 on suic: Hi, the == [] is redundant. You can write _not array_ instead, or: return array[-1] * sum([array[i] ... |
March 30, 2015, 5:14 a.m. |
+1 |
Experience for voted comment bryukh: 1 on suic: Hi, forget about myList and myList1 outside the checkio function. These are _global variables_. (loo... |
March 30, 2015, 5:13 a.m. |
+1 |
Experience for voted comment bryukh: 1 on suic: Hi, it should be checkio() and not ExecFunction and you have to return your result and not print: de... |
March 29, 2015, 1:59 a.m. |
+5 |
Experience for appreciated comment nicolasmartinlazo: 1 on suic: Hi, there's a redundant bracked on line 9: v return game_result[[0][0] |
March 29, 2015, 1:59 a.m. |
+1 |
Experience for voted comment nicolasmartinlazo: 1 on suic: Hi, there's a redundant bracked on line 9: v return game_result[[0][0] |
March 27, 2015, 8:31 p.m. |
+1 |
Experience for voted comment opritche: 1 on suic: Hi, look at _filter()_ built-in function. |
March 27, 2015, 8:31 p.m. |
+5 |
Experience for appreciated comment opritche: 1 on suic: Hi, look at _filter()_ built-in function. |
March 27, 2015, 8:30 p.m. |
+5 |
Experience for appreciated comment opritche: 1 on suic: Hi, instead of first for loop you could use _str.lower()_ method. Lines 6-11 you can replace with a ... |
March 27, 2015, 8:30 p.m. |
+1 |
Experience for voted comment opritche: 1 on suic: Hi, instead of first for loop you could use _str.lower()_ method. Lines 6-11 you can replace with a ... |
March 27, 2015, 8:30 p.m. |
+5 |
Experience for appreciated comment opritche: 1 on suic: Hi, look at _sum()_ built-in function. |
March 27, 2015, 8:30 p.m. |
+1 |
Experience for voted comment opritche: 1 on suic: Hi, look at _sum()_ built-in function. |
March 27, 2015, 8:29 p.m. |
+1 |
Experience for voted comment opritche: 1 on suic: Hi, instead of len(array) == 0 you can write: if not array: return 0 |
March 27, 2015, 8:29 p.m. |
+5 |
Experience for appreciated comment opritche: 1 on suic: Hi, instead of len(array) == 0 you can write: if not array: return 0 |
March 27, 2015, 7:09 p.m. |
+5 |
Experience for appreciated comment capybara23: 1 on suic: Hi, I'm glad to hear that my feedback is helpful :) |
March 27, 2015, 7:09 p.m. |
+1 |
Experience for voted comment capybara23: 1 on suic: Hi, I'm glad to hear that my feedback is helpful :) |
March 27, 2015, 3:49 p.m. |
+1 |
Experience for voted comment eB_s3000: 1 on suic: Hi, sort, print, tuple(), len(args) > 0, a and b are redundant: 1. You don't use sort, so why do you... |
March 27, 2015, 3:49 p.m. |
+5 |
Experience for appreciated comment eB_s3000: 1 on suic: Hi, sort, print, tuple(), len(args) > 0, a and b are redundant: 1. You don't use sort, so why do you... |
March 27, 2015, 3:47 p.m. |
+1 |
Experience for voted comment eB_s3000: 1 on suic: Hi, 1. the last if is redundant: return sorted(some2) == sorted(some1) 2. You do same things twice (... |
March 27, 2015, 3:47 p.m. |
+5 |
Experience for appreciated comment eB_s3000: 1 on suic: Hi, 1. the last if is redundant: return sorted(some2) == sorted(some1) 2. You do same things twice (... |
March 27, 2015, 2:24 p.m. |
+1 |
Experience for voted comment Peter.White: 1 on suic: In fact my point wasn't about category :) It reminded me on code I often see at work, when people us... |
March 27, 2015, 2:24 p.m. |
+5 |
Experience for appreciated comment Peter.White: 1 on suic: In fact my point wasn't about category :) It reminded me on code I often see at work, when people us... |
March 27, 2015, 2:15 p.m. |
+1 |
Experience for voted comment Peter.White: 1 on suic: Hi, consider the fact, that __str__ is immutable in Python. And look at _filter()_ function or _list... |
March 27, 2015, 2:15 p.m. |
+5 |
Experience for appreciated comment Peter.White: 1 on suic: Hi, consider the fact, that __str__ is immutable in Python. And look at _filter()_ function or _list... |
March 27, 2015, 2:15 p.m. |
+1 |
Experience for voted comment Peter.White: 1 on suic: message += c always creates a new string. as message is immutable. |
March 27, 2015, 2:15 p.m. |
+5 |
Experience for appreciated comment Peter.White: 1 on suic: message += c always creates a new string. as message is immutable. |
March 27, 2015, 2:11 p.m. |
+1 |
Experience for voted comment Peter.White: 1 on suic: Hi, you can write: count += chunk.isalpha() # instead of: count = count+1 if chunk.isalpha() else 0 |
March 27, 2015, 2:11 p.m. |
+5 |
Experience for appreciated comment Peter.White: 1 on suic: Hi, you can write: count += chunk.isalpha() # instead of: count = count+1 if chunk.isalpha() else 0 |
March 27, 2015, 2:05 p.m. |
+5 |
Experience for appreciated comment Peter.White: 1 on suic: Hi, instead of lines 2, 4-6 you can write this: return sum(word in ltext for word in words) |