tel8618217223380 / oasychev-moodle-plugins

Automatically exported from code.google.com/p/oasychev-moodle-plugins
0 stars 0 forks source link

Рефакторинг: выходные данные enum_analyzer #323

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Насколько я помню вначале подразумевалось 
что enum_analyzer будет переставлять лексемы 
правильной строки. НО: такой подход не 
очень удачен, т.к. потребует пересчитать 
индексы лексем правильной строки для 
lexical_mistakes и соответствующих matches, которые 
были сгенерированы раньше.

Предлагаю вместо этого создать 
дополнительное поле: enumprocessedstring, туда 
скопировать correctstring и менять там. ОЧЕНЬ 
СРОЧНО необходимо написать поле и bypass 
который будет его заполнять как есть, чтобы 
можно было поправить sequence_analyzer на работу с 
ним.

P.S. Вообще, если enum_analyzer уже находит lcs, то 
может быть логичным сливание его вместе с 
sequence_analyzer чтобы дважды ее не искать....

Original issue reported on code.google.com by oasyc...@gmail.com on 14 Jan 2015 at 7:04

GoogleCodeExporter commented 9 years ago
Перенесено из другой ветки.
-------------------------
Залил. В correct_mistakes был обнаружен очень 
странный код. Переписал с учетом своих 
знаний об архитектуре. 

Сделал функции построения отображений, 
правда необходимо сделать тесты. Пока меня 
смущает то, что в них используется correct_mistakes 
частично, точнее информация, которая 
генерируется внутри этого алгоритма. Мне 
кажется, что надо как-то отрефакторить его 
так, чтобы эти данные были получены сразу а 
не потом.

Кроме этого, думаю поля, связанные с 
перечислениями, добавленные в коммите, 
будут нужны и Вадиму.

Original comment by mamontov...@gmail.com on 18 Feb 2015 at 5:52

GoogleCodeExporter commented 9 years ago
Дмитрий, fixed?

Original comment by oasyc...@gmail.com on 9 Apr 2015 at 8:42

GoogleCodeExporter commented 9 years ago
Думаю, да.

Original comment by mamontov...@gmail.com on 10 Apr 2015 at 6:12

GoogleCodeExporter commented 9 years ago

Original comment by oasyc...@gmail.com on 3 Jul 2015 at 8:17