alexeykurbako/alco

alco's Issues

Code Review

Важные замечания

Занеси объявление констант в функцию getrandom() - это позволит быстро изменить диапазон рандома:

alco/alcoJS.js

Lines 35 to 36 in fcc0972

	const min=-10;
	const max= 10;

Также это упростит данную запись:

alco/alcoJS.js

Lines 37 to 38 in fcc0972

	const ch1 = getrandom(min,max);
	const ch2 = getrandom(min,max);

Реализуй отдельную функцию:

alco/alcoJS.js

Line 39 in fcc0972

const oper = Math.floor(Math.random()*4);

Объяви константу result и верни ей результат функции result(), имя которой нужно сделать более понятным:

alco/alcoJS.js

Line 41 in fcc0972

if(temp==result(ch1,ch2,oper)){

Лишнее отбрасывание дробной части:

alco/alcoJS.js

Lines 23 to 24 in fcc0972

	case 0: return rez = Math.floor(ch1+ch2); break;
	case 1: return rez = Math.floor(ch1-ch2); break;

Ненужная функция:

alco/alcoJS.js

Lines 14 to 20 in fcc0972

	function prov(ch2,oper){
	if(oper==1 && ch2<0){
	const sk=1;
	return -ch2;
	}
	return ch2;
	}

GRAMMAR NAZI

После служебного слова ставим пробел:

alco/alcoJS.js

Line 34 in fcc0972

while(true){

alco/alcoJS.js

Line 41 in fcc0972

if(temp==result(ch1,ch2,oper)){

Операторы окружаем пробелами:

alco/alcoJS.js

Lines 23 to 26 in fcc0972

	case 0: return rez = Math.floor(ch1+ch2); break;
	case 1: return rez = Math.floor(ch1-ch2); break;
	case 2: return rez = ch1*ch2; break;
	case 3: return rez = Math.floor((ch1/ch2)*10)/10; break;

alco/alcoJS.js

Lines 30 to 31 in fcc0972

	const result = Math.floor(Math.random()*(max-min))+min;
	if(result==0)return max;

alco/alcoJS.js

Line 39 in fcc0972

const oper = Math.floor(Math.random()*4);

Соблюдать правила форматирования кода:

alco/alcoJS.js

Lines 2 to 13 in fcc0972

	function operation(oper,ch2){
	switch (oper) {
	case 0:
	//if(ch2<0) return "";
	return "+";break;
	case 1:
	//if(ch2<0) return "+";
	return "-";break;
	case 2: return "*"; break;
	case 3: return "/"; break;
	}
	}

alco/alcoJS.js

Lines 21 to 28 in fcc0972

	function result(ch1,ch2,oper){
	switch (oper) {
	case 0: return rez = Math.floor(ch1+ch2); break;
	case 1: return rez = Math.floor(ch1-ch2); break;
	case 2: return rez = ch1*ch2; break;
	case 3: return rez = Math.floor((ch1/ch2)*10)/10; break;
	}
	}

alco/alcoJS.js

Lines 41 to 49 in fcc0972

	if(temp==result(ch1,ch2,oper)){
	alert('ещё по одной...');
	}
	else
	{
	alert('Хватит с тебя');
	break;
	}
	}

Непонятные имена констант:

alco/alcoJS.js

Lines 37 to 38 in fcc0972

	const ch1 = getrandom(min,max);
	const ch2 = getrandom(min,max);

При перечислении параметров после запятой ставим пробел:

alco/alcoJS.js

Line 2 in fcc0972

function operation(oper,ch2){

alco/alcoJS.js

Line 21 in fcc0972

function result(ch1,ch2,oper){

alco/alcoJS.js

Line 29 in fcc0972

function getrandom(min,max){

alco/alcoJS.js

Lines 37 to 38 in fcc0972

	const ch1 = getrandom(min,max);
	const ch2 = getrandom(min,max);

alco/alcoJS.js

Lines 40 to 41 in fcc0972

	const temp = prompt(`${ch1}${operation(oper,ch1,ch2)}${ch2}=?`);
	if(temp==result(ch1,ch2,oper)){