hubert
41 уровень

Говнокод #21

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