JavaRush/Помощь по Java задачам/В чем же нарушена безопасность? Не понимаю...

41 уровень

28.07.2018
1429просмотров
12комментариев

В чем же нарушена безопасность? Не понимаю...

Вопрос по задаче

Решен

результат вывода: [com.javarush.task.task35.task3507.data.Sheep@567d299b, com.javarush.task.task35.task3507.data.Cat@198e2867] т.е. размер и проч. соответствует условию. Но: Исключение системы безопасности JavaRush. Вы выполняете потенциально опасную или запрещенную операцию. Это что же такое криминальное я делаю?

Реализуй логику метода getAllAnimals.
Аргумент метода pathToAnimals - это абсолютный путь к директории, в которой хранятся скомпилированные классы.
Путь не обязательно содержит / в конце.
НЕ все классы наследуются от интерфейса Animal.
НЕ все классы имеют публичный конструктор без параметров.
Только для классов, которые наследуются от Animal и имеют публичный конструктор без параметров, - создать по одному объекту.
Добавить созданные объекты в результирующий сет и вернуть.
Метод main не участвует в тестировании.

Требования:

Размер множества возвращаемого методом getAllAnimals должен быть равен количеству классов поддерживающих интерфейс Animal и имеющих публичный конструктор без параметров (среди классов расположенных в директории переданной в качестве параметра).
В множестве возвращаемом методом getAllAnimals должны присутствовать все классы поддерживающие интерфейс Animal и имеющие публичный конструктор без параметров (среди классов расположенных в директории переданной в качестве параметра).
В множестве возвращаемом методом getAllAnimals НЕ должен присутствовать ни один класс не поддерживающий интерфейс Animal или не имеющий публичного конструктора без параметров (среди классов расположенных в директории переданной в качестве параметра).
Метод getAllAnimals должен быть статическим.

package com.javarush.task.task35.task3507; public interface Animal { }

Комментарии (12)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Roman Pryshchepa Java Developer

Expert

9 августа 2018, 16:01

Исключение системы безопасности JavaRush. Вы выполняете потенциально опасную или запрещенную операцию. возникает из-за запроса всех пропертей getProperties()

String sep = System.getProperties().getProperty("file.separator");

нужно брать только ту, которая нужна:

String sep = System.getProperty("file.separator");

Slana

Уровень 41

9 августа 2018, 23:13

Спасибо. От исключения системы безопасности избавилась. Но валидация не проходит. Хотя вывод программы соответствует заданному условию - привожу в вопросе, копировала из idea.

Roman Pryshchepa Java Developer

Expert

10 августа 2018, 08:02

Добавь в класс Elephant implements Animal и посмотри на вывод. Подсказка: Только для классов, которые наследуются от Animal и имеют публичный конструктор без параметров, - создать по одному объекту.

Slana

Уровень 41

10 августа 2018, 09:06

Спасибо!!! А вот это уже от моей невнимательности - сменила getDeclaredConstructors на getConstructors и все прошло.

Евгений ВласовFull Stack Developer в NAMI

7 августа 2018, 13:50

Проблема у вас, скорее всего, в проверке конструкторов на доступность. http://qaru.site/questions/992591/java-reflection-isaccessible-method Просто оберните вызов конструктора и set.add в блок try catch, при вызове приватного конструктора будет генериться исключение. Не сработает вызов, ну и хорошо, пропускаем.

Slana

Уровень 41

8 августа 2018, 08:55

Так обратите внимание - у меня как раз это сделано, блок try catch с 49 строки по 79, включает в себя и вызов конструктора и set.add. У меня на ПК программа отрабатывается полностью, все объекты в set добавляются, исключение InvocationTargetException выбрасывается в момент проверки завершения цикла по files( строка 50). Оно ловится, программа аккуратно завершается. Поэтому и не могу понять, откуда пртензии валидации по безопасности?

Slana

Уровень 41

8 августа 2018, 10:56

Вот сегодня делаю задачу 3606, так в ней аналогичная проблема на валидации, хотя в ней даже и никаких исключений не бросается. А задача вновь на loader и вновь на ПК все прекрасно отрабатывает.

Slana

Уровень 41

28 июля 2018, 11:02

Скорее, как раз для крутых..., а я, как "чайник", не особо умею. Я вот понимаю, что все "танцы с бубнами" из-за private конструктора в Sheep, и моя строка c.setAccessible(true) не являетя безопасной. Однако вариантов обойти другим способом private модификатор у конструктора не могу придумать.

Сергеев Виктор

Уровень 40

Master

28 июля 2018, 10:23

форматирование кода не для крутых пацанов, верно?

Slana

Уровень 41

1 августа 2018, 10:52

А кроме замечания по поводу форматирования кода? Я так и не могу понять, в чем же засада. Код отрабатывает полностью, добавляет Sheep в множество и уже после добавления выбрасывает InvocationTargetException при проверке окончания массива files.

Сергеев Виктор

Уровень 40

Master

1 августа 2018, 19:59

не отформатированный код очень сложно читать. А искать в нем ошибку ставится ещё сложнее, почти не возможно, потому что надо держать в память, что отступ не имеет отношения к вложенности, а открытая скобка закрыта на той же строке Например 27 строка. Никто не ожидает, что if будет закрыт на этой же строке, но такое бывает. Ок. Но все что идет ниже, у вас идет с отступом, как будто оно вложено в if. И если не заметить, что на 27 строке if закрывается, читаешь уже не о том. строки 58-61 смещены закрывающие скобки и вместо чтения надо считать сколько было открыто и какая к чему относится.

Slana

Уровень 41

1 августа 2018, 20:57

Прикрепила отформатированный, с добавлением пояснений. И, к слову, форматировать-то на этих курсах и не учат. А я - атавизм, программист старых языков, "до сотворения мира..." Самое интересное - ошибка бросается именно на последнем шаге, когда проход по массиву классов уже закончен, оба требуемых экземпляра получены и добавлены в set