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

    +139

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    public string ValidationTitle
{
    get { return this.Title; }
    set { }
}

    Запостил: ArtemASP, 03 Марта 2014

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

    • ava Vasiliy 03.03.2014 16:22 # −3

      Где тут C++ ?
      Ответить

    • ava someone 04.03.2014 10:11 # −2

      Обратная совместимость?
      Ответить

      • ava guest 18.03.2014 10:16 # 0

        возможно что интерфейс такой задан... и заимплеменчен... другое дело, зачем
        Ответить

    • ava absolut 04.03.2014 11:01 # +1

      говно в пустом сеттере, как минимум.
      Ответить

      • ava Soul_re@ver 04.03.2014 11:50 # +1

        А как шарп ведёт себя в таком случае? Позволяет присвоить свойству значение, но игнорирует его?
        Ответить

        • ava Sabrian 04.03.2014 11:53 # +2

          Да.
          Ответить

        • ava cyperh 04.03.2014 16:18 # +2

          Никто никого не игнорирует, будет произведен вызов аксесора set, в него передастся value, неявный входной параметр аксесора set, но так как тело пустое просто ничего не произойдет.
          Ответить

      • ava inkanus-gray 04.03.2014 18:22 # +2

        Почему же говно? Может быть, по условию задачи требуется свойство только для чтения. Или по канону в этом случае сеттер должен бросать исключение?
        Ответить

        • ava absolut 05.03.2014 08:11 # +1

          Убрать сеттер к ебеням и будет нормальный ридонли. А вот если интерфейс нужно поддерживать, то лучше уж с исключением.
          Ответить

          • ava chtulhu 05.03.2014 08:34 # −1

            >public string ValidationTitle
            на интефейс не похоже. В шарпике override надо писать
            Ответить

            • ava absolut 05.03.2014 11:47 # −1

              > В шарпике override надо писать
              путаешь интерфейс с базовым классом.
              http://ideone.com/T67I3v
              Ответить

          • ava TauSigma 26.03.2014 16:20 # 0

            Если это в LINQ to SQL (Entity), как partial класс (не помню как маппер называется).
            Или код из WCF (DataContractSerializer), то пустой сеттер просто нужен. Иначе рантайм исключение из дебрей .NET'а...

            Для красоты можно его спрятать в private set {}.
            Ответить

        • ava MHz 24.03.2014 11:05 # 0

          > Может быть, по условию задачи требуется свойство только для чтения.
          Зловонная кучка скрывается в том, что для стороннего разработчика свойство выглядит публичным. И он сильно удивится, когда простое изменение перерастет в карнавал с рефлексией.
          Ответить

    • ava Lure Of Chaos 04.03.2014 17:22 # +3

      такой себе readonly
      Ответить

    • ava roman-kashitsyn 04.03.2014 18:32 # +4

      Помню, коллега рассказывал про не менее забавный индусский 
      public bool Updated
{
    get { this._updated = true; }
}
      Ответить

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