1.4.0-beta1
От: AndrewVK Россия http://blogs.rsdn.org/avk
Дата: 18.05.17 07:03
Оценка:
https://www.nuget.org/packages/CodeJam/1.4.0-beta1

What's new in 1.4.0-beta1
-------------------
* IsDebugAssembly moved from ReflectionExtension to AssemblyExtension
* ReflectionExtension.GetModulePath method added
* Refactoring and code cleanup

... << RSDN@Home 1.0.0 alpha 5 rev. 0 on Windows 8 6.2.9200.0>>
AVK Blog
Re: 1.4.0-beta1
От: Дьяченко Александр Россия  
Дата: 21.05.17 16:49
Оценка:
Здравствуйте, AndrewVK.

Замечания и предложения:

1. В коментарии TableData/DataLine.cs#L16 должен быть не Object, а DataLine.

2. В папке [RangeExtensions] для операций над Range/CompositeRange стоит добавть атрибут [Pure].
... << RSDN@Home 1.0.0 alpha 5 rev. 0>>
Re[2]: 1.4.0-beta1
От: Sinix  
Дата: 21.05.17 18:00
Оценка: 4 (1)
Здравствуйте, Дьяченко Александр, Вы писали:

ДА>2. В папке [RangeExtensions] для операций над Range/CompositeRange стоит добавть атрибут [Pure].


Done. Nightly build в артефактах сборки, нюгет-фид — https://ci.appveyor.com/nuget/codejam
Re[3]: 1.4.0-beta1
От: Дьяченко Александр Россия  
Дата: 22.05.17 13:49
Оценка:
Здравствуйте, Sinix, Вы писали:

ДА>>2. В папке [RangeExtensions] для операций над Range/CompositeRange стоит добавть атрибут [Pure].


S>Done. Nightly build в артефактах сборки, нюгет-фид — https://ci.appveyor.com/nuget/codejam


Спасибо.

Добавь еще [NotNull] к следующим методам:

1. public static Range&lt;T&gt;[] GetIntersection&lt;T&gt;(this CompositeRange&lt;T&gt; compositeRange, T value)

2. public static Range&lt;T, TKey&gt;[] GetIntersection&lt;T, TKey&gt;(this CompositeRange&lt;T, TKey&gt; compositeRange, T value)
... << RSDN@Home 1.0.0 alpha 5 rev. 0>>
Re[3]: 1.4.0-beta1
От: Дьяченко Александр Россия  
Дата: 22.05.17 13:54
Оценка:
Здравствуйте, Sinix!

Еще в этом файле Main/src/Ranges/Range.cs у private и internal методов нет атрибута [Pure] или вы его применяете только для public методов?
... << RSDN@Home 1.0.0 alpha 5 rev. 0>>
Re[4]: 1.4.0-beta1
От: Sinix  
Дата: 22.05.17 19:26
Оценка: 4 (1)
Здравствуйте, Дьяченко Александр, Вы писали:

ДА>Добавь еще [NotNull] к следующим методам:

Done.

ДА>Еще в этом файле Main/src/Ranges/Range.cs у private и internal методов нет атрибута [Pure] или вы его применяете только для public методов?

Лично я — да, только к public api. Иначе этим атрибутом придётся размечать процентов девяносто кода
 
Подождите ...
Wait...
Пока на собственное сообщение не было ответов, его можно удалить.