Re[5]: DisjointSets и SuffixTree
От: Lexey Россия  
Дата: 13.11.16 17:10
Оценка:
Здравствуйте, AndrewVK, Вы писали:

AVK>>>2) Названия методов FindBranchingPoint и CreatePendingLink как то не очень коррелируют с void

L>>Есть предложения по другим названиям?

AVK>Для этого надо вникать в сам алгоритмы. А я чисто внешне, по возникшим WTF при просмотре кода.


Сделал изменения по 3-м из твоих замечаний:
https://github.com/rsdn/CodeJam/pull/28

Tuple трогать не стал, ибо там null нужно уметь возвращать (хотя, в принципе можно сделать и структуру, но тогда придется в нее пихать null в качестве Item1 и проверять его на null, что, КМК, сделает логику более запутанной).
"Будь достоин победы" (c) 8th Wizard's rule.
Re[6]: DisjointSets и SuffixTree
От: AndrewVK Россия http://blogs.rsdn.org/avk
Дата: 13.11.16 17:33
Оценка:
Здравствуйте, Lexey, Вы писали:

L>Tuple трогать не стал, ибо там null нужно уметь возвращать


ValueOption<ValueTuple<T>> или Nullable<ValueTuple<T>>, не?
... << RSDN@Home 1.0.0 alpha 5 rev. 0 on Windows 8 6.2.9200.0>>
AVK Blog
Re[7]: DisjointSets и SuffixTree
От: Lexey Россия  
Дата: 13.11.16 19:08
Оценка:
Здравствуйте, AndrewVK, Вы писали:

AVK>ValueOption<ValueTuple<T>> или Nullable<ValueTuple<T>>, не?


Вот, про наличие ValueOption не знал. Переделал на него.
"Будь достоин победы" (c) 8th Wizard's rule.
Подождите ...
Wait...
Пока на собственное сообщение не было ответов, его можно удалить.