1. Java / Говнокод #16736

    +73

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    String getFindList(StringBuffer sb, String[] src) {
    // int tid = Helper.parseType(src[2]);
    // if(tid < 1 || tid > 99) return "Error parse good type";
    int stk = Helper.parseType(src[3]);
    if(stk < 412 || stk > 416)	return "Error parse stock code";
    // --------------------------------------------------------
    IntHashtable work = new IntHashtable();
    double[] vals = null;
    Entry ent = null;
    Enumeration e=cache.getEntryElements();
    while(e.hasMoreElements()) {
    ent = (Entry)e.nextElement();
    if(ent.Credit != stk)	continue;
    if(ent.Status == 0)	continue;
    vals = (double[])work.get(ent.SubCred);
    if(vals == null) {
    vals = new double[2];
    vals[0] = ent.Value;
    work.put(ent.SubCred, vals);
    } else
    vals[0]+=ent.Value;
    }
    // ---------------------------------------------------------
    String s="SELECT  ... ";
    int id;
    String cod, gnm, uni;
    double amt,val,pack,vlr;
    double[] prcs = new double[4];
    Connection con = cache.getConnection();
    if(con == null)	return "No free conection";
    try {
    Statement stmt = con.createStatement();
    ResultSet rset = stmt.executeQuery(s);
    while(rset.next()) {
    id = rset.getInt("id");
    cod = rset.getString("code");
    gnm = rset.getString("name");
    uni = rset.getString("unit");
    pack = rset.getDouble("pack");
    amt = rset.getDouble("amount");
    val = rset.getDouble("value");
    prcs[0] = rset.getDouble("price");
    prcs[1] = rset.getDouble("price1");
    prcs[1]=(prcs[1] < 0.01) ? prcs[0] : prcs[1];
    prcs[2] = rset.getDouble("price2");
    prcs[2]=(prcs[2] < 0.01) ? prcs[1] : prcs[2];
    prcs[3] = rset.getDouble("sprice");
    prcs[3]=(prcs[3] < 0.01) ? prcs[2] : prcs[3];
    vals = (double[])work.get(id);
    vlr =(vals == null) ? 0 : vals[0];
    sb.append(id+",'"+gnm+"','"+cod+"','"+uni+"',"+pack+","+val+","+vlr+",");
    for(int i=0; i<prcs.length; i++)	sb.append(prcs[i]+",");
    sb.append((amt/val)+",\n");
    }
    rset.close();
    stmt.close();
    s = null;
    } catch (SQLException ex) {
    s=ex.getMessage();
    }
    cache.freeConnection(con);
    return s;
    }

    qw0, 21 Сентября 2014

    Комментарии (0)
  2. Java / Говнокод #16724

    +74

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    url = new URL("http://[FF01:0:0:0:0:0:0:0101]");
    String originalHostName = url.getHost();
    System.out.println(originalHostName);
    if (originalHostName.startsWith("[") && originalHostName.endsWith("]")) {
     i1 = originalHostName.indexOf("[");
     originalHostName = originalHostName.substring(i1 + 1);
     i2 = originalHostName.lastIndexOf("]");
     originalHostName = originalHostName.substring(0, i2);
     if (IPV6REGEX.matcher(originalHostName).matches()) {
      System.out.println(url + " is a ipv6 address");
      String hostName = InetAddress.getByName(originalHostName).getHostAddress().toLowerCase();
      if (hostName.contains(":")) {
       hostName = "[" + hostName + "]";
      }
     }
    }

    Взято отсюда: http://www.pretechsol.com/2013/05/ipv6-java-regular-expression-example.html#.VBtooLn69B0

    0rt, 19 Сентября 2014

    Комментарии (2)
  3. Java / Говнокод #16719

    +82

    1. 1
    Boolean hasRefId = !node.getAttributes().getNamedItem("refid").equals(null);

    equals(null)

    roman-kashitsyn, 18 Сентября 2014

    Комментарии (10)
  4. Java / Говнокод #16718

    +76

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    function returnUserId() {
        if (document.getElementById('userstap') != null )  {
            var obj = document.getElementById('userstap');
            <% if ((sUserIdForPrint != null) && !sUserIdForPrint.equals("")) {%>
            if (obj.options[obj.selectedIndex].value == 0) {
                return "<%=sUserIdForPrint%>";
            <% } %>
            <% if (((sAccGrpUserIdForPrint != null) && !sAccGrpUserIdForPrint.equals("")) && ((sUserIdForPrint != null) && !sUserIdForPrint.equals(""))) {%>
            } else if (obj.options[obj.selectedIndex].value == 1 ) {
                return "<%=sAccGrpUserIdForPrint%>";
            <% } else if ((sAccGrpUserIdForPrint != null) && !sAccGrpUserIdForPrint.equals("")){%>
            if (obj.options[obj.selectedIndex].value == 1 ) {
                return "<%=sAccGrpUserIdForPrint%>";
            <% } %>
            <% if (((sAccGrpUserIdForPrint != null) && !sAccGrpUserIdForPrint.equals("")) || ((sUserIdForPrint != null) && !sUserIdForPrint.equals(""))) {%>
            } else if (obj.options[obj.selectedIndex].value == 2) {
                return "<%=userId%>";
            }
            <% } else {%>
            if (obj.options[obj.selectedIndex].value == 2) {
                return "<%=userId%>";
            }
            <% } %>
        } else {
            return "<%=userId%>";
        }
    }

    Любите ли вы JSP так, как люблю его я?

    codingHorror, 18 Сентября 2014

    Комментарии (14)
  5. Java / Говнокод #16711

    +79

    1. 1
    2. 2
    // Number a, Number b
    return new Double(a.doubleValue() - numericTolerance).compareTo(b.doubleValue()) < 0

    В продолжение #16698, после рефакторинга. Видимо, диагноз: ООП головного мозга.

    Actine, 17 Сентября 2014

    Комментарии (14)
  6. Java / Говнокод #16698

    +73

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    public static <T extends Comparable<T>> boolean isLessThan(T a, T b, double numericTolerance) {
        if (a == null) {
            return b != null;
        }
        boolean isLessThan = a.compareTo(b) < 0;
        if (!isLessThan && a instanceof Number && b instanceof Number) {
            isLessThan = ((Comparable) (((Number) a).doubleValue() - numericTolerance)).compareTo(((Number) b).doubleValue()) < 0;
        }
        return isLessThan;
    }

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

    Actine, 16 Сентября 2014

    Комментарии (5)
  7. Java / Говнокод #16684

    +64

    1. 1
    int i;

    кода на самом деле нет, но это достойно говнокод.ру

    на сервере был http метод GET /rest/user/anon/create
    по непонятной причине был заменён на POST /rest/user/anon/create1
    старый метод теперь возвращает 405 ошибку "Method not allowed" с заголовком Allow: POST, OPTIONS

    kyzmitch, 12 Сентября 2014

    Комментарии (34)
  8. Java / Говнокод #16676

    +75

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    package com.seriouscompany.business.java.fizzbuzz.packagenamingpackage.impl;
    
    import com.seriouscompany.business.java.fizzbuzz.packagenamingpackage.interfaces.factories.FizzBuzzSolutionStrategyFactory;
    import com.seriouscompany.business.java.fizzbuzz.packagenamingpackage.impl.factories.EnterpriseGradeFizzBuzzSolutionStrategyFactory;
    import com.seriouscompany.business.java.fizzbuzz.packagenamingpackage.interfaces.strategies.FizzBuzzSolutionStrategy;
    
    public class FizzBuzz {
    	public void fizzBuzz(int nFizzBuzzUpperLimit) {
    		final FizzBuzzSolutionStrategyFactory mySolutionStrategyFactory =
    			new EnterpriseGradeFizzBuzzSolutionStrategyFactory();
    		final FizzBuzzSolutionStrategy mySolutionStrategy =
    			mySolutionStrategyFactory.createFizzBuzzSolutionStrategy();
    		mySolutionStrategy.runSolution(nFizzBuzzUpperLimit);
    	}
    }

    FizzBuzz Enterprise Edition на Java от уважаемых разработчиков.

    http://habrahabr.ru/company/abbyy/blog/173885/
    https://github.com/EnterpriseQualityCoding/FizzBuzzEnterpriseEdition

    gost, 10 Сентября 2014

    Комментарии (16)
  9. Java / Говнокод #16667

    +118

    1. 1
    protected static final float PI_2 = (float)(Math.PI / 2.0);

    someone, 09 Сентября 2014

    Комментарии (26)
  10. Java / Говнокод #16647

    +64

    1. 1
    2. 2
    3. 3
    4. 4
    int r = 5;
        if (r ==5) {
            throw new Exception(); 
        }

    air_raptor, 05 Сентября 2014

    Комментарии (20)