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

    +129

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    public void UpdateSession()
        {
            this.RequestTypeId = this.RequestTypeId;
            this.ServiceId = this.ServiceId;
            this.Name = this.Name;
        }

    "Обновление" сессии. Взято из реального проекта.

    Запостил: wader, 03 Мая 2011

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

    • ava abatishchev 03.05.2011 10:48 # 0

      внутри гетера - скореее всего, чтение из Request/Session/Application
      внутри сетера - обработка
      ваш К.О.

      кто-то считает это корявым, я же к примеру - нет
      всё лучше, чем UpdateSession(GetSession())
      Ответить

      • ava wader 03.05.2011 10:51 # +2

        спасибо, кэп :)

        там что-то типа:

        public int? RequestTypeId
        {
        get
        {
        if (Session["Value_xx"] != null)
        {
        return Convert.ToInt32(Session["Value_xx"]);
        }
        else
        {
        return 0;
        }
        }
        set
        {
        Session["Value_xx"] = value;
        }
        }

        может объясните: зачем нужна такая обработка?
        Ответить

        • ava abatishchev 03.05.2011 11:03 # 0

          на самом деле лучше так: 
          return Session["value"] as int? ?? 0;
          ааа
          если так, то это гавно код, конечно

          вот если бы
          set
{
    this.Session["value"] = value;
    switch (value)
    {
        case (int)RequestType.Foo:
            rowFoo.Visible = true;
        case (int)RequestType.Bar:
            rowBar.Visible = true;
    }
}
          тогда было бы вполне вменяемо
          Ответить

          • ava wader 03.05.2011 11:09 # +1

            ну да, виноват, не привел сразу код обработки. Но там действительно так, как я написал чуть ниже первого поста. Поэтому и решил запостить этот "код".
            Ответить

    • ava TarasB 03.05.2011 11:37 # −6

      показать все, что скрытоЗа это я не люблю проперти.
      Ответить

      • ava absolut 03.05.2011 11:58 # 0

        просто шарп - это двойные кресты
        Ответить

        • ava guest 03.05.2011 12:07 # −3

          кресты хоронят проекты, значит двойные в двойне?
          Ответить

      • ava wader 03.05.2011 12:14 # −4

        Возможно, "Вы просто не умеете их готовить" (с)

        Свойства очень хорошо подходят для описания полей сущностей БД (что первое пришло в голову).
        Ответить

        • ava absolut 03.05.2011 13:07 # −3

          перестрелка между ментами и бандосами.
          - сержант Петров, доложите обстановку.
          - пуля ...
          - что пуля?
          - я назвал первое, что попало мне в голову.
          Ответить

        • ava zlob.jc 03.05.2011 17:21 # −1

          Явное лучше, чем неявное.
          Сложное лучше, чем запутанное.
          Ответить

          • ava guest 03.05.2011 20:29 # 0

            +1
            Скотт Мэерс вроде говорил...
            Ответить

            • ava guest 03.05.2011 20:37 # −1

              философия питона, но на столько универсально что подходит для всех языков и не только. читать как мантру до полного просветления.
              http://ru.wikipedia.org/wiki/Python#.D0.A4.D0.B8.D0.BB.D0.BE.D1.81.D0 .BE.D1.84.D0.B8.D1.8F
              Ответить

          • ava guest 03.05.2011 21:06 # +2

            вместо пафосных рассуждений, не могли бы Вы привести конкретные аргументы не говнокодности этого творчества?
            Ответить

            • ava bugmenot 03.05.2011 21:24 # −2

              поциэнт пишет на до-диезе, значит - не осилил ни один из ЯВУ
              следовательно: не кидайте в него говном, он пишет как умеет

              сойдёт?
              Ответить

              • ava guest 03.05.2011 21:50 # 0

                может он освоил много, откуда знаешь, олух?
                Ответить

                • ava bugmenot 03.05.2011 22:29 # 0

                  по тебе сужу, няше4ка
                  некоторые задроты позорят анонимуса как коллективную сущность
                  Ответить

              • ava wader 03.05.2011 21:58 # 0

                поциент, если верить его словам, пишет на C# уже около 3 лет
                Ответить

                • ava bugmenot 03.05.2011 22:30 # 0

                  а до того?
                  три года это как бы мало для становления
                  Ответить

                  • ava wader 04.05.2011 07:09 # +1

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

    • ava guest 06.05.2011 03:26 # 0

      Код реально тупой. 101%
      Ответить

      • ava guest 06.05.2011 08:30 # 0

        наследование классов - для задротов
        правильные посоны перебирают все свойства по отдельности
        Ответить

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