Как уменьшить такой код?..

[combo]

function goToWay(way:String) { 
	var step:Number = 5;
	
	switch (way) {
		case "right":
			mc1._x -= step;
			break;
		case "left":
			mc1._x += step;
			break;
		case "up":
			mc1._y += step;
			break;
		case "down":
			mc1._y -= step;
			break;
	}
};

arrowRight_btn.onRelease = function() {
	goToWay("right");
};
arrowLeft_btn.onRelease = function() {
	goToWay("left");
};
arrowUp_btn.onRelease = function() {
	goToWay("up");
};
arrowDown_btn.onRelease = function() {
	goToWay("down");
};

Собст-но сабж. Думаю, можно написать как-то короче, но не знаю как. Кнопок в принципе может быть больше и не охота из-за этого сильно "раздувать" флешку.
Как оптимизировать?

[Aziz Zaynutdinoff]

function goToWay(way:String) {
	var step:Number = 5;
	var s:Number = (way == "left" && way != "up" && way != "down") ? -1 : 1;
	var d:Number = (way == "up" && way != "left" && way != "right") ? -1 : 1;
	var __x:Number = (way == "right" || way == "left") ? step*s : 0;
	var __y:Number = (way == "up" || way == "down") ? step*d : 0;
	mc1._x += __x;
	mc1._y += __y;
}

[combo]

Aziz Zaynutdinoff, спасибо, но по-моему так еще запутанее чем у меня.

И еще вопрос по оптимизации: как быть с баттонами? То есть можно ли, скажем, сделать какую-нибудь универсальную для этих кнопок функцию, чтобы при этом аргументы в goToWay они передавали разные ?

[iNils]

function goToWay (x:Number, y:Number) {
	var step:Number = 5;
	mc1._x += step * x;
	mc1._y += step * y;
}
arrowRight_btn.onRelease = function () {
	goToWay (1, 0);
};
arrowLeft_btn.onRelease = function () {
	goToWay (-1, 0);
};
arrowUp_btn.onRelease = function () {
	goToWay (0, -1);
};
arrowDown_btn.onRelease = function () {
	goToWay (0, 1);
};

[Aziz Zaynutdinoff]

знаешь на самом деле я бы просто передавал два параметра

function goToWay(__x:Number, __y:Number):Void
{
     __x = (__x == undefined) ? 0 : __x;
     __y = (__y == undefined) ? 0 : __y;
     _mc._x += __x;
     _mc._y += __y;
};
btn.onRelease = function()
{
     goToWay(-5); //влево
}
btn1.onRelease = function()
{
     goToWay(-5, 5); //вниз-влево
}

UPD:
iNils опередил хотя есть немного разница

[Kikasso]

var step = 5;
var wayObj = {
left:{dx: step, dy:0},
right:{dx:-step, dy:0},
down:{dy: step, dx:0},
up:{dy: -step, dx:0}
}
function goToWay(way:String) { 
var ob = wayObj[way];
 mc1._x += ob.dx;
 mc1._y += ob.dy;
};

arrowRight_btn.onRelease = function() {
	goToWay("right");
};
arrowLeft_btn.onRelease = function() {
	goToWay("left");
};
arrowUp_btn.onRelease = function() {
	goToWay("up");
};
arrowDown_btn.onRelease = function() {
	goToWay("down");
};

Длиннее, но гибче.

[aksios]

var step:Number = 5;
var n:Object = {r:{x:1,y:0},l:{x:-1,y:0},t:{x:0,y:-1},d:{x:0,y:1}};
function goToWay(way) { 
mc1._x+= step*n[way].x;
mc1._y+= step*n[way].y;
}
arrowRight_btn.onRelease = function()
{
	goToWay("r");
};
arrowLeft_btn.onRelease = function() {
	goToWay("l");
};
arrowUp_btn.onRelease = function() {
	goToWay("t");
};
arrowDown_btn.onRelease = function() {
	goToWay("d");
};

[Kikasso]

var step = 5;
var wayObj = {
left:{dx: step},
right:{dx:-step},
down:{dy: step},
up:{dy: -step},
down20px:{dy: 20},
downRight20px:{dy: step, dx:-20}
}
function goToWay(way:String) { 
var ob = wayObj[way];
 if( ob.dx ) mc1._x += ob.dx;
 if( ob.dy ) mc1._y += ob.dy;
};
function btnOnRelease() {
if( this.dir == undefined ) this.dir = (this._name.split("_"))[0];
gotoWay(this.dir);
};

right_btn.onRelease = left_btn.onRelease = up_btn.onRelease = btnOnRelease;
down_btn.onRelease = down20px_btn.onRelease = downRight20px_btn.onRelease = btnOnRelease;

[this.create]

Сообщение от combo

function goToWay(way:String) { 
    var step:Number = 5;
    
    switch (way) {
        case "right":
            mc1._x -= step;
            break;
        case "left":
            mc1._x += step;
            break;
        case "up":
            mc1._y += step;
            break;
        case "down":
            mc1._y -= step;
            break;
    }
};

arrowRight_btn.onRelease = function() {
    goToWay("right");
};
arrowLeft_btn.onRelease = function() {
    goToWay("left");
};
arrowUp_btn.onRelease = function() {
    goToWay("up");
};
arrowDown_btn.onRelease = function() {
    goToWay("down");
};

Собст-но сабж. Думаю, можно написать как-то короче, но не знаю как. Кнопок в принципе может быть больше и не охота из-за этого сильно "раздувать" флешку.
Как оптимизировать?

Вот короче:

var step:Number = 5;
arrowRight_btn.onRelease = function() {
    mc1._x -= step;
};
arrowLeft_btn.onRelease = function() {
    mc1._x += step;
};
arrowUp_btn.onRelease = function() {
    mc1._y += step;
};
arrowDown_btn.onRelease = function() {
    mc1._y -= step;
};

Про много кнопок: как вы их планируете вставлять в сцену. От этого будет зависить и оптимизация.
ps. А может вам просто класс Key нужен?

[combo]

Сообщение от this.create

Вот короче:

var step:Number = 5;
arrowRight_btn.onRelease = function() {
    mc1._x -= step;
};
arrowLeft_btn.onRelease = function() {
    mc1._x += step;
};
arrowUp_btn.onRelease = function() {
    mc1._y += step;
};
arrowDown_btn.onRelease = function() {
    mc1._y -= step;
};

Нет, такой вариант как раз не очень гибкий - а если еще нужно ходить по диагонали, то придётся опять дублировать код?

Про много кнопок: как вы их планируете вставлять в сцену. От этого будет зависить и оптимизация.
ps. А может вам просто класс Key нужен?

я не аттачу их из библиотеки, а вставляю сразу в редакторе. Ну, вот сейчас, например, у меня девять кнопок(на все возможные ходы) и для всех почти одинаковый код, отличающийся только передаваемым значением (way). Поэтому интересует, как можно упростить код, и не писать для каждой кнопки свой onRelease.