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

    +136

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    protected Boolean GetStatus(VinRequestStatusId statusId, Int16 id)
{
	if(statusId == VinRequestStatusId.CarDataNeeded && id == 0)
		return true;
	if(statusId == VinRequestStatusId.WaitingClientAnswer && id == 1)
		return true;
	return false;
}

    Разработчику гуя лень было лезть в DAL. Наложил прямо на форме.

    Запостил: TauSigma, 01 Июля 2013

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

    • ava kostoprav 01.07.2013 16:50 # 0

      Boolean в качестве статуса тоже доставляет.
      Ответить

      • ava TauSigma 01.07.2013 17:48 # −1

        Да ещё и id передаётся только 0 или 1.
        Автор решил так централизировать проверку на разные статусы:
        ctrlChangeCar.Visible=GetStatus(dataItem.Status,0);
    ctrlClientAnswer.Visible=GetStatus(dataItem.Status,1);

        вместо:
        ctrlChangeCar.Visible=dataItem.Status==VinRequestStatusId.CarDataNeeded;
    ctrlClientAnswer.Visible=dataItem.Status==VinRequestStatusId.WaitingClientAnswer;

        Хотя правильнее было вынести в DAL, т.к. сборка парралельно используется ещё в нескольких решениях.
        Т.е. примерно так:
        partial class SomeData {
    public Boolean IsChangeCarRequired { get { return this.Status==VinRequestStatusId.CarDataNeeded; } }
    public Boolean IsAnswerNeeded { get { return this.Statis==VinRequestStatusId.WaitingClientAnswer; } }
    pbulic Boolean IsUserInteractionNeeded { get { return this.IsChangeCarRequired || this.IsAnswerNeeded; } }
}
        Ответить

    • ava kegdan 01.07.2013 17:03 # +1

      А сам класс выглядит как то

      internal class VinRequestStatusId
      {
      protected static VinRequestStatusId CarDataNeeded;
      protected static VinRequestStatusId WaitingClientAnswer;
      }

      Это не богомерзко?
      Ответить

      • ava TauSigma 01.07.2013 17:39 # +1

        Это енум, а не класс...
        enum VinRequestStatusId {
    None,
    CarDataNeeded,
    WaitingClientAnswer,
    ...
}

class SomeData {
    public VinRequestStatusId Status {get;}
}
        Ответить

        • ava bormand 01.07.2013 17:50 # +1

          > Это енум, а не класс...
          Но в то же время енум это класс ;)

          UPD: Тьфу ты, это же шарп а не жаба... Попутал, сорри.
          Ответить

          • ava kostoprav 01.07.2013 17:54 # 0

            Не в шарпике )
            Ответить

          • ava TauSigma 01.07.2013 18:00 # 0

            Эх, руки чешуться до жабы добраться...
            Ответить

            • ava roman-kashitsyn 01.07.2013 18:07 # 0

              с какой целью добраться?
              Ответить

              • ava eth0 01.07.2013 18:45 # +9

                Облизать. Некоторые виды жаб содержат галлюциногены в кожных выделениях.
                Ответить

                • ava Lure Of Chaos 01.07.2013 19:42 # 0

                  целовать, это же так феншуйно.
                  Ответить

                • ava bormand 01.07.2013 21:20 # +3

                  > содержат галлюциногены
                  Поцеловал Иван-Царевич лягушку жабу, и превратилась она в царевну-красавицу...
                  Ответить

                  • ava Lure Of Chaos 01.07.2013 23:52 # +1

                    а люди глядели на него и фигели: "мало тебе девок было, извращенец проклятый? перестань же мучать животное, урод!"
                    Ответить

                    • ava guest 02.07.2013 00:01 # −18

                      показать все, что скрыто
                      * g o a t s e x * g o a t s e x * g o a t s e x *
g                                               g  
o /     \             \            /    \       o
a|       |             \          |      |      a
t|       `.             |         |       :     t
s`        |             |        \|       |     s
e \       | /       /  \\\   --__ \\       :    e
x  \      \/   _--~~          ~--__| \     |    x  
*   \      \_-~                    ~-_\    |    *
g    \_     \        _.--------.______\|   |    g
o      \     \______// _ ___ _ (_(__>  \   |    o
a       \   .  C ___)  ______ (_(____>  |  /    a
t       /\ |   C ____)/      \ (_____>  |_/     t
s      / /\|   C_____)       |  (___>   /  \    s
e     |   (   _C_____)\______/  // _/ /     \   e
x     |    \  |__   \\_________// (__/       |  x
*    | \    \____)   `----   --'             |  *
g    |  \_          ___\       /_          _/ | g
o   |              /    |     |  \            | o
a   |             |    /       \  \           | a
t   |          / /    |         |  \           |t
s   |         / /      \__/\___/    |          |s
e  |           /        |    |       |         |e
x  |          |         |    |       |         |x
* g o a t s e x * g o a t s e x * g o a t s e x *
                      Ответить

                  • ava inkanus-gray 02.07.2013 00:18 # +4

                    А я другую сказку вспомнил:
                    — Не пей, братец Иванушка, из питушиного следа...
                    Ответить

              • ava TauSigma 01.07.2013 18:49 # 0

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

            • ava Vasiliy 01.07.2013 18:22 # 0

              тфу блин прочитал до жопы.
              Ответить

        • ava kegdan 01.07.2013 19:42 # 0

          Сорри, тупанул)
          Ответить

          • ava TauSigma 01.07.2013 20:17 # 0

            Понимаю... Изредка от говнокода крышу сносит.
            Тут одному коллеге дал вроде как простенький говнокод, так он чуть factory не наваял...
            Ответить

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