Feb. 10, 2015, 4:23 a.m. |
+5 |
Experience for appreciated comment zebulan: 1 on suic: Hi zebulan, if you like (or don't like :)) my comments, can you please rate them with small + or - n... |
Feb. 10, 2015, 4:23 a.m. |
+1 |
Experience for voted comment zebulan: 1 on suic: Hi, instead of: lambda b, c: b * c # you could write: int.__mul__ |
Feb. 10, 2015, 4:23 a.m. |
+5 |
Experience for appreciated comment zebulan: 1 on suic: Hi, instead of: lambda b, c: b * c # you could write: int.__mul__ |
Feb. 10, 2015, 3:49 a.m. |
+5 |
Experience for appreciated comment srdi: 1 on suic: Hi, 1. why not: return total == 3 # instead of: if total!=3: return False elif total == 3: return Tr... |
Feb. 10, 2015, 3:49 a.m. |
+1 |
Experience for voted comment srdi: 1 on suic: Hi, 1. why not: return total == 3 # instead of: if total!=3: return False elif total == 3: return Tr... |
Feb. 10, 2015, 3:47 a.m. |
+1 |
Experience for voted comment srdi: 1 on suic: Hi, lines 14-17 work here, but this is not the correct way of doing this: 1. _is_ is not value compa... |
Feb. 10, 2015, 3:47 a.m. |
+5 |
Experience for appreciated comment srdi: 1 on suic: Hi, lines 14-17 work here, but this is not the correct way of doing this: 1. _is_ is not value compa... |
Feb. 9, 2015, 9:18 p.m. |
+1 |
Experience for voted comment Zeonik_d75eeb93918240b4b19eb9c: 1 on suic: Hi, when your condition is a boolean by itself, you can omit the if...else. E. g.: elif operator == ... |
Feb. 9, 2015, 9:18 p.m. |
+5 |
Experience for appreciated comment Zeonik_d75eeb93918240b4b19eb9c: 1 on suic: Hi, when your condition is a boolean by itself, you can omit the if...else. E. g.: elif operator == ... |
Feb. 9, 2015, 9:18 p.m. |
+5 |
Experience for appreciated comment Zeonik_d75eeb93918240b4b19eb9c: 1 on suic: Hi, the last if is redundant, you could omit it and un-indent return False by one level. |
Feb. 9, 2015, 9:18 p.m. |
+1 |
Experience for voted comment Zeonik_d75eeb93918240b4b19eb9c: 1 on suic: Hi, the last if is redundant, you could omit it and un-indent return False by one level. |
Feb. 9, 2015, 1:04 p.m. |
+1 |
Experience for voted comment Riddick: 1 on suic: Hi, I have two comments: 1. Look at _math.degrees_. 2. On lines 5-7 you have three times the same pa... |
Feb. 9, 2015, 1:04 p.m. |
+5 |
Experience for appreciated comment Riddick: 1 on suic: Hi, I have two comments: 1. Look at _math.degrees_. 2. On lines 5-7 you have three times the same pa... |
Feb. 9, 2015, 1:04 p.m. |
+5 |
Experience for appreciated comment Riddick: 1 on suic: One edit: # This would be even better: return len(data) >= 10 and re.search(...) # :) |
Feb. 9, 2015, 1:04 p.m. |
+1 |
Experience for voted comment Riddick: 1 on suic: One edit: # This would be even better: return len(data) >= 10 and re.search(...) # :) |
Feb. 9, 2015, 7:47 a.m. |
+2 |
Experience for voted solution Riddick: 2 on One-liner-suic |
Feb. 7, 2015, 2:19 p.m. |
+2 |
Experience for voted comment bukebuer: 1 on suic: Hi, it's subjective but it would be nicer to repace lines 11-15 with a generator expression and sum:... |
Feb. 7, 2015, 2:19 p.m. |
+5 |
Experience for appreciated comment bukebuer: 1 on suic: Hi, it's subjective but it would be nicer to repace lines 11-15 with a generator expression and sum:... |
Feb. 7, 2015, 1:12 p.m. |
+13 |
Experience for voted solution RiordanIX: 2 on One-liner (py2.7)-suic |
Feb. 6, 2015, 6:26 p.m. |
+1 |
Experience for voted comment Slepice1: 1 on suic: Hi, 1. There's an unnecessary semicolon :) 2. You could replace lines 5-8 to using all with this: ok... |
Feb. 6, 2015, 6:26 p.m. |
+5 |
Experience for appreciated comment Slepice1: 1 on suic: Hi, 1. There's an unnecessary semicolon :) 2. You could replace lines 5-8 to using all with this: ok... |
Feb. 6, 2015, 6:03 p.m. |
+20 |
Experience for solved mission Call to Home |
Feb. 6, 2015, 1:21 p.m. |
+5 |
Experience for voted solution Slepice1: 3 on Encoded Primes-suic |
Feb. 6, 2015, 12:36 p.m. |
+4 |
Experience for voted solution bryukh: 2 on First-suic |
Feb. 5, 2015, 3:36 p.m. |
+1 |
Experience for voted comment HanleyWashington: 1 on suic: Hi, a small stylistic thing. This would be the correct indentation here: return ",".join(sorted([wor... |
Feb. 5, 2015, 3:36 p.m. |
+5 |
Experience for appreciated comment HanleyWashington: 1 on suic: Hi, a small stylistic thing. This would be the correct indentation here: return ",".join(sorted([wor... |
Feb. 5, 2015, 11:43 a.m. |
+1 |
Experience for voted comment OMFGNuts: 1 on suic: Hi, I have two comments: 1. Don's use _tuple_ as variable name as [tuple is a built-in type](https:/... |
Feb. 5, 2015, 8:02 a.m. |
+5 |
Experience for appreciated comment laserpez: 1 on suic: Hi, you could use str.isupper instead of t in ascii_uppercase. In that case you don't need to import... |
Feb. 5, 2015, 8:02 a.m. |
+1 |
Experience for voted comment laserpez: 1 on suic: Hi, you could use str.isupper instead of t in ascii_uppercase. In that case you don't need to import... |
Feb. 4, 2015, 7:36 p.m. |
+7 |
Experience for voted solution ok00: 1 on One-liner-suic |
Feb. 4, 2015, 2:18 p.m. |
+1 |
Experience for voted comment evoynov: 1 on suic: Hi, _list_ is redundant. You can the set directly. |
Feb. 4, 2015, 2:18 p.m. |
+5 |
Experience for appreciated comment evoynov: 1 on suic: Hi, _list_ is redundant. You can the set directly. |
Feb. 4, 2015, 1:26 p.m. |
+5 |
Experience for appreciated comment polly: 1 on suic: Hi, you can omit the [] braces. |
Feb. 4, 2015, 1:26 p.m. |
+1 |
Experience for voted comment polly: 1 on suic: Hi, you can omit the [] braces. |
Feb. 4, 2015, 1:14 p.m. |
+8 |
Experience for solved mission Ghosts Age |
Feb. 4, 2015, 1:10 p.m. |
+8 |
Experience for solved mission Common Words |
Feb. 4, 2015, 8:04 a.m. |
+1 |
Experience for voted comment Baxter: 1 on suic: Hi, you don't the _True ... if ... else False_ as your condition is already a boolean value so: retu... |
Feb. 4, 2015, 8:04 a.m. |
+5 |
Experience for appreciated comment Baxter: 1 on suic: Hi, you don't the _True ... if ... else False_ as your condition is already a boolean value so: retu... |
Feb. 4, 2015, 8:03 a.m. |
+1 |
Experience for voted comment Baxter: 1 on suic: Hi, what about using slices? # e. g.: summa = sum(array[::2]) |
Feb. 4, 2015, 8:03 a.m. |
+5 |
Experience for appreciated comment Baxter: 1 on suic: Hi, what about using slices? # e. g.: summa = sum(array[::2]) |
Feb. 3, 2015, 9:39 p.m. |
+2 |
Experience for voted solution AlexS: 2 on One-liner-suic |
Feb. 3, 2015, 8:46 p.m. |
+1 |
Experience for voted comment mscmorris: 1 on suic: Hi, no there isn't. |
Feb. 3, 2015, 8:46 p.m. |
+5 |
Experience for appreciated comment mscmorris: 1 on suic: Hi, no there isn't. |
Feb. 3, 2015, 8:45 p.m. |
+5 |
Experience for appreciated comment mscmorris: 1 on suic: Hi, you could write it also this way: return bool(re.compile('.*?[a-zA-Z]+ [a-zA-Z]+ [a-zA-Z]+.*?').... |
Feb. 3, 2015, 8:45 p.m. |
+1 |
Experience for voted comment mscmorris: 1 on suic: Hi, you could write it also this way: return bool(re.compile('.*?[a-zA-Z]+ [a-zA-Z]+ [a-zA-Z]+.*?').... |
Feb. 3, 2015, 7:52 p.m. |
+1 |
Experience for voted comment HanleyWashington: 1 on suic: Hi, in fact you don't need to import string here. Look at str.isdigit, str.islower and str.isupper m... |
Feb. 3, 2015, 7:52 p.m. |
+5 |
Experience for appreciated comment HanleyWashington: 1 on suic: Hi, in fact you don't need to import string here. Look at str.isdigit, str.islower and str.isupper m... |
Feb. 3, 2015, 9:12 a.m. |
+1 |
Experience for voted comment OMFGNuts: 1 on suic: Hi, look at [FAQ](http://www.checkio.org/forum/post/3269/6-the-most-common-mistakes/) _Iterate and r... |
Feb. 1, 2015, 8:30 p.m. |
+5 |
Experience for appreciated comment Dodge_545c07bd039b4e1aa1a6dd88: 1 on suic: Hi, lambda is redundant as: (lambda s: s.isupper()) == str.isupper # without ()! |
Feb. 1, 2015, 8:30 p.m. |
+1 |
Experience for voted comment Dodge_545c07bd039b4e1aa1a6dd88: 1 on suic: Hi, lambda is redundant as: (lambda s: s.isupper()) == str.isupper # without ()! |
Jan. 31, 2015, 2:33 p.m. |
+2 |
Experience for voted comment Riddick: 1 on suic: Hi, you missed a bit the concept of list comprehension: You could write: new = [i for i in data if d... |
Jan. 31, 2015, 2:33 p.m. |
+5 |
Experience for appreciated comment Riddick: 1 on suic: Hi, you missed a bit the concept of list comprehension: You could write: new = [i for i in data if d... |
Jan. 30, 2015, 10:37 p.m. |
+8 |
Experience for solved mission The Most Wanted Letter |
Jan. 30, 2015, 1:32 p.m. |
+1 |
Experience for voted solution VitaliySeyz: 1 on One-liner-suic |
Jan. 28, 2015, 9:53 p.m. |
+2 |
Experience for voted solution spoty: 1 on One-liner with magic number-suic |
Jan. 28, 2015, 8:37 p.m. |
+2 |
Experience for voted solution ok00: 1 on Shortened-suic |
Jan. 28, 2015, 3:25 p.m. |
+5 |
Experience for appreciated comment OMFGNuts: 1 on suic: Hi, that's python's syntax binary representation of integers (look at _Syntax for supported radices_... |
Jan. 28, 2015, 3:25 p.m. |
+1 |
Experience for voted comment OMFGNuts: 1 on suic: Hi, that's python's syntax binary representation of integers (look at _Syntax for supported radices_... |
Jan. 28, 2015, 2:56 p.m. |
+1 |
Experience for voted comment GabrielHahmann: 1 on suic: Hi, that's python's syntax binary representation of integers (look at _Syntax for supported radices_... |
Jan. 28, 2015, 2:56 p.m. |
+5 |
Experience for appreciated comment GabrielHahmann: 1 on suic: Hi, that's python's syntax binary representation of integers (look at _Syntax for supported radices_... |
Jan. 28, 2015, 7:47 a.m. |
+1 |
Experience for voted comment bryukh: 1 on suic: Hi, look at the result of bin(5). |
Jan. 28, 2015, 7:47 a.m. |
+5 |
Experience for appreciated comment bryukh: 1 on suic: Hi, that's python's syntax binary representation of integers (look at _Syntax for supported radices_... |
Jan. 28, 2015, 7:47 a.m. |
+1 |
Experience for voted comment bryukh: 1 on suic: Hi, that's python's syntax binary representation of integers (look at _Syntax for supported radices_... |
Jan. 28, 2015, 6:53 a.m. |
+8 |
Experience for solved mission Pangram |
Jan. 27, 2015, 7:55 p.m. |
+5 |
Experience for appreciated comment abhi.iitdmnc: 1 on suic: Hi, have a look at these two forum threads ([Learning Python, tips?](http://www.checkio.org/forum/po... |
Jan. 27, 2015, 7:55 p.m. |
+1 |
Experience for voted comment abhi.iitdmnc: 1 on suic: Hi, have a look at these two forum threads ([Learning Python, tips?](http://www.checkio.org/forum/po... |
Jan. 27, 2015, 7:55 p.m. |
+1 |
Experience for voted comment abhi.iitdmnc: 1 on suic: Hi, I have a few comments: 1. Line 11 is useless as it's never reached because it returns on line 10... |
Jan. 27, 2015, 7:55 p.m. |
+5 |
Experience for appreciated comment abhi.iitdmnc: 1 on suic: Hi, I have a few comments: 1. Line 11 is useless as it's never reached because it returns on line 10... |
Jan. 27, 2015, 7:10 p.m. |
+1 |
Experience for voted comment bryukh: 1 on suic: Hi, it should work like this: 1. Click for example on Elementary, then you should see the list of mi... |
Jan. 27, 2015, 4:07 p.m. |
+5 |
Experience for appreciated comment AdamToth: 1 on suic: Maybe, it's less clear at first sight. I can guarantee that once you get used to list comprehensions... |
Jan. 27, 2015, 4:07 p.m. |
+1 |
Experience for voted comment AdamToth: 1 on suic: Maybe, it's less clear at first sight. I can guarantee that once you get used to list comprehensions... |
Jan. 27, 2015, 4:07 p.m. |
+5 |
Experience for appreciated comment AdamToth: 1 on suic: Hi, on lines 4 - 6 you have three times the same pattern. You could reduce the redundancy by using l... |
Jan. 27, 2015, 4:07 p.m. |
+1 |
Experience for voted comment AdamToth: 1 on suic: Hi, on lines 4 - 6 you have three times the same pattern. You could reduce the redundancy by using l... |
Jan. 27, 2015, 3:05 p.m. |
+1 |
Experience for voted comment OMFGNuts: 1 on suic: Hi, it should work like this: 1. Click for example on Elementary, then you should see the list of mi... |
Jan. 27, 2015, 2:54 p.m. |
+2 |
Experience for voted solution DiZ: 1 on One-liner with magic number-suic |
Jan. 27, 2015, 12:10 p.m. |
+1 |
Experience for voted comment OMFGNuts: 1 on suic: Hi, there are several things. Let me mention three: 1. Lines 9-11 do nothing. You check e. g. if i i... |
Jan. 27, 2015, 7:45 a.m. |
+1 |
Experience for voted solution Mark_Heywood: 1 on One-liner-suic |
Jan. 27, 2015, 6:07 a.m. |
+1 |
Experience for voted comment bryukh: 1 on suic: Hi, there are several things. Let me mention three: 1. Lines 9-11 do nothing. You check e. g. if i i... |
Jan. 27, 2015, 12:32 a.m. |
+5 |
Experience for appreciated comment hellerve_f2e46923d3ff406194333: 1 on suic: Hi, have you heard about str.isupper() (check help(str.isupper))? You could also write: secret += i ... |
Jan. 27, 2015, 12:32 a.m. |
+1 |
Experience for voted comment hellerve_f2e46923d3ff406194333: 1 on suic: Hi, have you heard about str.isupper() (check help(str.isupper))? You could also write: secret += i ... |
Jan. 27, 2015, 12:31 a.m. |
+1 |
Experience for voted comment hellerve_f2e46923d3ff406194333: 1 on suic: Hi, first thing: _sum_ is not a good variable name as it overrides a built-in function called __sum_... |
Jan. 27, 2015, 12:31 a.m. |
+5 |
Experience for appreciated comment hellerve_f2e46923d3ff406194333: 1 on suic: Hi, first thing: _sum_ is not a good variable name as it overrides a built-in function called __sum_... |
Jan. 27, 2015, 12:31 a.m. |
+1 |
Experience for voted comment hellerve_f2e46923d3ff406194333: 1 on suic: Plus one for "code-golfy" :) |
Jan. 27, 2015, 12:27 a.m. |
+1 |
Experience for voted solution hellerve_f2e46923d3ff406194333: 1 on One-liner-suic |
Jan. 26, 2015, 9:57 a.m. |
+1 |
Experience for voted comment DiZ: 1 on suic: > we have to print output From where did you get it? It's not true and it does not make sense, as: 1... |
Jan. 26, 2015, 8:58 a.m. |
+13 |
Experience for voted solution kacerr: 2 on First-suic |
Jan. 26, 2015, 8 a.m. |
+2 |
Experience for voted solution kacerr: 1 on One-liner-suic |
Jan. 26, 2015, 6:17 a.m. |
+1 |
Experience for voted comment bryukh: 1 on suic: > we have to print output From where did you get it? It's not true and it does not make sense, as: 1... |
Jan. 26, 2015, 6:17 a.m. |
+1 |
Experience for voted comment bryukh: 1 on suic: Hi, you have to __return__ your result and __not print__. |
Jan. 26, 2015, 6:10 a.m. |
+1 |
Experience for voted comment bryukh: 1 on suic: Hi, read the description: > your code __[has to] be shorter than 140 characters__ (with whitespaces)... |
Jan. 26, 2015, 6:10 a.m. |
+1 |
Experience for voted comment bryukh: 1 on suic: Hi list.append() __modifies__ the list and __returns None__. # So: result.append(x) # instead of: re... |
Jan. 26, 2015, 1:52 a.m. |
+5 |
Experience for appreciated comment flex: 1 on suic: Hi, you can use if it works :) (To be honest I've never used maketrans.) I have two suggestions: 1. ... |
Jan. 26, 2015, 1:52 a.m. |
+1 |
Experience for voted comment flex: 1 on suic: Hi, you can use if it works :) (To be honest I've never used maketrans.) I have two suggestions: 1. ... |
Jan. 26, 2015, 1:31 a.m. |
+5 |
Experience for appreciated comment Finomyato: 1 on suic: Hi, I have a few comments/suggestions: 1) text.lower() is 4 times in your. It would be better to ass... |
Jan. 26, 2015, 1:31 a.m. |
+1 |
Experience for voted comment Finomyato: 1 on suic: Hi, I have a few comments/suggestions: 1) text.lower() is 4 times in your. It would be better to ass... |
Jan. 25, 2015, 11:34 p.m. |
+1 |
Experience for voted comment flex: 1 on suic: Hi, you have to remove not only the dots, but also the commas :) |
Jan. 25, 2015, 11:34 p.m. |
+5 |
Experience for appreciated comment flex: 1 on suic: Hi, you have to remove not only the dots, but also the commas :) |
Jan. 25, 2015, 9:38 p.m. |
+5 |
Experience for appreciated comment flex: 1 on suic: Hi, one small thing: The parentheses around ",".join(phrases) are superfluous as you can chain metho... |
Jan. 25, 2015, 9:38 p.m. |
+1 |
Experience for voted comment flex: 1 on suic: Hi, one small thing: The parentheses around ",".join(phrases) are superfluous as you can chain metho... |
Jan. 25, 2015, 6:57 p.m. |
+5 |
Experience for appreciated comment coole_r: 1 on suic: Hi, 1. once again: indentation i. e.: if...: elif: else: 2. You removed a piece of important part of... |