5

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

AxisX = ((currentTouchPos.x - startTouchPos.x) / workZoneInPixX);
AxisY = ((currentTouchPos.y - startTouchPos.y) / workZoneInPixY);

Отличаются только оси и чуть чуть название переменных. Как передать это в метод, чтобы обрабатывать все похожие запросы этим? Чтобы сократить и избежать дублирования кода Вижу это как то примерно вот так

float FormatXY(x_or_y)
Axis = ((currentTouchPos.x_or_y - startTouchPos.x_or_y) / workZoneInPixX_or_Y);
return Axis;

Но понятно, что таким образом я просто передам значения x или y, а мне бы передать именно параметр x компонент вектора или y компонент вектора, и как то подставить это в методе.

Чтобы в результате, код

AxisX = FormatXY(X)

Был эквивалентен коду

AxisX = ((currentTouchPos.x - startTouchPos.x) / workZoneInPixX);

А код

AxisX = FormatXY(Y)

Был эквивалентен коду

AxisY = ((currentTouchPos.y - startTouchPos.y) / workZoneInPixY);
3
  • 1
    Можно, если задействовать рефлексию, но я не думаю что вы захотите увеличить время выполнения такой простой операции в разы, если не порядки.
    – rdorn
    26 окт 2016 в 8:09
  • 1
    @rdorn да зачем рефлексия? сергей вон предложил норм вариант. Остаётся только на вход функции дать char и сделать в нем два if в зависимости от char и всё. самый простой вариант. По сути Dmitrii сам всё себе расписал) просто чуть поправить его надо было) 26 окт 2016 в 9:00
  • 1
    дак я не против предложенного варианта =) но и так как вы хотели изначально тоже можно сделать, но не стоит из-за накладных расходов, разве что эксперимент с целью повышения образованности =)
    – rdorn
    26 окт 2016 в 9:04

2 ответа 2

2

На беглый взгляд серебренной пули тут не найдешь. Думаю лучше написать две функции, типа CurrentTouchX() и CurrentTouchY() - куда запихать соответствующие выражения. Код сразу станет почище. Не думаю, что стоит городить что-то большее, ну может кто и придумает решение поинтереснее.

0

Не судите строго первое что на ум пришло.

enum formatAx{
    "X",
    "Y"
}
float FormatXY(formatAx type){
    if (type == formatAx.x) {
        return = ((currentTouchPos.x - startTouchPos.x) / workZoneInPixX);
    }else {
        return = ((currentTouchPos.y - startTouchPos.y) / workZoneInPixY);
    }
}

Вызов будет таким:

AxisX = FormatXY(formatAx.X)

Мог ошибиться, сам не проверял

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.