Говнокод #6889 — Java — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

Java / Говнокод #6889

+75

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) RSS

Lure Of Chaos 08.06.2011 11:40 # +2
только вот Desktop появился лишь в 6ой версии, так что exec был оправдан.
код, видимо представляет собой Diff

по оставшемуся коду могу предложить такой еще патч:
```
1.32 -       String dnFileName = temp.getName() + File.separator + fName;
    1.33 - 
    1.34 -       File dlFname = new File(dnFileName);
    1.35 +       File dlFname = new File(temp, fName);
```
Ответить
- gegMOPO4 08.06.2011 13:53 # +1
  
  И даже для старых жав следовало использовать exec, принимающий массив, а не строку.
  Ответить
  - Lure Of Chaos 08.06.2011 18:28 # 0
    
    да, тогда можно было обойтись без кавычек
    Ответить
    - gegMOPO4 08.06.2011 20:00 # 0
      
      И, подозреваю, без немалой части замен.
      Ответить
guest8 08.04.2019 20:54 # −999

показать все, что скрытоvanished
Ответить
guest8 09.04.2019 10:54 # −999

показать все, что скрытоvanished
Ответить

Говнокод: по колено в коде.

Java / Говнокод #6889

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

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