Une façon plus compacte de faire cela?

Question

J'ai quelques fonctions qui entourent les cellules environnantes d'une cellule. La grille est contenue dans un tableau.

Dans mon code, j'ai des vérifications pour m'assurer que ce n'est pas l'une des cellules de bord, car la vérification d'une cellule indéfinie provoque une erreur.

En tant que tel, j'ai le code comme ceci:

if(x > 0) { 
    var firstX = x - 1; 
} else { 
    var firstX = x; 
} 
if(x < 199) { 
    var lastX = x + 1; 
} else { 
    var lastX = x; 
} 

if(y > 0) { 
    var firstY = y - 1; 
} else { 
    var firstY = y; 
} 
if(y < 199) { 
    var lastY = y + 1; 
} else { 
    var lastY = y; 
} 

Beaucoup de lignes de code pour faire très peu. Y a-t-il une manière plus élégante de faire ceci?

Answer 1

Vous pouvez utiliser l'opérateur conditionnel:

var firstX = x > 0 ? x - 1 : x; 
var lastX = x < 199 ? x + 1 : x; 
var firstY = y > 0 ? y - 1 : y; 
var lastY = y < 199 ? y + 1 : y; 

Vous pouvez supprimer la redondance en écrivant une fonction pour calculer « premier » donné une valeur et un similaire pour « dernier » - mais je pense ce serait exagéré dans ce cas.

Answer 2

Vous pouvez utiliser l'opérateur conditionnel:

var firstX = x - (x > 0 ? 1:0); 
var lastX = x + (x < 199 ? 1:0); 
var firstY = y - (y > 0 ? 1:0); 
var lastY = y + (y < 199 ? 1:0); 

Edit:
offert une autre façon de l'utiliser, comme Jon déjà posté code "mon". ;)

Edit 2:
Comme Rafael a fait remarquer, la condition peut être implicitement converti en un certain nombre, de sorte que l'opérateur conditionnelle n'est pas nécessaire:

var firstX = x - (x > 0); 
var lastX = x + (x < 199); 
var firstY = y - (y > 0); 
var lastY = y + (y < 199); 

Cependant, il est moins évident que ce code fait Est-ce que. De mes tests, il semble que Javascript utilise systématiquement la valeur 1 pour true, mais dans les langages de programmation, la valeur -1 est tout aussi communément utilisée.

Answer 3

Ou plus clairement:

var firstX = Math.max(x - 1, 0); 
var lastX = Math.min(x + 1, 199); 
var firstY = Math.max(y - 1, 0); 
var lastY = Math.min(y + 1, 199); 

Answer 4

Utilisez les variables que vous cocher (x et y) au lieu de premier/LASTx et Première/Lasty

if(x > 0 && x < 199) x-=1; 
else if(x > 0) x+=1; 

if(y > 0 && y < 199) y-=1; 
else if(y > 0) y +=1; 

Il suffit de cocher x et y après. :)