Как использовать рефлексию для вызова частного метода? - PullRequest
Новая
       31

Как использовать рефлексию для вызова частного метода?

298 голосов
/ 25 сентября 2008

В моем классе есть группа частных методов, и мне нужно вызывать один динамически на основе входного значения. И вызывающий код, и целевые методы находятся в одном экземпляре. Код выглядит так: 

MethodInfo dynMethod = this.GetType().GetMethod("Draw_" + itemType);
dynMethod.Invoke(this, new object[] { methodParams });

В этом случае GetMethod() не вернет приватные методы. Что BindingFlags мне нужно предоставить GetMethod(), чтобы он мог найти частные методы?

Ответы [ 11 ]

467 голосов
/ 25 сентября 2008

Просто измените свой код, чтобы использовать перегруженную версию GetMethod, которая принимает BindingFlags: 

MethodInfo dynMethod = this.GetType().GetMethod("Draw_" + itemType, 
    BindingFlags.NonPublic | BindingFlags.Instance);
dynMethod.Invoke(this, new object[] { methodParams });

Вот перечислимая документация BindingFlags .

65 голосов
/ 25 сентября 2008

BindingFlags.NonPublic не вернет никаких результатов. Как выясняется, объединение с BindingFlags.Instance делает свое дело. 

MethodInfo dynMethod = this.GetType().GetMethod("Draw_" + itemType, 
    BindingFlags.NonPublic | BindingFlags.Instance);
50 голосов
/ 21 августа 2014

И если вы действительно хотите попасть в неприятности, упростите выполнение, написав метод расширения: 

static class AccessExtensions
{
    public static object call(this object o, string methodName, params object[] args)
    {
        var mi = o.GetType ().GetMethod (methodName, System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance );
        if (mi != null) {
            return mi.Invoke (o, args);
        }
        return null;
    }
}

И использование: 

    class Counter
    {
        public int count { get; private set; }
        void incr(int value) { count += value; }
    }

    [Test]
    public void making_questionable_life_choices()
    {
        Counter c = new Counter ();
        c.call ("incr", 2);             // "incr" is private !
        c.call ("incr", 3);
        Assert.AreEqual (5, c.count);
    }
14 голосов
/ 08 января 2017

Microsoft недавно изменила API отражения , сделав большинство этих ответов устаревшими. Следующее должно работать на современных платформах (включая Xamarin.Forms и UWP): 

obj.GetType().GetTypeInfo().GetDeclaredMethod("MethodName").Invoke(obj, yourArgsHere);

Или как метод расширения: 

public static object InvokeMethod<T>(this T obj, string methodName, params object[] args)
{
    var type = typeof(T);
    var method = type.GetTypeInfo().GetDeclaredMethod(methodName);
    return method.Invoke(obj, args);
}

Примечание:

  • Если требуемый метод находится в суперклассе obj, для универсального T необходимо явно указать тип суперкласса.

  • Если метод асинхронный, вы можете использовать await (Task) obj.InvokeMethod(…).

11 голосов
/ 25 сентября 2008

Вы абсолютно уверены, что это невозможно сделать с помощью наследования? Отражение - это самое последнее, на что вы должны обращать внимание при решении проблемы, оно усложняет рефакторинг, понимание вашего кода и любой автоматический анализ.

Похоже, у вас должен быть класс DrawItem1, DrawItem2 и т. Д., Который переопределяет ваш dynMethod.

6 голосов
/ 20 апреля 2017

Отражение, особенно в отношении частных лиц, неверно

  • Отражение нарушает безопасность типов. Вы можете попытаться вызвать метод, который не существует (больше), или с неправильными параметрами, или с слишком большим количеством параметров, или недостаточно ... или даже в неправильном порядке (это мой любимый :)). Кстати, тип возвращаемого значения также может измениться.
  • Отражение медленное.

Разрывы отражения частных членов инкапсуляция принцип и, следовательно, предоставление вашему коду следующего:

  • Увеличьте сложность вашего кода, потому что он должен обрабатывать внутреннее поведение классов. То, что скрыто, должно оставаться скрытым.
  • Облегчает взлом вашего кода , так как он будет компилироваться, но не будет работать, если метод изменил свое имя.
  • Облегчает взлом приватного кода , потому что если он приватный, его не следует называть таким образом. Может быть, закрытый метод ожидает некоторое внутреннее состояние перед вызовом.

