ShadPython - 1 - Danila Chesnokov - CS Novosibirsk

task1 - 577586
task2 - 577588
task3 - 577605
task4 - 576667

[ChesnokovDS, 2015-03-04 20:06:02]

Chesnokov Danila ShAD Novosibirsk
task1 - 577586
task2 - 577588
task3 - 577605
task4 - 576667

[shad.python, 2015-03-06 11:38:13]

On 2015/03/04 20:06:02, ChesnokovDS wrote:
> Chesnokov Danila ShAD Novosibirsk
> task1 - 577586
> task2 - 577588
> task3 - 577605
> task4 - 576667

Здравствуйте!

Учет сдачи и оценок будет вестись через систему AnyTask (anytask.org). Если у
вас уже есть там аккаунт, то вам нужно зайти на страницу курса "ШАД > python 2
семестр" и нажать кнопку "Записаться" в правом верхнем углу. Если у вас нет
аккаунта в этой системе, то для регистрации в ней нужен инвайт (напишите об этом
в ответ на это письмо, и я пришлю). Просьба при регистрации правильно указывать
фамилию и имя, чтобы потом не возникло проблем при выставлении оценки за курс.
После регистрации нужно аналогичным образом записаться на курс.

После записи на курс вам нужно прислать в систему ваше решение домашнего
задания. Для этого нужно перейти на страницу курса и нажать на соответствующую
ячейку в таблице результатов (с цифрой 0), и на открывшейся странице отправки
ввести ссылку на ваше Issue в CodeReview. Отправьте решение и ожидайте проверки.

[chesnokovds_gmail.com, 2015-03-06 13:28:45]

Пришлите инвайт для регистрации в системе AnyTask

6 марта 2015 г., 17:38 пользователь <shad.python@gmail.com> написал:

> On 2015/03/04 20:06:02, ChesnokovDS wrote:
>
>> Chesnokov Danila ShAD Novosibirsk
>> task1 - 577586
>> task2 - 577588
>> task3 - 577605
>> task4 - 576667
>>
>
> Здравствуйте!
>
> Учет сдачи и оценок будет вестись через систему AnyTask (anytask.org).
> Если у вас уже есть там аккаунт, то вам нужно зайти на страницу курса
> "ШАД > python 2 семестр" и нажать кнопку "Записаться" в правом верхнем
> углу. Если у вас нет аккаунта в этой системе, то для регистрации в ней
> нужен инвайт (напишите об этом в ответ на это письмо, и я пришлю).
> Просьба при регистрации правильно указывать фамилию и имя, чтобы потом
> не возникло проблем при выставлении оценки за курс. После регистрации
> нужно аналогичным образом записаться на курс.
>
> После записи на курс вам нужно прислать в систему ваше решение домашнего
> задания. Для этого нужно перейти на страницу курса и нажать на
> соответствующую ячейку в таблице результатов (с цифрой 0), и на
> открывшейся странице отправки ввести ссылку на ваше Issue в CodeReview.
> Отправьте решение и ожидайте проверки.
>
> https://codereview.appspot.com/214740043/
>

[shad.python, 2015-04-13 15:35:14]

Можно исправлять дальше.

--
Григорий Носырев

https://codereview.appspot.com/214740043/diff/1/task4.py
File task4.py (right):

https://codereview.appspot.com/214740043/diff/1/task4.py#newcode6
task4.py:6: ln = int(stdin.readline())
плохое название, буквы на названиях лучше не экономить.

https://codereview.appspot.com/214740043/diff/1/task4.py#newcode17
task4.py:17: if len(word) > space:
обрезку строки и вывод лучше разделить на разные функции.

https://codereview.appspot.com/214740043/diff/20001/task2.py
File task2.py (right):

https://codereview.appspot.com/214740043/diff/20001/task2.py#newcode9
task2.py:9: sum += pow(abs(x_i), p)
Самое главное замечание по этой задаче и не поправили, нехорошо. Это относится и
к остальным задачам.
---
Называть переменные так же как и стандартные классы в питоне - очень плохая 
практика. Нужно переписать.

https://codereview.appspot.com/214740043/diff/20001/task3.py
File task3.py (right):

https://codereview.appspot.com/214740043/diff/20001/task3.py#newcode6
task3.py:6: class SortedLetters:
классы лучше всего сразу наследовать от objects, почему - см лекции

https://codereview.appspot.com/214740043/diff/20001/task3.py#newcode14
task3.py:14: elif self.count > other.count \
символов переноса строки лучше избегать. Для этого можно просто условие писать в
скобках, например вот так:
elif (
    self.count > other.count or
    (self.count == other.count and self.symbol < other.symbol)
):
    ...

https://codereview.appspot.com/214740043/diff/20001/task3.py#newcode25
task3.py:25: if symbol in result:
Про словарь не прочитали?

https://codereview.appspot.com/214740043/diff/20001/task3.py#newcode33
task3.py:33: stdout.write(str(k) + ': ' + str(v) + '\n')
В таких случаях для улучшения читабельности удобно пользоваться
функцией .format() у строки
https://docs.python.org/2/library/stdtypes.html#str.format

https://codereview.appspot.com/214740043/diff/20001/task3.py#newcode39
task3.py:39: allLetters = []
гораздо проще было бы сортировать сразу, а критерий сортировки передавать лямбда
функцией

[ChesnokovDS, 2015-04-24 20:44:55]

Замечания исправлены. За исключением
https://codereview.appspot.com/214740043/diff/20001/task3.py#newcode39. Мне не
приходит в голову, как написать лямда-функцию, которая выполняла поставленную
задачу и не ухудшала читабельность кода.

[shad.python, 2015-04-25 15:27:51]

хехе, все замечания распространяются на все задачи же.

--
Григорий Носырев

https://codereview.appspot.com/214740043/diff/40001/task2.py
File task2.py (right):

https://codereview.appspot.com/214740043/diff/40001/task2.py#newcode20
task2.py:20: stdout.write(str(distM(p, x)))
писать вывод лучше в одном месте. Тут можно по разному суммы считать.

https://codereview.appspot.com/214740043/diff/40001/task3.py
File task3.py (right):

https://codereview.appspot.com/214740043/diff/40001/task3.py#newcode33
task3.py:33: stdout.write(str(k) + ': ' + str(v) + '\n')
используйте функцию .format, она удобная и разобраться с ней будет полезно

https://codereview.appspot.com/214740043/diff/40001/task3.py#newcode38
task3.py:38: dict = symbols(txt)
Неужели в каждом ревью придётся писать? (
Называть переменные так же как и стандартные классы в питоне - очень плохая 
практика. Нужно переписать.

https://codereview.appspot.com/214740043/diff/40001/task3.py#newcode40
task3.py:40: for k, v in dict.items():
и однобуквенные переменные не комильфо

https://codereview.appspot.com/214740043/diff/40001/task3.py#newcode41
task3.py:41: allLetters.append(SortedLetters(k, v))
очень просто же!
key=lambda (key, freq): (-freq, key)