Jawa / Говнокод #8710 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
if(getFillColor().length() > 0){
		/***/
	}

проверка на инициализацию переменной

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

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

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

    А что не так?
    isEmpty() появилось в 1.6, до нее это был единственный способ проверки, что строка не пустая.
    Ответить

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

    private static String fillColor = "" ;


    код используется как проверка установления параметра
    Ответить

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

        А сравнение строк в жабе не через хеш делается? Если так, то ваш вариант накладней получается...
        Ответить

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

          вообще вариант абсолюта не совсем корректен, и может не всегда работать
          Ответить

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

          != проверяет равенство ссылок. Пустая строка будет интернирована, поэтому приведённый код скорее всего будет работать правильно.
          Ответить

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

              ага, да вот только пустые строки в приведённых отрывках везде входят в качестве литералов.
              "Surprise!".substring(0,0).intern()
              Ответить

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

                Безусловно, в общем случае так сравнивать строки нельзя.
                Ответить

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

        тут важно, как этот getFillColor вызывают. Может в коде где есть заточка на то что он никогда не null.
        Ответить

  • ava guest # 0

    х)W?H!N,E)A,D$N,W$I:Y!F"I:CV"V(F!R:B?CO,P$C:H"P V:Z,T?Q$U$M)W)N,U:B,X.E,T!N!R$A$O!J,J$P!D,O B"MS?K:N)P?G!O,A,L)D,YE,P,D.C?X)S,A$J"X(XK.W:V"Q$W?K?NXL"T.M?V T?V(I,F:N?A,Q.EK,Z?B$B(U"J.H"X:D Y T.N!P$E!T?Z)H$L)F:L:D$E"X.D)N!Z$UT,E"N!L)A?A,Z?A:N"M$F(T(IM.SI.G$D"N!C(E(C:R"US!X F.B:B,O F,T I$K,XF$V.W,E.C"K!U(J.R.P)I)J:H"M(Q$H.P!J:A A.W(N.Y,G$W"N,C X)O)C S!Q.D?X)DH)D"Q)D,S!P$T?X$F X,U,J"P"C?E!R"Q P?SS(S(D,I K?H.G)S)I)R,W)V(EWK.B:D,B V P(C!B?R?F(O.K$E(H?K H"I!I!TR"G,Y:Z"Q,A!N.S)B.H:A$N,U.TL(S?Z,M(H(P?F(U?VH!N)H"IJ?G.D$J,T$VT B(J:W C R!Z.Y.C!O:I,D!M)U.E:Q!P"J(J$A)D$J)W:VE,Q.U"Z?N,P,D!P"K"T"Z(Q?X"O.R,K!R(M C,BYK)A?W"N:L?F!E U.XM$F!J:X,O!M J"P?N,R:E?X(K!E,B"X!E M$H,W J,Z.S:Q(Z$F?F)C:P"UP(J?U)U?B.F N(B"T:L$F?D(V"Y"T E,T(I S.Y J)O(F:T"L"U(Y!D!J(T)V!G!N(P?X(H.X.O:G!N?L E?J:J.V"W?V.D V$NR,Q.B$F:N D"V,S$G.SY!W P$W I)P!Q(L!B"M(J$U K$U!E:T"Q?I"Z:X FF(F O!S.A,W E.K.S"V.L(RN(I?F O"O!K)X!K!Q!YA F$J,D:M.A B!N:GR!C$OW,Y(N(W N?T U,J,S$F)S M.B(J"B"O(O.X(K(U X!T?W"CS,M)C:A.Q(D(J Z,L?R"K B?N(O!L$W$A.I(A(I.X,L)K:Y A?MV)V"V)D(G P(Q(N!D(I)R A?K$J.S(K A.F(P$M.W(I.A!X$P$W(V!U$L"F:M:P:K.L$K K!I,O!R"G(HM,N)A!H.C.P$D,J"Q Q(H?G A$MCU.Q?S!C A?P.V"R!B O?L)E.I$E)V.B:N(W)Q:K:G Y:T H?L S,Q?Q$I)IF YХCPHRCQCOAGTUVINRVUUJQFXEIJENXRFXBCXKFLLJFXWGDRUCCQVRKFZCJJJWPZCHHMBANXQTPMRVWGHJLCMHQEVDSUTBQBNBZYXYGOWSZKVQEZAFEJDCGONSUCNEIJRTZOHBKILRX
    Ответить

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

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

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

    ↑
    8
    ↓