Что если я все равно должен это сделать?

Бывают случаи, когда вы зависите от третьей стороны или вам нужно, чтобы какой-то API не был выставлен, вам нужно подумать. Некоторые также используют его для тестирования некоторых классов, которыми они владеют, но они не хотят менять интерфейс, чтобы предоставить доступ к внутренним элементам только для тестов.

Если вы делаете это, делайте это правильно

  • Смягчить легко ломается:

Чтобы устранить проблему, которую легко сломать, лучше всего обнаружить любой потенциальный сбой путем тестирования в модульных тестах, которые будут выполняться в сборке с непрерывной интеграцией или тому подобном. Конечно, это означает, что вы всегда используете одну и ту же сборку (которая содержит закрытые элементы). Если вы используете динамическую нагрузку и отражение, вам нравится играть с огнем, но вы всегда можете поймать исключение, которое может вызвать вызов.

  • Смягчить медлительность отражения:

В последних версиях .Net Framework CreateDelegate в 50 раз превосходил вызов MethodInfo: 

// The following should be done once since this does some reflection
var method = this.GetType().GetMethod("Draw_" + itemType, 
  BindingFlags.NonPublic | BindingFlags.Instance);

// Here we create a Func that targets the instance of type which has the 
// Draw_ItemType method
var draw = (Func<TInput, Output[]>)_method.CreateDelegate(
                 typeof(Func<TInput, TOutput[]>), this);

draw звонки будут примерно в 50 раз быстрее, чем MethodInfo.Invoke используйте draw как стандарт Func вот так: 

var res = draw(methodParams);

Отметьте этот мой пост , чтобы увидеть эталонный тест по различным вызовам методов

2 голосов
/ 27 сентября 2008

Не могли бы вы просто иметь разные методы Draw для каждого типа, который вы хотите рисовать? Затем вызовите перегруженный метод Draw, передавая объект типа itemType для рисования.

Ваш вопрос не дает понять, действительно ли itemType относится к объектам разных типов.

1 голос
/ 08 февраля 2018

Вызывает любой метод, несмотря на его уровень защиты на экземпляре объекта. Наслаждайтесь! 

public static object InvokeMethod(object obj, string methodName, params object[] methodParams)
{
    var methodParamTypes = methodParams?.Select(p => p.GetType()).ToArray() ?? new Type[] { };
    var bindingFlags = BindingFlags.NonPublic | BindingFlags.Public | BindingFlags.Instance | BindingFlags.Static;
    MethodInfo method = null;
    var type = obj.GetType();
    while (method == null && type != null)
    {
        method = type.GetMethod(methodName, bindingFlags, Type.DefaultBinder, methodParamTypes, null);
        type = type.BaseType;
    }

    return method?.Invoke(obj, methodParams);
}
1 голос
/ 25 сентября 2008

Я думаю, вы можете передать его BindingFlags.NonPublic, где это - это метод GetMethod.

0 голосов
/ 23 мая 2018

Прочтите этот (дополнительный) ответ (который иногда является ответом), чтобы понять, куда это идет, и почему некоторые люди в этой теме жалуются, что "он все еще не работает"

Я написал точно такой же код, как один из ответов здесь . Но у меня все еще была проблема. Я установил точку останова на 

var mi = o.GetType().GetMethod(methodName, BindingFlags.NonPublic | BindingFlags.Instance );

Выполнено, но mi == null

И так продолжалось до тех пор, пока я не перестроил все вовлеченные проекты. Я тестировал одну сборку, пока метод отражения находился в третьей сборке. Это было очень странно, но я использовал Immediate Window для обнаружения методов и обнаружил, что закрытый метод, который я пытался выполнить модульным тестом, имел старое имя (я переименовал его). Это сказало мне, что старая сборка или PDB все еще существует, даже если проект модульного тестирования собирается - по какой-то причине проект, который он не тестировал "перестроить" сработало

Страница:
...