• ,

Говнокод #22


	public boolean ignore(File f)
	{
		String[] dirs = f.getParent().replace("\\", "/").split("/");
		return dirs[dirs.length - 1].equals("ignore");
	}

4 комментария

ravlion
public boolean ignore(File f)
{
    return f.getParent().endsWith("ignore");
}
Victor34
Я слишком мало знаю, что бы оценивать чей то код. Но, если кто-то делится «говнокодом», он должен написать почему он считает это таким и предложить правильное с его точки зрения написание кода.
Archartem
  • Archartem
  • 0
  • Комментарий отредактирован 2016-09-29 13:04:59 пользователем Archartem
Людына проверяет, называется ли его файл словом «ignore». Берёт полный путь к файлу (что верно), а потом своеручно расковыривает этот путь по фрагментам, отыскивает среди них последний… в то время как можно было воспользоваться функцией, которая делает это сама, и сэкономить код.
Выше и ниже — примеры приличного разрешения этой ситуации: выше — упрощённый механический, ниже — более «профессиональный», где подразумевается, что компьютер сам знает, какая часть пути есть имя файла, и допускает, что файл может называться не только «ignore», но и любым другим словом.
saul
  • saul
  • 0
public boolean ignoreFileName(File f, String fileNameToIgnore)
{
    Path filePath = Paths.get(f.getPath());
    Path ignore = Paths.get(fileNameToIgnore);
    return (filePath.getFileName().equals(ignore));
}
Только зарегистрированные и авторизованные пользователи могут оставлять комментарии.