User hubert
hubert
41 уровень

Говнокод #21

Статья из группы Архив info.javarush.ru
return Long.parseLong(content.substring(content.lastIndexOf('/' + 1)));
Комментарии (3)
ЧТОБЫ ПОСМОТРЕТЬ ВСЕ КОММЕНТАРИИ ИЛИ ОСТАВИТЬ КОММЕНТАРИЙ,
ПЕРЕЙДИТЕ В ПОЛНУЮ ВЕРСИЮ
cyrillkurochkin Уровень 27, Киев, Украина
24 декабря 2014
hubert ресурс то у вас обучающий, неплохо бы и объяснить начинающим в чем тут ошибки и почему так нельзя (я про всю рубрику Говнокод, выбрал ластпост).
Тем не мнении за рубрику спасибо, поржал, поучился.
Izhak Уровень 22, Москва, Россия
27 мая 2014
Здесь, мне кажется, есть три проблемы
1. '/'+1 = '0'
либо имелась в виду позиция '/' плюс 1. Но тогда скобка стоит не там. И могут быть исключения, если не будет вообще искомого символа .contains
2. Нет проверки оставшейся в content подстроки на длину числа
3. Нет проверки вырезаемой подстроки на запрещённые символы