1. C# / Говнокод #4426

    +114

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    public class XmlSettings
{
    XDocument x = XDocument.Load(@"...Settings.xml");
    public XDocument Settings { get { return x; } }
}

    Manager настроек из предудыщего проекта

    Запостил: HIMen, 23 Октября 2010

    Комментарии (4) RSS

    • ava Nigma143 23.10.2010 18:13 # 0

      ну и? Загрузка xml документа при init класса, далее отдача свойством Settings где возврат XDocument(как понимаю это ваш класс где уже распарсин xml документ).
      Что минус - жёсткая привязка пути сеттинга, хотя бы в константу надо было бы вывести.
      Ответить

      • ava Анонимус 23.10.2010 18:48 # +3

        Это совершенно ненужный класс, говорящий о дерьмовости архитектуры.
        Если бы он хотя бы имплементил интерфейс (FileBasesSettings : IXmlSettings) было бы лучше
        Ответить

        • ava Nigma143 23.10.2010 19:13 # 0

          Про архитектуру некто тут и не говорил. Бог его знает для чего он именно так сделал. По факту я судил код а не архитектуру
          Ответить

    • ava Lure Of Chaos 23.10.2010 22:00 # 0

      был бы он хотя бы статик
      Ответить

    Добавить комментарий