JavaScript переміщує елемент масиву вперед

Я хочу перевірити, чи містить масив "role". Якщо це станеться, я хочу перенести "role"на передню частину масиву.

var data= ["email","role","type","name"];
if ("role" in data) data.remove(data.indexOf("role")); data.unshift("role")
data;

Ось я отримав результат:

["role", "email", "role", "type", "name"]

Як я можу це виправити?

Ви можете сортувати масив і вказати, що значення "role"стоїть перед усіма іншими значеннями, а всі інші значення рівні:

var first = "role";
data.sort(function(x,y){ return x == first ? -1 : y == first ? 1 : 0; });

Демо: http://jsfiddle.net/Guffa/7ST24/

На мою думку, найчистіше рішення в ES6:

let data = ["email","role","type","name"];
data = data.filter(item => item !== "role");
data.unshift("role");

Моя перша думка була б такою:

var data= ["email","role","type","name"];

// if it's not there, or is already the first element (of index 0)
// then there's no point going further:
if (data.indexOf('role') > 0) {
    // find the current index of 'role':
    var index = data.indexOf('role');
    // using splice to remove elements from the array, starting at
    // the identified index, and affecting 1 element(s):
    data.splice(index,1);
    // putting the 'role' string back in the array:
    data.unshift('role');
}

console.log(data);

Щоб переглянути та трохи привести в порядок:

if (data.indexOf('role') > 0) {
    data.splice(data.indexOf('role'), 1);
    data.unshift('role');
}

Список літератури:

• Array.indexOf().
• Array.prototype.splice().
• Array.unshift().

let data = [0, 1, 2, 3, 4, 5];
let index = 3;
data.unshift(data.splice(index, 1)[0]);
// data = [3, 0, 1, 2, 4, 5]

Якщо ви не хочете змінювати існуючий масив, ви можете використовувати ES6 деструктуризацію з методом фільтра, щоб створити нову копію, зберігаючи порядок інших елементів.

const data = ["email", "role", "type", "name"];
const newData = ['role', ...data.filter(item => item !== 'role')];

Ось незмінне рішення, якщо потрібно:

      const newData = [
          data.find(item => item === 'role'),
          ...data.filter(item => item !== 'role'),
        ],

Якщо у вас є масив об’єктів, ви можете зрушити індекс початку за допомогою сплайсингу та натискання. Зрощування замінює вихідний масив частиною масиву, починаючи з потрібного індексу, і повертає ту частину, яку він видаляє (матеріали перед індексом), яку ви натискаєте.

let friends = [{
    id: 1,
    name: "Sam",
  },
  {
    id: 2,
    name: "Steven",
  },
  {
    id: 3,
    name: "Tom",
  },
  {
    id: 4,
    name: "Nora",
  },
  {
    id: 5,
    name: "Jessy",
  }
];

const tomsIndex = friends.findIndex(friend => friend.name == 'Tom');
friends.push(...friends.splice(0, tomsIndex));

console.log(friends);

Щоб перевірити, чи існує елемент у масиві, який слід використовувати .includes()замість in(як уже зазначалося тут,in це стосується властивостей об’єктів).

Ця функція робить те, що ви шукаєте: (видаляє предмет з позиції, в якій він знаходиться, і читає спереду)

   data = ["email","role","type","name"];
   moveToFirst("role", data)
    
   function moveToFirst(s, r){
      if (r.includes(s)){
        r.splice(r.indexOf(s),1);
        r.unshift(s);
      } 
    }
    
    console.log(data)

Я б погодився з цим рішенням ES6. Він не мутує вихідний масив (враховуючи, що він не вкладений), не проходить через масив (фільтр), і ви не просто обмежуєтесь 0-м індексом для переміщення елемента масиву.

const moveArrayItem = (array, fromIndex, toIndex) => {
    const arr = [...array];
    arr.splice(toIndex, 0, ...arr.splice(fromIndex, 1));
    return arr;
}


const arr = ["a", "b", "c", "d", "e", "f", "g"];
console.log(moveArrayItem(arr, 4, 0))
// [ 'e', 'a', 'b', 'c', 'd', 'f', 'g' ]

1. inоператор про властивості, а нема про предметах , в масивах. Див. Як перевірити, чи містить масив об’єкт у JavaScript? для чого використовувати ще.
2. Вам бракує фігурних дужок навколо двох (!) Операторів у вашому блоці if-block
3. Я не впевнений, чи ця .remove()функція, яку ви використовуєте, бере індекс елемента.

var data= ["email","role","type","name"];

data.splice(data.indexOf("role"), 1);
data.unshift('role');

Подібно до відповіді @ Tandroid, але більш загальне рішення:

const putItemsFirst = ({ findFunction, array }) => [
    ...array.filter(findFunction), 
    ...array.filter(signer => !findFunction(signer)),
]; 

Можна використовувати так

putItemsFirst({ 
    array: ["email","role","type","name"], 
    findFunction: item => item === 'role',
})

Щось подібне на це я використовував,

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

var data = ["email", "role", "type", "name"];

data.sort((a, b) => (b === 'role') - (a === 'role'));

console.log(data);

Використання lodash _.sortBy . Якщо товар є role, його буде відсортовано першим, інакше другим. Це теж добре працює, якщо немаєrole

var data = ["email", "role", "type", "name"];

var sorted = _.sortBy(data, function(item) {
  return item === 'role' ? 0 : 1;
});

console.log(sorted);

<script src="https://cdnjs.cloudflare.com/ajax/libs/lodash.js/4.17.4/lodash.min.js"></script>

Просто хотів залишити це тут, оскільки, згідно з іншими коментарями, відповідь Гуффи, схоже, набирає популярності, остаточний третій рівень - що було одним із негативних коментарів до цієї відповіді непотрібний. Також використання функцій стрілок робить його набагато чистішим.

Крім того, його легко розширити для обробки масивів об'єктів.

const first = "role";
data.sort((x, y) => first === x ? -1 : first === y)

Я вважаю, що це також повинно впоратись з турботою щодо решти масиву, на який впливає. Коли функція сортування повертає число менше 0 (перший === x), елемент рухатиметься до початку масиву, коли поверне 0 (перший! == y), руху не буде, і коли число більше за 0 (спочатку === y), x рухатиметься до кінця масиву, все відносно x та y. Отже, коли ні x, ні y не еквівалентні бажаному першому елементу (або це ідентифікатор у випадку сортування об’єктів), не буде переміщення двох по відношенню один до одного.

Для об’єкта:

const unsorted = [{'id': 'test'}, {'id': 'something'}, {'id': 'else'}];
const first = 'something';
const sorted = unsorted.sort((x,y) => x['id'] === first ? -1 : y['id'] === first);

var i = -1;
while (i < data.length) {
    if (data[i] === "role") {
        data.splice(i, 1);
        break;
    }
    i++;
}
data.unshift("role");

indexOfмає лише обмежену підтримку браузера, не визнаний IE7-8. Тому я б не використовував його на вашому місці, навіть за рахунок стислості коду в кілька рядків. Ви також хочете поставити крапку з комою в кінці оператора "unshift". Перший аргумент splice () визначає індекс для початку видалення елементів, а другий аргумент - кількість аргументів для видалення.

var data= ["email","role","type","name"];
if ("role" in data) data.splice(data.indexOf("role"),1); data.unshift("role");
data;