Си диез / Говнокод #9521 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
  18. 18
public partial class EditorForm : Form
{
	private List<IEditObject> _readOnlyObjects;
	public IEditObject EditValue {get; set;}
	
	public EditorForm(IEditObject value, List<IEditObject> readOnlyObjects)
	{
		if (value != null)
		{
			EditValue = value;
		}
		
		if (readOnlyObjects != null && readOnlyObjects.Count > 0)
		{
			_readOnlyObjects = readOnlyObjects;
		}
	}
}

Во избежание лишних присвоений.

Запостил: ICELedyanoj ICELedyanoj, (Updated )

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

  • ava absolut # Ссылка на оригинал 0

    Несмотря на проверки, если вызвать EditorForm(null, null) внутри будут null.
    Ответить

    • ava ICELedyanoj # Ссылка на оригинал 0 ⇈

      Ну так именно это меня и повеселило.
      Это же не какой-нибудь метод Init, а конструктор.
      Ответить

        • ava martin # Ссылка на оригинал 0 ⇈

          возможно, что дальнейшую логику наличие там null'ов не смущает и тогда кидатся лишний раз исключениями смысла нет.
          Ответить

          • ava absolut # Ссылка на оригинал 0 ⇈

            Если бы не смущало, не было бы проверок. Скорее всего в дальнейшем просто возникнет совершенно другое исключение. Уже системного характера.
            Ответить

            • ava ICELedyanoj # Ссылка на оригинал 0 ⇈

              Нет. Уже больше месяца работаю над расшифровкой тонн кода, доставшегося после этого автора в наследство, и могу смело утверждать, что это просто такой авторский стиль.
              Чем больше проверок и трайкотчев - тем надежнее - так он считал.
              Доводилось видеть методы, состоящие из одной строчки - подпись на событие UIшного элемента, заключенной в трайкотч, в обработчике идет просто запись в лог. И таких было немало.
              Просто больная фантазия, не ищите смысла 🙂
              Ответить

  • ava guest # 0

    о)U$AA"N$F!R"Z(K)F,L:G(I$W"B"I"A?U:O.LD U,H F?C?O$K D)O!C!O R,X,V.H)E K$X$T"F(HT$P$I.O:Q$G:A?V?Y(PW,FL$O,C"Q V)D,X$Y.L VP!J)S?K.X)Q!T$Y:W:L)YWR"E!O"H)H!L(N)BJ,C,R$H)I,F)N(E$VS!G$E"P(E,Y I!A?Y!E?X!B"G(F.B(J.S?X)O,G?A(M:L?H!L)C F:T)X!P"Y:S!H)N.B W.E)A.W.P)G?C,S I(O"V:X?A I:C L"O,Y)J(DD(U.F,P)L:F:V!I B.W(Q)U.D,J?F.W K R E:Z"N)R$E H,Z,L.C?I,C(F)X(F"J$C(H)B)Z(N,B:N$U.C!G?E"L$U X H$M"Y F!O!C?I,A$KW?N!U,D.I:AW!K$X$DP?O Y:V:UYD(J(N NW$R,Q,X.B$M)F.V D H!J.RW N?K.J O,S?Q,L Y)C)C$E:D:Y.A$R:KG)E E!GC!J)C)G)L)J,G$U!X)T?U M.R!Q H.V?D,I?E(L"X(S.O!QD:TU.W?Q!F!I?A"G?V Y!Y,WW T:BA"D$P?B.C"W"O$N"R.G?M(V(E W WJ"BT:FH(G!J.IV(S)W"H.Z.J,A!I$H"E(X.H?H$A"VR)RL B:H!S,TO?T.T!H,K!JN.M$O,F$M$C$H:E:X!L)E"X?B.L$S"AW S.C H S!Z!W,I?J(Q.A)F:S)U:BI(C.K,P"UC"U(B.Q.C(K,L(YP!G)U(B(K:N!R,M(N C"H?V(S YJ B(T(H(I)Z(V.A$I"E)P.Y(C,T W"XG$RED!F.E$Z.W"N"J?GP,E:R:M)V$B$V,W:Z)H"O K$C)C!I?S:R(T,Q.W?S!WU E)K(C.A$L"W)X(P.I.I"R:W V?X,O.S,I"L:I,L)P?O.Z$L,L,W,D!G,V TS!I(M(I,W:X"Y:Z$E!F"T)IR:A.C,M!L)Z(Q$K!T"L,D$D$N(C E)M?B(G!A"Q$N!O)N$P(C M,G(S)X:O?O!R:B.M:A I$J R:T,F,W$P L,E(I:E?O O!Q"U,H:C$A)K"Q(KK!T"CO"J!O)TN!FR!W"G(N,J,J$D(H.ZC,D!ND?S!I(D?BБSSXRONXMXPGZGLUOJVYQCKUTGYPHZOMSIPZZYDBTOIMIYUYNBHDAELVGZMRFQINEVODNLCRQHFQGYKCYDHUPALMSJXOGCWMJMLGLXWEQLJEFSWVIGMRNICUGGHRTSKOEMPLTWNXGAGATRBMSWVAFQYVKPWCI
    Ответить

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

Семь раз отмерь — один отрежь, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