Ostrovsky Lev - 4

[LevSky93, 2015-05-12 11:53:31]

Высылаю код по 4 заданию. Юнит-тесты, файл получения отчета, и другие файлы
послал в архиве в Anytask.

[shad.python, 2015-05-25 18:42:46]

Замечания нужно поправить, тесты в исходном архиве не запускаются из-за
захардкоженных путей к конфигу, это тоже нужно переделать, совсем не хорошо.

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

https://codereview.appspot.com/232370043/diff/1/main.py
File main.py (right):

https://codereview.appspot.com/232370043/diff/1/main.py#newcode9
main.py:9: # Generate_Discrete_Distribution
почему не докстрингами, зачем дублировать документацию просто комментарием?

https://codereview.appspot.com/232370043/diff/1/main.py#newcode16
main.py:16: # 0 - victim ,1 - predator, 2 - empty, 3 - barrier
Константы нужно вынести и сделать отдельными переменными, в первую очередь для
того чтобы самому потом не путаться в цифрах

https://codereview.appspot.com/232370043/diff/1/main.py#newcode112
main.py:112: '''This method initialize the start state of the ocean.
докстринги лучше оформлять по pep257

https://codereview.appspot.com/232370043/diff/1/main.py#newcode168
main.py:168: for ptr in neighboors_coordinates:
ptr плохое название, лично у меня ассоциируется с указателем, например.

https://codereview.appspot.com/232370043/diff/1/main.py#newcode231
main.py:231: print 'Processing' + str(x_ptr) + ' ' + str(y_ptr)
Почему бы тут не воспользоваться функцией .format у строки? Это же удобнее.

https://codereview.appspot.com/232370043/diff/1/main.py#newcode251
main.py:251: def process(self, count_iteration, type):
не нужно называть переменные именами используемыми по-умолчанию в питоне. Можно
потом очень долго искать что же пошло не так в случае чего.

https://codereview.appspot.com/232370043/diff/1/main.py#newcode259
main.py:259: residents = ['v', 'p', 'e', 'b']
это тоже явно константы

https://codereview.appspot.com/232370043/diff/1/main.py#newcode273
main.py:273: parser.add_argument('-type')
Что такое type не понятно совершенно, если не читать код

[LevSky93, 2015-06-03 12:02:34]

Высылаю исправленную версию.

https://codereview.appspot.com/232370043/diff/1/main.py
File main.py (right):

https://codereview.appspot.com/232370043/diff/1/main.py#newcode9
main.py:9: # Generate_Discrete_Distribution
On 2015/05/25 18:42:46, shad.python wrote:
> почему не докстрингами, зачем дублировать документацию просто комментарием?

Done.

https://codereview.appspot.com/232370043/diff/1/main.py#newcode16
main.py:16: # 0 - victim ,1 - predator, 2 - empty, 3 - barrier
On 2015/05/25 18:42:46, shad.python wrote:
> Константы нужно вынести и сделать отдельными переменными, в первую очередь для
> того чтобы самому потом не путаться в цифрах

Done.

https://codereview.appspot.com/232370043/diff/1/main.py#newcode112
main.py:112: '''This method initialize the start state of the ocean.
On 2015/05/25 18:42:46, shad.python wrote:
> докстринги лучше оформлять по pep257

Done.

https://codereview.appspot.com/232370043/diff/1/main.py#newcode168
main.py:168: for ptr in neighboors_coordinates:
On 2015/05/25 18:42:46, shad.python wrote:
> ptr плохое название, лично у меня ассоциируется с указателем, например.

Done.

https://codereview.appspot.com/232370043/diff/1/main.py#newcode231
main.py:231: print 'Processing' + str(x_ptr) + ' ' + str(y_ptr)
On 2015/05/25 18:42:46, shad.python wrote:
> Почему бы тут не воспользоваться функцией .format у строки? Это же удобнее.

Done.

https://codereview.appspot.com/232370043/diff/1/main.py#newcode251
main.py:251: def process(self, count_iteration, type):
On 2015/05/25 18:42:46, shad.python wrote:
> не нужно называть переменные именами используемыми по-умолчанию в питоне.
Можно
> потом очень долго искать что же пошло не так в случае чего.

Done.

https://codereview.appspot.com/232370043/diff/1/main.py#newcode259
main.py:259: residents = ['v', 'p', 'e', 'b']
On 2015/05/25 18:42:46, shad.python wrote:
> это тоже явно константы

Done.

https://codereview.appspot.com/232370043/diff/1/main.py#newcode273
main.py:273: parser.add_argument('-type')
On 2015/05/25 18:42:46, shad.python wrote:
> Что такое type не понятно совершенно, если не читать код

Done.

[shad.python, 2015-06-05 16:41:25]

Неужели даже не запускаете перед тем как отправить? (

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

https://codereview.appspot.com/232370043/diff/20001/main.py
File main.py (right):

https://codereview.appspot.com/232370043/diff/20001/main.py#newcode1
main.py:1: # -*- coding: utf-8 -*-
Юнит тесты до сих пор падают:

PS C:\Users\nosyrev\Downloads\python_dz4\HW-4\HW-4> python .\test_main.py
.....EE..
======================================================================
ERROR: test_1 (__main__.test_ocean)
----------------------------------------------------------------------
Traceback (most recent call last):
  File ".\test_main.py", line 61, in test_1
    ocean = main.Ocean(CONFIG_PATH)
  File "C:\Users\nosyrev\Downloads\python_dz4\HW-4\HW-4\main.py", line 125, in _
_init__
    size = self.get_size()
  File "C:\Users\nosyrev\Downloads\python_dz4\HW-4\HW-4\main.py", line 161, in g
et_size
    return int(self.cfgParser.get('Ocean', 'size'))
  File "C:\Python27\lib\ConfigParser.py", line 607, in get
    raise NoSectionError(section)
NoSectionError: No section: 'Ocean'

======================================================================
ERROR: test_2 (__main__.test_ocean)
----------------------------------------------------------------------
Traceback (most recent call last):
  File ".\test_main.py", line 78, in test_2
    ocean = main.Ocean(CONFIG_PATH)
  File "C:\Users\nosyrev\Downloads\python_dz4\HW-4\HW-4\main.py", line 125, in _
_init__
    size = self.get_size()
  File "C:\Users\nosyrev\Downloads\python_dz4\HW-4\HW-4\main.py", line 161, in g
et_size
    return int(self.cfgParser.get('Ocean', 'size'))
  File "C:\Python27\lib\ConfigParser.py", line 607, in get
    raise NoSectionError(section)
NoSectionError: No section: 'Ocean'

----------------------------------------------------------------------
Ran 9 tests in 0.067s

FAILED (errors=2)

https://codereview.appspot.com/232370043/diff/20001/main.py#newcode285
main.py:285: #main()
Теперь код вообще не запускается) Поправьте