Тестовое задание на позицию frontend-разработчик

Задача №1 — code refactoring

Первое, что бросается в глаза — имя функции и входные параметры

function func(s, a, b)

имя должно отображать суть происходящего: в данном примере функция ищет в строке совпадения по входящим символам и возвращает максимальный индекс совпадений.

далее внутри функции первым действием происходит проверка на пустую строку

s.match(/^$/)

для этого можно использовать простую проверку

s === '' || s == null

далее стоит уйти от while со сложным условием остановки

while ((aIndex == -1) && (bIndex == -1) && (i > 0))

в пользу цикла для массива символов входящей строки

код функции - оригинал

function func(s, a, b) {

	if (s.match(/^$/)) {
		return -1;
	}

	var i = s.length -1;
	var aIndex =     -1;
	var bIndex =     -1;

	while ((aIndex == -1) && (bIndex == -1) && (i > 0)) {
	    if (s.substring(i, i +1) == a) {
	    	aIndex = i;
    	}
	    if (s.substring(i, i +1) == b) {
	    	bIndex = i;
    	}
	    i = i - 1;
	}

	if (aIndex != -1) {
	    if (bIndex == -1) {
	        return aIndex;
	    }
	    else {
	        return Math.max(aIndex, bIndex);
	    }
	}

	if (bIndex != -1) {
	    return bIndex;
	}
	else {
	    return -1;
	}
}

код функции - рефакторинг

function getSignIndex(str, firstSign, secondSign) {
	if (str === '' || str == null) {
		return -1
	}

  let firstIndex = -1;
  let secondIndex = -1;
  const arStr = [...str].reverse();
  const length = arStr.length - 1;

  arStr.every((element, index) => {
    if (element === firstSign) {
      firstIndex = length - index;
    }
    if (element === secondSign) {
      secondIndex = length - index;
    }
    if (firstIndex >= 0 || secondIndex >= 0 || index === length - 1) {
      return false;
    } else {
      return true;
    }
  })
  return firstIndex < 0 ? secondIndex : firstIndex;
}

💪 С тестами сильно помог

Mochajs

сами тесты в ./test/test.js

./node_modules/mocha/bin/mocha

---

Задача №2 — практическое задание

💰 Терминал оплаты услуг сотовой связи

🔨 Установка проекта

# install dependencies
yarn

# serve with hot reload at localhost:8080
yarn run dev

# build for production with minification
yarn run build

Ссылка на макет (if not open use vpn)

https://vue-test-job.netlify.com

💪 В создании проекта принимали участие

VueJS | Vuex | Vue-router | Vuetifyjs | Vue-cli | Vue-toasted | Webpack | es6