L2CCCP
9 уровень
Калининград

Говнокод #22

Пост из группы Архив info.javarush.ru
3786 участников
public boolean ignore(File f) { String[] dirs = f.getParent().replace("\\", "/").split("/"); return dirs[dirs.length - 1].equals("ignore"); }
Комментарии (4)
  • популярные
  • новые
  • старые
Для того, что бы оставить комментарий вы должны авторизоваться
saul22 уровень, Иркутск
13 декабря 2015, 18:07
public boolean ignoreFileName(File f, String fileNameToIgnore)
{
    Path filePath = Paths.get(f.getPath());
    Path ignore = Paths.get(fileNameToIgnore);
    return (filePath.getFileName().equals(ignore));
}
Victor3413 уровень
21 сентября 2015, 09:46
Я слишком мало знаю, что бы оценивать чей то код. Но, если кто-то делится «говнокодом», он должен написать почему он считает это таким и предложить правильное с его точки зрения написание кода.
Archartem33 уровень
29 сентября 2016, 13:04
Людына проверяет, называется ли его файл словом «ignore». Берёт полный путь к файлу (что верно), а потом своеручно расковыривает этот путь по фрагментам, отыскивает среди них последний… в то время как можно было воспользоваться функцией, которая делает это сама, и сэкономить код.
Выше и ниже — примеры приличного разрешения этой ситуации: выше — упрощённый механический, ниже — более «профессиональный», где подразумевается, что компьютер сам знает, какая часть пути есть имя файла, и допускает, что файл может называться не только «ignore», но и любым другим словом.
ravlion9 уровень
6 июля 2015, 15:02
public boolean ignore(File f)
{
    return f.getParent().endsWith("ignore");
}