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

Issue #323 closed
Oleg Sychev repo owner created an issue

Originally reported on Google Code with ID 323

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

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

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

Reported by oasychev on 2015-01-14 19:04:46

Comments (4)

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

    Reported by mamontov.dp on 2015-02-18 05:52:34

  2. Former user Account Deleted
    Думаю, да.
    

    Reported by mamontov.dp on 2015-04-10 06:12:03 - Status changed: Fixed

  3. Log in to comment