Nov. 4, 2015, 9:39 a.m. |
+1 |
Experience for voted comment ermichin158: 1 on suic: Hi, `listToString` is reinventing the wheel. Look at [_str.join_](https://docs.python.org/3.5/librar... |
Nov. 4, 2015, 9:39 a.m. |
+5 |
Experience for appreciated comment ermichin158: 1 on suic: Hi, this: string = "" for smbNum in str_number: string = smbNum + string; is slow and not pythonic: ... |
Nov. 4, 2015, 9:39 a.m. |
+1 |
Experience for voted comment ermichin158: 1 on suic: Hi, this: string = "" for smbNum in str_number: string = smbNum + string; is slow and not pythonic: ... |
Nov. 4, 2015, 9:38 a.m. |
+1 |
Experience for voted comment ermichin158: 1 on suic: Hi, one of the most important and powerful feature of Python is `in`: for word in words: now = 0; wo... |
Nov. 4, 2015, 9:38 a.m. |
+5 |
Experience for appreciated comment ermichin158: 1 on suic: Hi, one of the most important and powerful feature of Python is `in`: for word in words: now = 0; wo... |
Nov. 4, 2015, 9:37 a.m. |
+1 |
Experience for voted comment ermichin158: 1 on suic: Hi, 1. Lines 10-11 are redundant. Look at the _abs()_ built-in. 2. Line 7 is redundant. It does noth... |
Nov. 4, 2015, 9:37 a.m. |
+5 |
Experience for appreciated comment ermichin158: 1 on suic: Hi, 1. Lines 10-11 are redundant. Look at the _abs()_ built-in. 2. Line 7 is redundant. It does noth... |
Nov. 4, 2015, 9:35 a.m. |
+5 |
Experience for appreciated comment ermichin158: 1 on suic: Hi, _inArray_ is reinventing the wheel. Use `in` instead: if other not in safe: safe.append(other) |
Nov. 4, 2015, 9:35 a.m. |
+1 |
Experience for voted comment ermichin158: 1 on suic: Hi, _inArray_ is reinventing the wheel. Use `in` instead: if other not in safe: safe.append(other) |
Nov. 4, 2015, 9:34 a.m. |
+5 |
Experience for appreciated comment ermichin158: 1 on suic: Hi, you could write `return sorted([A, B, C])`. |
Nov. 4, 2015, 9:34 a.m. |
+1 |
Experience for voted comment ermichin158: 1 on suic: Hi, you could write `return sorted([A, B, C])`. |
Nov. 4, 2015, 9:29 a.m. |
+1 |
Experience for voted comment ermichin158: 1 on suic: I guess, the _reason_ behind this is: `C-style code => C like speed` :D |
Nov. 4, 2015, 9:29 a.m. |
+5 |
Experience for appreciated comment ermichin158: 1 on suic: I guess, the _reason_ behind this is: `C-style code => C like speed` :D |
Nov. 2, 2015, 10:22 p.m. |
+5 |
Experience for appreciated comment Marpop: 1 on suic: Hi, look at [_sum()_](https://docs.python.org/3.5/library/functions.html?highlight=sum#sum) built-in... |
Nov. 2, 2015, 10:22 p.m. |
+1 |
Experience for voted comment Marpop: 1 on suic: Hi, look at [_sum()_](https://docs.python.org/3.5/library/functions.html?highlight=sum#sum) built-in... |
Nov. 2, 2015, 10:15 p.m. |
+5 |
Experience for appreciated comment Marpop: 1 on suic: Hi, 1. Look at [_list comprehensions_](https://docs.python.org/3.5/howto/functional.html?highlight=l... |
Nov. 2, 2015, 10:15 p.m. |
+1 |
Experience for voted comment Marpop: 1 on suic: Hi, 1. Look at [_list comprehensions_](https://docs.python.org/3.5/howto/functional.html?highlight=l... |
Nov. 2, 2015, 10:10 p.m. |
+5 |
Experience for appreciated comment Marpop: 1 on suic: Hi, 1. that `else` branch is redundant: 2. Why do you compute the result len(args) times? if not arg... |
Nov. 2, 2015, 10:10 p.m. |
+1 |
Experience for voted comment Marpop: 1 on suic: Hi, 1. that `else` branch is redundant: 2. Why do you compute the result len(args) times? if not arg... |
Nov. 2, 2015, 10:08 p.m. |
+5 |
Experience for appreciated comment Marpop: 1 on suic: Hi, 1. `&` stands for [_set intersection_](https://docs.python.org/3.5/library/stdtypes.html?highlig... |
Nov. 2, 2015, 10:08 p.m. |
+1 |
Experience for voted comment Marpop: 1 on suic: Hi, 1. `&` stands for [_set intersection_](https://docs.python.org/3.5/library/stdtypes.html?highlig... |
Nov. 2, 2015, 8:05 p.m. |
+1 |
Experience for voted comment Marpop: 1 on suic: Hi, instead of `alphabet` you could use `string.ascii_lowercase`. |
Nov. 2, 2015, 8:05 p.m. |
+5 |
Experience for appreciated comment Marpop: 1 on suic: Hi, instead of `alphabet` you could use `string.ascii_lowercase`. |
Nov. 2, 2015, 5:44 p.m. |
+5 |
Experience for appreciated comment Alexona: 1 on suic: Hi, _bin()_ returns a __str__, which has _str.count()_ method: return bin(number).count('1') # That'... |
Nov. 2, 2015, 5:44 p.m. |
+1 |
Experience for voted comment Alexona: 1 on suic: Hi, _bin()_ returns a __str__, which has _str.count()_ method: return bin(number).count('1') # That'... |
Nov. 2, 2015, 5:43 p.m. |
+1 |
Experience for voted comment Alexona: 1 on suic: Hi, look at: 1. [_fractions.gcd()_](https://docs.python.org/3.5/library/fractions.html?highlight=fra... |
Nov. 2, 2015, 5:43 p.m. |
+5 |
Experience for appreciated comment Alexona: 1 on suic: Hi, look at: 1. [_fractions.gcd()_](https://docs.python.org/3.5/library/fractions.html?highlight=fra... |
Nov. 2, 2015, 5:41 p.m. |
+1 |
Experience for voted comment Alexona: 1 on suic: Hi, 1. this `text.lower().find(l) == -1` is very un-pythonic. One of the most useful and powerful th... |
Nov. 2, 2015, 5:41 p.m. |
+5 |
Experience for appreciated comment Alexona: 1 on suic: Hi, 1. this `text.lower().find(l) == -1` is very un-pythonic. One of the most useful and powerful th... |
Nov. 2, 2015, 5:24 p.m. |
+1 |
Experience for voted comment Alexona: 1 on suic: Hi, look at 1. [__collections.Counter__](https://docs.python.org/3.5/library/collections.html?highli... |
Nov. 2, 2015, 5:24 p.m. |
+5 |
Experience for appreciated comment Alexona: 1 on suic: Hi, look at 1. [__collections.Counter__](https://docs.python.org/3.5/library/collections.html?highli... |
Oct. 29, 2015, 9:49 a.m. |
+1 |
Experience for voted comment Melchiore: 1 on suic: Can you please rate my comment (with little + and - next to the zero in comment header)? (I already ... |
Oct. 29, 2015, 9:49 a.m. |
+5 |
Experience for appreciated comment Melchiore: 1 on suic: Can you please rate my comment (with little + and - next to the zero in comment header)? (I already ... |
Oct. 29, 2015, 9:49 a.m. |
+1 |
Experience for voted comment Melchiore: 1 on suic: Hi, from top to bottom left to right: # The first if checks if n_row are n_col are valid indices. # ... |
Oct. 27, 2015, 12:15 p.m. |
+5 |
Experience for appreciated comment abha_kabra: 1 on suic: Hi, a little code review: 1. Lines 6-8 are redundant. 2. Parentheses on line 2 and 3 are redundant: ... |
Oct. 27, 2015, 12:15 p.m. |
+1 |
Experience for voted comment abha_kabra: 1 on suic: Hi, a little code review: 1. Lines 6-8 are redundant. 2. Parentheses on line 2 and 3 are redundant: ... |
Oct. 25, 2015, 6:27 a.m. |
+1 |
Experience for voted solution datalin: 1 on One-liner variation-suic |
Oct. 21, 2015, 12:18 a.m. |
+1 |
Experience for voted comment aledruetta: 1 on suic: You don't need the if ... else at all: def checkio(array): return len(array) and sum(array[::2]) * a... |
Oct. 20, 2015, 11:26 a.m. |
+1 |
Experience for voted solution james.shi.ch: 1 on First-suic |
Oct. 18, 2015, 10:07 a.m. |
+1 |
Experience for voted comment inga.s: 1 on suic: Hi, `1` is the _neutral element of multiplication_ lines 8 and 9 are redundant (`total * 1 == total`... |
Oct. 18, 2015, 10:07 a.m. |
+5 |
Experience for appreciated comment inga.s: 1 on suic: Hi, `1` is the _neutral element of multiplication_ lines 8 and 9 are redundant (`total * 1 == total`... |
Oct. 18, 2015, 10:06 a.m. |
+5 |
Experience for appreciated comment inga.s: 1 on suic: Hi, __bool__ is a subtype of __int__, therefore the nested `if`s are redundant, e. g.: elif operatio... |
Oct. 18, 2015, 10:06 a.m. |
+1 |
Experience for voted comment inga.s: 1 on suic: Hi, __bool__ is a subtype of __int__, therefore the nested `if`s are redundant, e. g.: elif operatio... |
Oct. 18, 2015, 10:06 a.m. |
+5 |
Experience for appreciated comment inga.s: 1 on suic: Hi, 1. `a`, `b` and `result` are redundant: `return max(args) - min(args)` is enough. 2. Same with l... |
Oct. 18, 2015, 10:06 a.m. |
+1 |
Experience for voted comment inga.s: 1 on suic: Hi, 1. `a`, `b` and `result` are redundant: `return max(args) - min(args)` is enough. 2. Same with l... |
Oct. 18, 2015, 10:05 a.m. |
+5 |
Experience for appreciated comment inga.s: 1 on suic: Hi, look at _list comprehensions_ e. g.: message = [x for x in text if x.isupper()] # or the filter(... |
Oct. 18, 2015, 10:05 a.m. |
+1 |
Experience for voted comment inga.s: 1 on suic: Hi, look at _list comprehensions_ e. g.: message = [x for x in text if x.isupper()] # or the filter(... |
Oct. 18, 2015, 10:04 a.m. |
+5 |
Experience for appreciated comment inga.s: 1 on suic: Hi, 1. Line 9 is never reached => redundant. 2. `if` on line 12 is redundant. |
Oct. 18, 2015, 10:04 a.m. |
+1 |
Experience for voted comment inga.s: 1 on suic: Hi, 1. Line 9 is never reached => redundant. 2. `if` on line 12 is redundant. |
Oct. 18, 2015, 10:04 a.m. |
+5 |
Experience for appreciated comment inga.s: 1 on suic: Hi, 1. `word` is an __int__ so `return word` is enough. 2. You could use _generator expression_ and ... |
Oct. 18, 2015, 10:04 a.m. |
+1 |
Experience for voted comment inga.s: 1 on suic: Hi, 1. `word` is an __int__ so `return word` is enough. 2. You could use _generator expression_ and ... |
Oct. 18, 2015, 10:04 a.m. |
+5 |
Experience for appreciated comment inga.s: 1 on suic: Hi, 1. You could import `ascii_lowercase` from `string` module instead of typing the alphabet. 2. Lo... |
Oct. 18, 2015, 10:04 a.m. |
+1 |
Experience for voted comment inga.s: 1 on suic: Hi, 1. You could import `ascii_lowercase` from `string` module instead of typing the alphabet. 2. Lo... |
Oct. 18, 2015, 10:02 a.m. |
+5 |
Experience for appreciated comment inga.s: 1 on suic: Hi, line 9 is redundant as it's never reached. |
Oct. 18, 2015, 10:02 a.m. |
+1 |
Experience for voted comment inga.s: 1 on suic: Hi, line 9 is redundant as it's never reached. |
Oct. 18, 2015, 10:01 a.m. |
+5 |
Experience for appreciated comment inga.s: 1 on suic: Hi, look at [_extended slices_](https://docs.python.org/3/whatsnew/2.3.html?highlight=extended%20sli... |
Oct. 18, 2015, 10:01 a.m. |
+1 |
Experience for voted comment inga.s: 1 on suic: Hi, look at [_extended slices_](https://docs.python.org/3/whatsnew/2.3.html?highlight=extended%20sli... |
Oct. 17, 2015, 7:22 p.m. |
+1 |
Experience for voted comment PyPoet: 1 on suic: Answer is Python 2.x :) In Python 2.x `/` is _integer division_ not _true division_. Fortunately in ... |
Oct. 17, 2015, 7:19 p.m. |
+1 |
Experience for voted comment PyPoet: 1 on suic: Or even: datetime.date(*date1) # :) |
Oct. 15, 2015, 3:41 p.m. |
+5 |
Experience for appreciated comment killi: 1 on suic: Hi, this is typical _global variable_ issue. Read [this](http://www.checkio.org/forum/post/3269/6-th... |
Oct. 15, 2015, 3:41 p.m. |
+1 |
Experience for voted comment killi: 1 on suic: Hi, this is typical _global variable_ issue. Read [this](http://www.checkio.org/forum/post/3269/6-th... |
Oct. 10, 2015, 6:32 p.m. |
+1 |
Experience for voted comment abha_kabra: 1 on suic: Hi, few comments: 1. Line 12 is never reached => redundant. 2. Line 13-15 are also redundant. 3. You... |
Oct. 10, 2015, 6:32 p.m. |
+5 |
Experience for appreciated comment abha_kabra: 1 on suic: Hi, few comments: 1. Line 12 is never reached => redundant. 2. Line 13-15 are also redundant. 3. You... |
Oct. 7, 2015, 1:51 p.m. |
+5 |
Experience for appreciated comment abha_kabra: 1 on suic: Hi, forget about ";" in Python :) |
Oct. 7, 2015, 1:51 p.m. |
+1 |
Experience for voted comment abha_kabra: 1 on suic: Hi, forget about ";" in Python :) |
Oct. 6, 2015, 12:15 a.m. |
+1 |
Experience for voted solution travis42: 1 on One-liner variation-suic |
Oct. 5, 2015, 8:09 p.m. |
+1 |
Experience for voted solution Detka_v_Ogne: 1 on One-liner-suic |
Sept. 28, 2015, 6:20 p.m. |
+1 |
Experience for voted comment jtokaz: 1 on suic: Hi, thanks. I've heard about _Code Eval_ and solved some of the _Project Euler_ problems, but I've n... |
Sept. 28, 2015, 6:20 p.m. |
+5 |
Experience for appreciated comment jtokaz: 1 on suic: Hi, thanks. I've heard about _Code Eval_ and solved some of the _Project Euler_ problems, but I've n... |
Sept. 25, 2015, 8:59 a.m. |
+1 |
Experience for voted comment Golli19: 1 on suic: Hi, 1. `else:` branch is redundant. You can remove it and dedent lines 9-16. 2. `gefunden` is redund... |
Sept. 25, 2015, 8:59 a.m. |
+5 |
Experience for appreciated comment Golli19: 1 on suic: Hi, 1. `else:` branch is redundant. You can remove it and dedent lines 9-16. 2. `gefunden` is redund... |
Sept. 25, 2015, 8:58 a.m. |
+1 |
Experience for voted comment Golli19: 1 on suic: Hi, 1. There's the _abs()_ built-in function: `return abs(d.days)` 2. You can unpack variables with ... |
Sept. 25, 2015, 8:58 a.m. |
+5 |
Experience for appreciated comment Golli19: 1 on suic: Hi, 1. There's the _abs()_ built-in function: `return abs(d.days)` 2. You can unpack variables with ... |
Sept. 25, 2015, 8:58 a.m. |
+1 |
Experience for voted comment Golli19: 1 on suic: Hi, this is not Pythonic: 1. `phrases` is a __tuple__ which is iterable, therefore you can write: fo... |
Sept. 25, 2015, 8:58 a.m. |
+5 |
Experience for appreciated comment Golli19: 1 on suic: Hi, this is not Pythonic: 1. `phrases` is a __tuple__ which is iterable, therefore you can write: fo... |
Sept. 25, 2015, 8:57 a.m. |
+1 |
Experience for voted comment Golli19: 1 on suic: Hi, this is way to complicated and completely misses the beauty and simplicity of Python: 1. Line 2:... |
Sept. 25, 2015, 8:57 a.m. |
+5 |
Experience for appreciated comment Golli19: 1 on suic: Hi, this is way to complicated and completely misses the beauty and simplicity of Python: 1. Line 2:... |
Sept. 25, 2015, 8:56 a.m. |
+1 |
Experience for voted comment Golli19: 1 on suic: Hi, `bin(number)` is a __str__, which has _str.count()_ method: return bin(number).count("1") |
Sept. 25, 2015, 8:56 a.m. |
+5 |
Experience for appreciated comment Golli19: 1 on suic: Hi, `bin(number)` is a __str__, which has _str.count()_ method: return bin(number).count("1") |
Sept. 25, 2015, 8:55 a.m. |
+1 |
Experience for voted comment Golli19: 1 on suic: Hi, look at _any()_ and _generator expressions_: return any(i != j and (i.endswith(j) or j.endswith(... |
Sept. 25, 2015, 8:55 a.m. |
+5 |
Experience for appreciated comment Golli19: 1 on suic: Hi, look at _any()_ and _generator expressions_: return any(i != j and (i.endswith(j) or j.endswith(... |
Sept. 25, 2015, 8:55 a.m. |
+1 |
Experience for voted comment Golli19: 1 on suic: Hi, look at _filter()_ and _str.join()_. |
Sept. 25, 2015, 8:55 a.m. |
+5 |
Experience for appreciated comment Golli19: 1 on suic: Hi, look at _filter()_ and _str.join()_. |
Sept. 25, 2015, 8:54 a.m. |
+1 |
Experience for voted comment Golli19: 1 on suic: Hi, 1. Line 14 is never reached. 2. Look at _list comprehensions_. |
Sept. 25, 2015, 8:54 a.m. |
+5 |
Experience for appreciated comment Golli19: 1 on suic: Hi, 1. Line 14 is never reached. 2. Look at _list comprehensions_. |
Sept. 22, 2015, 1:28 p.m. |
+1 |
Experience for voted comment Poiu: 1 on suic: Hi, 1. `array[i]` is an __int__, which isn't iterable and _sum()_ takes an interable (see the [docum... |
Sept. 22, 2015, 1:28 p.m. |
+5 |
Experience for appreciated comment Poiu: 1 on suic: Hi, 1. `array[i]` is an __int__, which isn't iterable and _sum()_ takes an interable (see the [docum... |
Sept. 21, 2015, 6:57 p.m. |
+4 |
Experience for voted solution arekrob: 2 on Easy-suic |
Sept. 21, 2015, 3:57 p.m. |
+1 |
Experience for voted comment oduvan: 1 on suic: Hi, what's the name of your function? `NoExecFunction` means that the _checkio()_ function is missin... |
Sept. 16, 2015, 8:07 p.m. |
+1 |
Experience for voted comment csaszarhunor: 1 on suic: A problema az hogy modositod a `data` listat. Itt felhasznalhatod a CodeAcademy javaslatat azaz a `f... |
Sept. 16, 2015, 8:07 p.m. |
+5 |
Experience for appreciated comment csaszarhunor: 1 on suic: A problema az hogy modositod a `data` listat. Itt felhasznalhatod a CodeAcademy javaslatat azaz a `f... |
Sept. 16, 2015, 8:06 p.m. |
+1 |
Experience for voted comment csaszarhunor: 1 on suic: Bocs, az elozobol kimaradt egy `a`. "a globalis valtozokat" akart lenni. A `result` valtozora gondol... |
Sept. 16, 2015, 8:06 p.m. |
+5 |
Experience for appreciated comment csaszarhunor: 1 on suic: Bocs, az elozobol kimaradt egy `a`. "a globalis valtozokat" akart lenni. A `result` valtozora gondol... |
Sept. 16, 2015, 8:06 p.m. |
+5 |
Experience for appreciated comment csaszarhunor: 1 on suic: Hi, check [The Most Common Mistakes](http://www.checkio.org/forum/post/3269/6-the-most-common-mistak... |
Sept. 16, 2015, 8:06 p.m. |
+1 |
Experience for voted comment csaszarhunor: 1 on suic: Hi, check [The Most Common Mistakes](http://www.checkio.org/forum/post/3269/6-the-most-common-mistak... |
Sept. 16, 2015, 8:06 p.m. |
+1 |
Experience for voted comment csaszarhunor: 1 on suic: > azt javasolta, hogy listáknál a módosítás helyett, inkább egy új listába rakjuk a keresett elemeke... |
Sept. 16, 2015, 8:06 p.m. |
+5 |
Experience for appreciated comment csaszarhunor: 1 on suic: > azt javasolta, hogy listáknál a módosítás helyett, inkább egy új listába rakjuk a keresett elemeke... |
Sept. 16, 2015, 6:45 p.m. |
+1 |
Experience for voted comment marko_anton: 1 on suic: Hi, no it's not a bug. The trick is that __list__ is mutable. Check the [Programming FAQ](https://do... |
Sept. 16, 2015, 6:45 p.m. |
+5 |
Experience for appreciated comment marko_anton: 1 on suic: Hi, no it's not a bug. The trick is that __list__ is mutable. Check the [Programming FAQ](https://do... |
Sept. 15, 2015, 6:16 p.m. |
+1 |
Experience for voted comment marko_anton: 1 on suic: Hi, you don't need `lambda`. _abs()_ is a function, so: return sorted(numbers_array, key=abs) is eno... |