1. Список говнокодов пользователя maxt

    Всего: 12

  2. JavaScript / Говнокод #9480

    +155

    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
    data = Ext.JSON.decode(resp.responseText);
                Ext.getCmp('id').setValue(id);
                Ext.getCmp('address').setValue(data.address);
                Ext.getCmp('dateImp').setValue(data.dateImp);
                Ext.getCmp('numberCompl').setValue(data.numberCompl);
                Ext.getCmp('dateReg').setValue(data.dateReg);
                Ext.getCmp('numberOut').setValue(data.numberOut);
                Ext.getCmp('dateOut').setValue(data.dateOut);
                Ext.getCmp('inResolution').setValue(data.inResolution);
                Ext.getCmp('inFio').setValue(data.inFio);
                Ext.getCmp('shortText').setValue(data.shortText);
                Ext.getCmp('dateControl').setValue(data.dateControl);
                Ext.getCmp('dateClose').setValue(data.dateClose);
                Ext.getCmp('dateChange').setValue(data.dateChange);
                //            Ext.getCmp('department').setValue(data.department);
                //            Ext.getCmp('userview').setValue(data.userName);
                Ext.getCmp('useredit').setRawValue(data.user);
                Ext.getCmp('statusCompl').setRawValue(data.statusCompl);
                Ext.getCmp('description').setValue(data.description);
                
                if (data.canAdmin) {
                    Ext.getCmp('useredit').show();
                    //                Ext.getCmp('userview').hide();
                    Ext.getCmp('removeButton').show();
                    Ext.getCmp('copyComp').show();             
                } else {
                    Ext.getCmp('useredit').hide();
                    //                Ext.getCmp('userview').show();
                    Ext.getCmp('removeButton').hide();
                    Ext.getCmp('copyComp').hide();
                }
                
                if (data.canEdit) {
                    Ext.getCmp('save').hide();
                }
            }

    проект на Ext.js4
    Много Ext.getCmp('');
    Весь код - сплошная каша, динамическая загрузка Ext.js4 не используется, прикручено require.js,
    ни одного класса.

    maxt, 18 Февраля 2012

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

    +84

    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
    Query q;
          //для хэширования
    
          q =
             session.createQuery("select idElType from " + LinkToAtributeValue.class.getName()
             + " where idEl=" + idObject.toString());
    
          Vector<Integer> vec = new Vector<Integer>(q.list());
          int t;
          for(int i = 0; i < vec.size() - 1; i++) {
             t = vec.get(i);
             for(int j = i + 1; j < vec.size(); j++) {
                if(t == vec.get(j)) {
                   vec.remove(j);
                }
             }
          }

    "DISTINCT для трусов"
    или
    "Хорошего кода должно быть много"

    maxt, 08 Сентября 2011

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

    +70

    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
    //обработка поведения кнопки копировать
          if(tni.isTreeStruct()) {
             copyDocuments.setEnabled(false);
          }
          else {
             copyDocuments.setEnabled(true);
             if(jTree.getSelectionCount() > 1) {
                copyDocuments.setEnabled(false);
             }
             else {
                copyDocuments.setEnabled(true);
             }
          }
          //-----------------------------------------------------------
    
          //обработка поведения кнопки копировать c файлами
          if(tni.isTreeStruct()) {
             copyDocsWithFile.setEnabled(false);
          }
          else {
             copyDocsWithFile.setEnabled(true);
             if(jTree.getSelectionCount() > 1) {
                copyDocsWithFile.setEnabled(false);
             }
             else {
                copyDocsWithFile.setEnabled(true);
             }
          }
          //-----------------------------------------------------------

    Это - "Гребаный копипаст"

    maxt, 01 Июля 2011

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

    +84

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    if(obj instanceof TreeDictEntry) {
             TreeDictEntry tde = (TreeDictEntry) obj;
             Integer index = hm_IdDocs_idVectors.get(tde.getId());
             if(index == null) {
                Exception ex =
                   new Exception("Epic Faile в хэш мапе элементов не найден индекс в векторе");
                ex.printStackTrace();
             }
             vecElements.removeElementAt(index);
             return;
          }

    к этому я даже название придумал: "Эпичный говнокод"

    maxt, 08 Июня 2011

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

    +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
    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
    1.3 @@ -89,9 +89,6 @@
         1.4     public void openFile(File f) {
         1.5        try {
         1.6           Desktop.getDesktop().open(f);
         1.7 -//         String s = "cmd /c \"" + f.getAbsolutePath() + "\" ";
         1.8 -//         System.out.println(s);
         1.9 -//         java.lang.Process p = Runtime.getRuntime().exec(s);
        1.10        }
        1.11        catch (Exception e) {
        1.12           JOptionPane.showMessageDialog(null, "Ошибка при открытии файла", "Ошибка", JOptionPane.ERROR_MESSAGE);
        1.13 @@ -112,21 +109,6 @@
        1.14           temp.mkdir();
        1.15        }
        1.16        String fName = entry.getName();
        1.17 -      fName = fName.replaceAll("\\&", "_");
        1.18 -      fName = fName.replaceAll("\\(", "_");
        1.19 -      fName = fName.replaceAll("\\)", "_");
        1.20 -      fName = fName.replaceAll("\\[", "_");
        1.21 -      fName = fName.replaceAll("\\]", "_");
        1.22 -      fName = fName.replaceAll("\\{", "_");
        1.23 -      fName = fName.replaceAll("\\}", "_");
        1.24 -      fName = fName.replaceAll("\\^", "_");
        1.25 -      fName = fName.replaceAll("\\=", "_");
        1.26 -      fName = fName.replaceAll("\\!", "_");
        1.27 -      fName = fName.replaceAll("\\`", "_");
        1.28 -      fName = fName.replaceAll("\\+", "_");
        1.29 -      fName = fName.replaceAll("\\~", "_");
        1.30 -      fName = fName.replaceAll(" ", "_");
        1.31 -      fName = fName.replaceAll(("№"), "_");
        1.32        String dnFileName = temp.getName() + File.separator + fName;
        1.33  
        1.34        File dlFname = new File(dnFileName);

    Так как открытие файла происходило весьма оригинальным образом через cmd.exe - вполне логично было написано фильтрование символов, неперевариваемых этой cmd.exe.
    Как видно - задача решалась одной строкой - " Desktop.getDesktop().open(f);".

    maxt, 08 Июня 2011

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

    +74.4

    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
    private String getNullString(int kol){
            String nullstr="";
            for(int i=0;i<kol;i++){
                nullstr+="0";
            }
            return nullstr;
        }
    
    public String getNumber(int idd){
            String regid="";
            String idds=(idd+"");
            if(idds.length()<getNumberLenth()){
                regid+=getNullString(getNumberLenth()-idds.length());
                regid+=idds;
            }
            String regnum="";
            regnum+=getNumberPrefix()+regid;
            if(getNumberPeriod()!=NUM_PERIOD_NOPERIOD){
                GregorianCalendar c=new GregorianCalendar();
                switch(getNumberPeriod()){
                    case NUM_PERIOD_MONTH:
                        int m=c.get(Calendar.MONTH)+1;
                        regnum+="-"+m;
                        regnum+=".";
                        regnum+=c.get(Calendar.YEAR);
                        break;
                    case NUM_PERIOD_YEAR:
                        regnum+="-"+c.get(Calendar.YEAR);
                }
            }
            return regnum;
        }

    Ну незнал я про String.format (((..

    maxt, 01 Апреля 2010

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

    +70.8

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    if (tp != null) {
    		DocElement d = (DocElement)tp.getLastPathComponent();
    		if (d.getElementType() == Constants.TYPE_FOLDER) {
    			context.setCursor(DragSource.DefaultMoveDrop);
    		if (d.getElementType()==Constants.TYPE_FOLDER) {
    			tree.setSelectionPath(tp);
    			tree.expandPath(tp);
    		}
    		}else {
    			context.setCursor(DragSource.DefaultMoveNoDrop);
    		}
    	}

    моё, так сказать носом ткнули)
    условие if (d.getElementType()==Constants.TYPE_FOLD ER)

    maxt, 29 Марта 2010

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

    +76.8

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    public void isChangenabledExc(){
            if(isNew){
                return;
            }
            if(true)return;//TODO
            if(!isChangenabled()){
                throw new ChangeEIsNotLockedException();
            }
        }

    В этой функции находится единственный коментарий из всего файла.
    Видимо в планах эта функция должна была делать нечто хорошее, но
    планы так и не были осуществелны.
    В результате получилась функция, которая ничего не делает, а только проверяет какие то условия.
    поиском обнаружил 5 вызовов этой функции.

    maxt, 26 Марта 2010

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

    +87.2

    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
    else {
                                                 if(codeName == SummaryConstants.TYPE_COST) {
                                                    return new Cost(rtOBJ);
                                                 }
                                                 else {
                                                    if(codeName == SummaryConstants.TYPE_EQUIPMENT) {
                                                       return new Equipment(rtOBJ);
                                                    }
                                                 }
                                              }
                                           }
                                        }
                                     }
                                  }
                               }
                            }
                         }
                      }
                   }
                }
             }

    не стал утомлять еще и километром вложенных if ов..
    а говнокод в том, что здесь можно было использовать switch

    maxt, 25 Марта 2010

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

    +79.1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    if (at == null) {
    	try {
    	throw new NullPointerException();
    	}catch (Exception e) {
    	e.printStackTrace();
    	throw new NullPointerException();
    	}
    }

    проверка корректности полученного аргумента в функции.
    хорошо придумано, что бы не вызывать
    throw new IllegalArgumentException();

    maxt, 13 Марта 2010

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