Search
Write a publication
Pull to refresh

Comments 6

Во первых спасибо!

И пара вопросов.
1. Зачем вам свойство List в модуле Iso3166Countries? (Ну и вообще класть в переменную list словарь — необычно, лучше уж map или countriesLookup или вроде того :))
2. Почему бы не возвращать целиком Iso3166Country? То бишь сделать дополнительный метод. Может кому понадобится, а форкать будет лениво.
3. Можно в Iso3166Country сделать readonly поля вместо свойств, будет ощутимо быстрее на рантаймах где нет или плохой инлайнинг (моно этим грешит). Единственное не уверен насчет разного рода MVVM'ов, там наверное может оказаться что на полях не работает биндинг (проверить бы)

Еще тесты покрывают только удачные кейсы, ни null, ни string.Emtpy.
Еще наверное было бы очень удобно добавитьфлажок CaseSensitive. Часто страны хранят в LC.
Да не за что!

1. Ага, точно, переименую.
2. Согласен, сделаю.
3. Не, оставлю проперти. С биндингом действительно могут быть проблемы.

По тестам и специальным случаям — доработаю, упустил. Ну знаете, как это бывает — жадно и быстро :-)
CaseSensitive — хорошая идея. Тоже добавлю, пожалуй!

Спасибо!
UFO landed and left these words here
Оно ведь итак без подключения к Интернету. Зашито в Dictionary в коде. Отпарсено 1 раз.
Вы точно уверены, что этой поддержки нет? MSDN убеждает в обратном — классы CultureInfo и RegionInfo.

Или все дело в Андорре?

CultureInfo[] cinfo = CultureInfo.GetCultures(CultureTypes.AllCultures & ~CultureTypes.NeutralCultures);
foreach (CultureInfo cul in cinfo)
{
var ri = new RegionInfo(cul.Name);
var name = ri.TwoLetterISORegionName;
}
Да, она есть, конечно. Но того, чего нужно было — в самый ответственный момент не хватило.
Как-то забавно вышло с Андоррой, да? :-)
Наверное, если постараться, можно еще кейсы найти.
Sign up to leave a comment.

Articles