2

function truncate(str, maxlength){
	if(str.length > maxlength.length){
		alert(str.substr(0,3) + "...");
		// alert(a);
	}
	else{
		alert(str);
	};
};

truncate("громомпораженный", 'asasas');
Это правильно решение ? или можно улучшить ?

1
  • после } не нужно делать ; - это только исключение, когда вы в переменной пишите анонимную или стрелочную функцию. так-как блок переменной должен завершиться ;. В остальном - это избыточный код.
    – And
    Commented 25 нояб. 2018 в 8:13

1 ответ 1

2

Если вы хотите как можно меньше кода, то держите:

function truncate(str, maxlength){
  alert(str.length > maxlength.length ? str.substr(0,3) + "..." : str);
}

truncate("громомпораженный", 'asasas');
truncate("html", 'asasas');

Если же вы смотрите на это так 😯, то читайте тернарный оператор.

6
  • @xes ? это if, :это else, ничего сложного. Ну а перед ? условие if.
    – Ver Nick
    Commented 25 нояб. 2018 в 8:32
  • я и написал что это если грубо сказать замена условного оператора
    – xes
    Commented 25 нояб. 2018 в 8:34
  • @xes Ну да. :))
    – Ver Nick
    Commented 25 нояб. 2018 в 8:35
  • логичнее: alert(str.length > maxlength.length ? str.substr(0,maxlength-3) + "..." : str);
    – sair
    Commented 26 нояб. 2018 в 10:11
  • @sair ваш код неправильно работает.
    – Ver Nick
    Commented 26 нояб. 2018 в 10:14

Ваш ответ

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

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