2rameel: предложения по AssemblyExtensions
От: Sinix  
Дата: 14.04.16 06:37
Оценка:
Привет!

Только собрался добавить, как увидел, что уже сделано

Пара мелких предложений:

1. Вместо uri.AbsolutePath использовать .LocalPath, он с учётом соглашений текущей ОС работает.
2. В сообщениях исключений указывать имена сборки/ресурса. Дико экономит время, особенно если всё, что до тебя дошло — это текст исключения.
3. В GetAssemblyDirectory
    public static string GetAssemblyDirectory([NotNull] this Assembly assembly) =>
        Path.GetDirectoryName(GetAssemblyPath(assembly)) ?? "";

возвращать результат как есть. На крайний случай — не пустую строку, а ".\"
Re: 2rameel: предложения по AssemblyExtensions
От: rameel https://github.com/rsdn/CodeJam
Дата: 14.04.16 08:54
Оценка:
Здравствуйте, Sinix, Вы писали:

S>Только собрался добавить, как увидел, что уже сделано


Это не моя заслуга) Это Андрей делал, я только вынес в отдельный файл все, что было связано с Assembly.

S>Пара мелких предложений:


Ok, как буду у компьютера гляну
... << RSDN@Home 1.0.0 alpha 5 rev. 0>>
Re[2]: 2rameel: предложения по AssemblyExtensions
От: AndrewVK Россия http://blogs.rsdn.org/avk
Дата: 14.04.16 10:02
Оценка: 16 (2)
Здравствуйте, rameel, Вы писали:

S>>Пара мелких предложений:

R>Ok, как буду у компьютера гляну

Я уже поправил
... << RSDN@Home 1.0.0 alpha 5 rev. 0 on Windows 8 6.2.9200.0>>
AVK Blog
 
Подождите ...
Wait...
Пока на собственное сообщение не было ответов, его можно удалить.