0

Доброго времени суток, код будущего морского боя.

 var sea =     [[0,0,0,0,0],
                [0,0,0,0,0],
                [0,0,0,0,0],
                [0,0,0,0,0],
                [0,0,0,0,0]];
    // Оригинальный массив, хранящий положение кораблей. 

    var uniqSea = ['0,0','0,1','0,2','0,3','0,4',
                   '1,0','1,1','1,2','1,3','1,4',
                   '2,0','2,1','2,2','2,3','2,4',
                   '3,0','3,1','3,2','3,3','3,4',
                   '4,0','4,1','4,2','4,3','4,4'];
    // Массив уникальных координат, из которого я получаю координаты. 

    function setShip(shipSize) {
        var cord = getRandomSpot().split(','); // Получаю случайную координату из массива uniqSea. 
        var x = parseInt(cord[0]); // Беру значения x и y из неё. 
        var y = parseInt(cord[1]);
        var uniqFromSea = Math.round(parseInt(cord.join('')) + y)/2; // Определяю, чему равен индекс этой случайной координаты в массиве uniqSea 
        sea[x][y] = shipSize; // Ставлю корабль в массив sea.
        console.log('[X]: ' + x + ' [Y]: ' + y);
        console.log('COORD FROM uniqSea: ' + uniqFromSea);
        uniqSea.splice(uniqFromSea, 1); // Удаляю уникальную координату, чтобы не зарандомить её снова. 
    }

    function getRandomSpot() {
        return uniqSea[Math.floor((Math.random() * uniqSea.length))];
    }

    setShip(1);
    setShip(1);
    setShip(1);

    console.log(sea);
    console.log(uniqSea);

В комментариях написал, что происходит в коде. Помогите со следующей проблемой: Когда я удаляю уникальную координату в 23 строке, то splice, удаляет только значение, получается, что сдвигаются значения, но индексы остаются прежними. В результате, когда я захочу удалить другую уникальную координату, то удалю не её, а следующую в массиве. Например :

[X]: 0 [Y]: 2 // Зарандомило 0,2 и удалило 0,2
COORD FROM uniqSea: 2
[X]: 2 [Y]: 1
COORD FROM uniqSea: 11 //Зарандомило 2,1 и удалило 2,2, полный бред..
[ [ 0, 0, 1, 0, 0 ],
  [ 0, 0, 0, 0, 0 ],
  [ 0, 1, 0, 0, 0 ],
  [ 0, 0, 0, 0, 0 ],
  [ 0, 0, 0, 0, 0 ] ]
[ '0,0', '0,1',        '0,3','0,4',
  '1,0', '1,1','1,2', '1,3','1,4',
  '2,0', '2,1',        '2,3','2,4',
  '3,0', '3,1','3,2', '3,3','3,4',
  '4,0', '4,1','4,2', '4,3','4,4' ]
3
  • 1
    var uniqFromSea = uniqSea.indexOf(cord.join(',')); ?
    – br3t
    Commented 12 июн. 2017 в 11:51
  • А вообще - getRandomSpot может возвращать только индекс, в setShip извлекать ячецку по этому индексу и удалять по нему элемент.
    – br3t
    Commented 12 июн. 2017 в 11:52
  • Большое спасибо. Даже обидно стало, как все просто. Commented 12 июн. 2017 в 11:58

1 ответ 1

0

немного переделал массивы, чтоб не путаться. Ты, если что, потом вернешь назад

var sea =  [[0,0,0,0,0],
            [0,0,0,0,0],
            [0,0,0,0,0],
            [0,0,0,0,0],
            [0,0,0,0,0]];

var uniqSea = [[0,0],[0,1],[0,2],[0,3],[0,4],
               [1,0],[1,1],[1,2],[1,3],[1,4],
               [2,0],[2,1],[2,2],[2,3],[2,4],
               [3,0],[3,1],[3,2],[3,3],[3,4],
               [4,0],[4,1],[4,2],[4,3],[4,4]];

var newArr=[];
var Nar1;
function setShip(shipSize) {
    var cord = getRandomSpot(); // Получаю случайную координату из массива uniqSea. 
    var x = cord[0]; // Беру значения x и y из неё. 
    var y = cord[1];
    sea[x][y] = shipSize; // Ставлю корабль в массив sea.
    console.log('[X]: ' + x + ' [Y]: ' + y);
    newArr.push(cord); ///пушаю корд в новый массив
    Nar1 = uniqSea.filter(function( el ){
     return  newArr.indexOf(el) === -1; 
    }); //отсортированный массив выходной без заданных координат
 }      

function getRandomSpot() {
    return uniqSea[Math.floor((Math.random() * uniqSea.length))];
}

setShip(1);
setShip(1);
setShip(1);
console.log(sea);
console.log(Nar1);

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

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