ShadPython - 1 - Fabarisov Dmitriy - AD

Первое домашнее задание. Не сильно заморачивался с краткостью, т.к. еще не познал дзен питона :)

[shad.python, 2015-03-06 11:31:52]

On 2015/03/03 22:39:29, brain2805 wrote:

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

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

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

[brain2805, 2015-03-06 17:26:43]

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

У меня нет инвайта для регистрации.

С уважением, Фабарисов Дмитрий
1 курс ШАД (АД)

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

> On 2015/03/03 22:39:29, brain2805 wrote:
>
> Здравствуйте!
>
> Учет сдачи и оценок будет вестись через систему AnyTask (anytask.org).
> Если у вас уже есть там аккаунт, то вам нужно зайти на страницу курса
> "ШАД > python 2 семестр" и нажать кнопку "Записаться" в правом верхнем
> углу. Если у вас нет аккаунта в этой системе, то для регистрации в ней
> нужен инвайт (напишите об этом в ответ на это письмо, и я пришлю).
> Просьба при регистрации правильно указывать фамилию и имя, чтобы потом
> не возникло проблем при выставлении оценки за курс. После регистрации
> нужно аналогичным образом записаться на курс.
>
> После записи на курс вам нужно прислать в систему ваше решение домашнего
> задания. Для этого нужно перейти на страницу курса и нажать на
> соответствующую ячейку в таблице результатов (с цифрой 0), и на
> открывшейся странице отправки ввести ссылку на ваше Issue в CodeReview и
> номера успешных посылок в Я.Контест. Отправьте решение и ожидайте
> проверки.
>
> https://codereview.appspot.com/212780043/
>

[shad.python, 2015-03-17 20:07:22]

https://codereview.appspot.com/212780043/diff/60001/problem1.1.py
File problem1.1.py (right):

https://codereview.appspot.com/212780043/diff/60001/problem1.1.py#newcode6
problem1.1.py:6: if index < 2:
Для основных содержательных действий в программе лучше создавать отдельные
функции. Это позволит легче переиспользовать их в дальнейшем.

https://codereview.appspot.com/212780043/diff/60001/problem1.1.py#newcode7
problem1.1.py:7: print index
Ввод/вывод и содержательные действия лучше отделять друг от друга. Это позволит
в дальнейшем этот код переиспользовать, даже если формат ввода и вывода
поменяется.

https://codereview.appspot.com/212780043/diff/60001/problem1.3.py
File problem1.3.py (right):

https://codereview.appspot.com/212780043/diff/60001/problem1.3.py#newcode4
problem1.3.py:4: f = open('input.txt', 'r')
При работе с файлами лучше пользоваться конструкцией with..as.., чтобы точно
контролировать момент их закрытия.

https://codereview.appspot.com/212780043/diff/60001/problem1.4.py
File problem1.4.py (right):

https://codereview.appspot.com/212780043/diff/60001/problem1.4.py#newcode20
problem1.4.py:20: curstr += ' ' + word
Лучше не пользоваться оператором += у строк, так как в некоторых реализациях
питона он работает медленно. Вместо этого лучше собирать части строк в список, а
потом использовать join.

[brain2805, 2015-03-20 14:11:57]

Исправил все замечания

https://codereview.appspot.com/212780043/diff/60001/problem1.1.py
File problem1.1.py (right):

https://codereview.appspot.com/212780043/diff/60001/problem1.1.py#newcode6
problem1.1.py:6: if index < 2:
On 2015/03/17 20:07:21, shad.python wrote:
> Для основных содержательных действий в программе лучше создавать отдельные
> функции. Это позволит легче переиспользовать их в дальнейшем.

Done.

https://codereview.appspot.com/212780043/diff/60001/problem1.1.py#newcode7
problem1.1.py:7: print index
On 2015/03/17 20:07:21, shad.python wrote:
> Ввод/вывод и содержательные действия лучше отделять друг от друга. Это
позволит
> в дальнейшем этот код переиспользовать, даже если формат ввода и вывода
> поменяется.

Done.

https://codereview.appspot.com/212780043/diff/60001/problem1.3.py
File problem1.3.py (right):

https://codereview.appspot.com/212780043/diff/60001/problem1.3.py#newcode4
problem1.3.py:4: f = open('input.txt', 'r')
On 2015/03/17 20:07:21, shad.python wrote:
> При работе с файлами лучше пользоваться конструкцией with..as.., чтобы точно
> контролировать момент их закрытия.

Done.

https://codereview.appspot.com/212780043/diff/60001/problem1.4.py
File problem1.4.py (right):

https://codereview.appspot.com/212780043/diff/60001/problem1.4.py#newcode20
problem1.4.py:20: curstr += ' ' + word
On 2015/03/17 20:07:21, shad.python wrote:
> Лучше не пользоваться оператором += у строк, так как в некоторых реализациях
> питона он работает медленно. Вместо этого лучше собирать части строк в список,
а
> потом использовать join.

Done.

[brain2805, 2015-03-20 14:14:13]

В предыдущей посылке не обновилась 4 задача, исправил.

[shad.python, 2015-04-06 17:27:13]

https://codereview.appspot.com/212780043/diff/100001/problem1.1.py
File problem1.1.py (right):

https://codereview.appspot.com/212780043/diff/100001/problem1.1.py#newcode28
problem1.1.py:28: indexFibonacci = readIndex()
На этой и некоторых других строках что-то случилось с отступами.

https://codereview.appspot.com/212780043/diff/100001/problem1.2.py
File problem1.2.py (right):

https://codereview.appspot.com/212780043/diff/100001/problem1.2.py#newcode12
problem1.2.py:12: for coord in Coordinates:
Здесь тоже нужно учесть замечания про функции.

https://codereview.appspot.com/212780043/diff/100001/problem1.3.py
File problem1.3.py (right):

https://codereview.appspot.com/212780043/diff/100001/problem1.3.py#newcode5
problem1.3.py:5: for line in f:
Здесь тоже нужно учесть замечание про функции и замечание про ввод/вывод.

https://codereview.appspot.com/212780043/diff/100001/problem1.4.py
File problem1.4.py (right):

https://codereview.appspot.com/212780043/diff/100001/problem1.4.py#newcode3
problem1.4.py:3: with open('input.txt', 'r') as f:
Здесь тоже нужно учесть замечание про функции и замечание про ввод/вывод

[brain2805, 2015-05-12 01:47:57]

Исправил все замечания по стилю

[shad.python, 2015-05-16 13:26:14]

https://codereview.appspot.com/212780043/diff/180001/problem1.3.py
File problem1.3.py (right):

https://codereview.appspot.com/212780043/diff/180001/problem1.3.py#newcode3
problem1.3.py:3: def form_dictionary_from_file(filename):
Это не очень понятное название функции - непонятно, что за словарь строится из
файла. В названиях функций важнее указывать их основные действия, чем то, какой
тип данных они получают и возвращают. Последнее гораздо проще определить по
коду.