2013-12-12 4 views
0

Вы можете видеть, что нижеследующая функция содержит длинную строку кода ... Я хочу упростить это ... Одна идея заключалась бы в использовании нескольких переменных, но я не хочу для этого ...Нужно упростить эту длинную строку javascript

Любая идея о том, как я могу это упростить?

function updateStatusBar() { 
    var text = textarea.value; 
    statusBar.value = "Words: " + (text.split(/\b\S+\b/g).length - 1) + " Characters: " + text.replace(/\n|[" "]/g, "").length + "/" + text.replace(/\n/g, "").length; 
} 
+3

Этот вопрос, вероятно, больше подходит для [Обзор Code] (http://codereview.stackexchange.com). –

+0

Я думаю, что линия делает то, что она должна делать как можно короче ... –

+2

Этот вопрос не соответствует теме, потому что он принадлежит http://codereview.stackexchange.com/ – Quentin

ответ

3

Вы можете использовать другие функции, чтобы сломать функциональность:

function updateStatusBar() { 
    var text = textarea.value; 
    statusBar.value = "Words: " + getWordCount(text) + " Characters: " + getCharCount(text); 
} 

function getWordCount(text) { 
    return (text.split(/\b\S+\b/g).length - 1); 
} 

function getCharCount(text) { 
    return text.replace(/\n|[" "]/g, "").length + "/" + text.replace(/\n/g, "").length; 
} 
Смежные вопросы