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

    +132

    1. 1
    Удалено по просьбе автора

    Запостил: alt0, 08 Августа 2014

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

    • ava TauSigma 08.08.2014 17:27 # 0 

      public void Clone(int userId, int? id)//По иерархии, идЮзера в приоритете должен быть
{
    if(id==null) throw new ArgumentNullException("id","Куда шаблонъ дел?");

    using (var context = this.GetContext())
        context.CopyTemplate(userId, id.Value);
}


      Хотя если это реализация интерфейса, то не прокатит.
      Ответить

      • ava kegdan 08.08.2014 17:37 # 0

        зачем id в nullable обернул?
        Ответить

        • ava TauSigma 08.08.2014 17:40 # 0

          Ну, это, наклал... 0:)
          Я сначала хотел внешнюю часть не трогать, чтобы оставить для инкапсуляции, а потом резулт убрал, а про нуллаблю забыл.
          Ответить

          • ava kegdan 08.08.2014 17:43 # 0

            Это ж как надо кодить, что бы программа в один прекрасный момент начал юзеров из нихера создавать)
            Ответить

            • ava bormand 08.08.2014 17:44 # 0

              > начал юзеров из нихера создавать
              Да она же просто упадёт на касте в шестой строке.
              Ответить

              • ava kegdan 08.08.2014 17:50 # 0

                Ясен пень, я про логику вообще. То есть может случиться, что вместо шаблона придет null - говно же
                А сверху 

                try
    {
     ....   Clone(ShittyTemplateID, ShittyID);
    }
    catch (InvalidOperationException)
    {

        CraeteUserWithoutTemplate(ShittyID);
    }
                Ответить

              • ava TauSigma 08.08.2014 17:54 # 0

                Nullable<Int32> в Int32 скастуется.
                Но при implicit преобразовании может вылететь в InvalidOperationException.
                Ответить

                • ava bormand 08.08.2014 18:39 # 0

                  > Nullable<Int32> в Int32 скастуется.
                  Даже при null'е? О_о
                  Ответить

                  • ava TauSigma 08.08.2014 18:47 # +1

                    Не, при null'е оно бросит InvalidOperationException.
                    Ответить

                  • ava kegdan 08.08.2014 18:48 # +3

                    ну так когда начнет неявно кастоваться -тогда и сдохнет. Об этом и говорил Тау Кита Сигма
                    Ответить

                    • ava TauSigma 08.08.2014 19:02 # 0

                      Странно, что при добавлении структуры Nullable<T>, M$ не добавил годный парс:
                      public static class NullableParse
{
    public static Int32? ToInt32(this String prm)
    {
        Int32 result;
        return Int32.TryParse(prm, out result)?result:(Int32?)null;
    }
    //....
}
                      Ответить

                      • ava kegdan 08.08.2014 19:06 # 0

                        nullable нинужно нужно в очень редких случаях, да и то проще свой класс описать ибо, скорее всего, понадобятся дополнительные фичи
                        Ответить

                        • ava TauSigma 08.08.2014 19:22 # 0

                          Нуллаблы нужны для работы с СУБД, дабы не городить конструкций с нулом, как это было с типизированными датасетами:
                          https://connect.microsoft.com/VisualStudio/feedbackdetail/view/284009/nullable-types-for-typed-dataset

                          >да и то проще свой класс описать ибо, скорее всего, понадобятся дополнительные фичи
                          В нынешних ORM'ах, весь "свой" класс идёт в partial.
                          Ответить

            • ava TauSigma 08.08.2014 17:50 # 0

              DAL нам на некий шаблон намекает.
              Ответить

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