Как вызвать метод?

Question

[synapse_people]

Добрый день!
Есть такой интерфейс:

public static interface Invokable {
	public void invoke();
};

и объект, который его реализует.
Каким образом можно добится того, чтобы этот метод можно было вызвать только из определенного места, а в других местах - нельзя(например, вылетало бы исключение)...?
Объясню зачем это нужно:
Есть модель, реализующая этот метод. Есть объект - сервер, который его будет вызывать.. Необходимо сделать так, чтобы пользователь не мог сам вызвать метод, а сервер - мог.
То есть, чтобы нельзя было написать model.invoke() и метод бы выполнился..
Подходит любая реализация задуманного.
Схема работы такая:
-сервер получает запрос и собирает Context, в котором инфа на пользователя и о подключении.
-объект контекста должен обязательно быть передан модели
-сервер вызывает invoke на определенной модели
Вопрос в том, как сделать так, чтобы пользователь не смог сам вызывать invoke!
Я думал сделать следующее:
класс ContextHolder с методами setCtx, getCtx.. Далее сервер вызывает Holder::setCtx, Model::invoke, а модель получает контект getCtx.... НО! Ведь пользователь тоже сможет вызвать setCtx.......... и все равно обойдет это.
*Объявлять метод как private, а потом при помощи рефлексии менять уровень доступа-не предлагать))

Comments

[Riĥard Brugekĥaim]

А не проще ли заблокировать юзверю доступ к самому объекту?

[synapse_people]

Riĥard Brugekĥaim, нельзя, на модели могут быть другие методы, которые нужно вызывать..

[Сергей Горностаев]

У вас пользователи код пишут?

[synapse_people]

Сергей Горностаев, da

[Igor Makarov]

Заблокировать доступ к объекту можно разными способами. В данном случае можно вложить объект Invokable, который будет знать о своем владельце, в закрытое свойство модели.

[synapse_people]

Igor Makarov, модели создаются через new, конструктор no-arg

[Igor Makarov]

synapse_people, будет очень не очень, если на входе в invoke() получать стэк и проверять от кого пришел вызов)

[Денис Загаевский]

Igor Makarov, За такое надо убивать.

Answer 1

[Денис Загаевский]

Если invoke() обязан быть интерфейсным методом, то видится такое решение:
1) Интерфейс Invokable должен лежать в том же пакете, откуда он будет вызываться
2) В этом же пакете нужно создать некий package-private класс (проще всего - enum) с одним-единственным инстансом:

enum Lock { 
    LOCK;
 }

3) в метод invoke передавать Lock:

public static interface Invokable {
  public void invoke(@NonNull Lock lock);
};

4) В реализации дополнительно проверять переданный Lock на null
5) В месте "правильного" вызова передавать: model.invoke(Lock.LOCK);

Если метод invoke не обязан быть интерфейсным, то просто сделать его package-private и держать реализацию в том же пакете.

Но мне кажется, что где-то ошибка проектирования. Если изложишь задачу более подробно, можно найти более нормальное решение.

Comments

[synapse_people]

Ошибки в проектировании нет, просто не хочется, чтобы кто-то посторонний, кроме сервера, мог вызывать событие.
Попробую сделать как ты сказал, отпишусь.

[Денис Загаевский]

synapse_people, я, конечно рад, что ты в этом так уверен, но раз такая проблема возникла - то ошибка, с большой вероятностью, есть.

[synapse_people]

в общем я сделал абстрактную модель в том же пакете, что и сервер реализован
В а.модели добавил свойство package-private, которое реализует слушатель
Получается у сервера есть к нему доступ и у модели, а у других - нет
Я сначала спроектировал интерфейсами API, а теперь вот на реализации застрял... Поэтому с архитектурой 100%- все гладко и отвечает требованиям
Спасибо

[synapse_people]

Денис Загаевский, Ну как бы ты сделал:
Модели инстансируются через new, привязки к конкретному серверу быть не должно. Модели добавляются к серверу(на самом деле - нет, но упростим).
Есть 1 объект Server - он получает события нижнего уровня(из юникс/инет сокета), потом передает их моделям, реализующим интерфейс Invokable, которые добавлены к серверу.
*Это менее 5% от того, что на самом делеле там происходит
Задача сделать так, чтобы сервер мог передать событие в модель,т.е. вызвать метод Model::invoke(), а другие-нет
НО Не обязательно иметь интерфейс, т.к. далее разбирается пришедшая команда, затем рефлексией получается нужный protected/private метод на этой же модели и данный метод вызывается..
Все

[synapse_people]

У меня еще вопрос, можно ли рефлексией получить свойсво package-private и заменить там значение
?)
Особенно, если оно final
?)

[Денис Загаевский]

synapse_people, значит ты когда проектировал, не учёл, что метод не должен быть доступен. С абстрактным классом ок, имхо.
Зачем там рефлексия не понял. Грязно это, я бы делал без неё.
Получить свойство можно. Изменить final - не знаю, надо смотреть.

[synapse_people]

Денис Загаевский, Не складывается! Пользователь может в отдельом jar-е создать такой-же пакет, как и модель и получить доступ к методу

[Денис Загаевский]

synapse_people, чувак, если этот метод есть, то так или иначе вызвать его можно. Если надо "защиту", то размотай стек-трейс. Но и это можно обойти - просто занопать в байт-коде.

[synapse_people]

Денис Загаевский, ну не на столько же) просто нужно сделать прозрачную передачу событий с одного объекта на другой так, чтобы никто(другие программеры) не смог всунуть туда свое событие(объект)

[synapse_people]

Денис Загаевский, приблизительно так, как работает тред локал, но в него все равно можно будет положить значение *из вне*

[Денис Загаевский]

synapse_people, остановись. Если они настолько захотят это сделать, что буду делать такие же пакеты, то они не остановятся и перед патчингом байт-кода. Джава предоставляет инструменты для защиты от дурака и проектирования. И не предоставляет - для защиты от злонамеренного использования. Всё что ты можешь сделать - это выполнять этот код на удалённой машине, которая только под твоим контролем.

[synapse_people]

Денис Загаевский, в общем я вообще решил выбросить класс модели и интерфейс модели
Будет пользователь писать абстрактную модель, аля:

public abstract class TestModel {
@ Invokable
public abstract void sayHi(String helloWorld);
@ RemoteInvokable
protected void recvStr(@User SpaceUser sender, String str) {
//...
}
}

Затем класс модели добавляется в сервер, где создается прокси для него,т.е. реализация
И Все
Больше нет public void ivnoke