Re: Rsdn.Editor
От: Аноним  
Дата: 17.01.06 16:42
Оценка:
У меня парочка мыслей появилась после прочтения статьи и беглово знакомства с исходниками:
1. Зачем делать word wrap для строк которые не видны на экране? Лишняя работа.
2. Для адресации позиции в тексте достаточно знать смещение символа от начала текста, а уж строку которой принадлежит символ и смещение в строке можно расчитать. Тогда можно было бы обойтись без Position<View> и Position<Document>. Понятно, что если использовать, грубо говоря, string[] в качестве хранилища текста, то вариант с адресацией символов по номеру строки и смещению выглядит естественно. IMHO простота string[] обманчива. Упретесь рано или поздно в масштабируемость.
3. IMHO не получилось полностью отделить документ от вида. Зачем в Document-е ссылки на IView и Formatter? IMHO разумно в классе Document держать только код для доступа к тексту, его изменеию (insert/erase) и undo/redo. Модификация документа IMHO никак не связана с тем как он отформатирован.

Алексей
 
Подождите ...
Wait...
Пока на собственное сообщение не было ответов, его можно удалить.