1

Как видно ниже, в коде много повторяющихся конструкций - отличие лишь в переменной. Как этот код можно переписать более профессионально и красивее?

if (type == 'private') {

    if (increment == 'plus') { privateCount++;} else if(increment == 'minus') { privateCount--;}
         io.sockets.emit('counter_'+usersOnline[socket.id], JSON.stringify({ 'privateCount' : privateCount }));

      } else if (type == 'public') {

        if (increment == 'plus') { publicCount++;} else if(increment == 'minus') { publicCount--;}
        io.sockets.emit('counter_'+usersOnline[socket.id], JSON.stringify({ 'publicCount' : publicCount }));

    } else if (type == 'chat'){

        if (increment == 'plus') { chatCount++;} else if(increment == 'minus') { publicCount--;}
        io.sockets.emit('counter_'+usersOnline[socket.id], JSON.stringify({ 'chatCount' : chatCount }));
    }
 }
1
  • @IOleg, Если вам дан исчерпывающий ответ, отметьте его как верный (нажмите на галку рядом с выбранным ответом). 29 дек 2014 в 10:31

4 ответа 4

5

Вроде ничего не забыл...

 (function(type) { //чтобы не задеть message или counters если они вдруг есть в родительском scope;
 var counters = {
    'private': parseInt(privateCount)||0,
    'public': parseInt(publicCount)||0,
    'chat': parseInt(chatCount)||0
};
if (counters[type] !== undefined) {
    counters[type] += increment == 'plus' ? 1 : -1;
    var message = {};
    message[type + 'Count'] = counters[type];

    io.sockets.emit(
        'counter_' + usersOnline[socket.id],
        JSON.stringify(message));
    //тут можно вместо JSON.stringify, самому строку собрать, типа так
    //'{"'+type+'Count": '+counters[type] +'}';
    //обычно это делать нехорошо, но тут типы всех данных известны

    privateCount = counters.private;
    publicCount = counters.public;
    chatCount = counters.chat;
}
})(type);
2
  • не придраться :), хотя если в increment будет мусор, count сминусуется. 26 фев 2014 в 22:22
  • я знаю, думал написать это в комменте или нет..
    – zb'
    26 фев 2014 в 22:28
2

Не использовать, но зато 2 строчки:

eval(type + "Count += (increment == 'plus') ? 1 : -1;");
console.log('counter_', '{"'+type+'Count":"'+eval(type + "Count")+'"}');
0

C вольными модификациями

var counters = {'private':0, 'public':0, 'chat':0 };
if (['private', 'public', 'chat'].indexOf(type) === -1) {
    return false;
}
if (increment) { // пушо нет никакого смысла в переменную "прибавление" писать не булев, а строку "минус"
    counters[type]++; // так как я тупой и не знаю, можно ли выудить переменную по названию, проще (и удобнее!) завести объект для хранения счетчиков
} else {
    counters[type]--;
}
var key = type+'Count';
io.sockets.emit('counter_'+usersOnline[socket.id], JSON.stringify({key: counters[type]}));
9
  • А есть вообще в node.js понятие класса? чтобы применить полиморфизм
    – IOleg
    26 фев 2014 в 19:20
  • @IOleg, есть, но не в node.js, а просто в js.
    – etki
    26 фев 2014 в 19:21
  • С комментариями в коде будьте полегче - это оскорбление
    – IOleg
    26 фев 2014 в 19:23
  • @IOleg, поверьте, когда я назвал себя тупым, мои чувства задеты не были.
    – etki
    26 фев 2014 в 19:27
  • 2
    JSON.stringify({key: counters[type]})); нельзя так, key будет строкой key
    – zb'
    26 фев 2014 в 21:36
-4
if (type == 'private') {
    if (increment == 'plus') {
        privateCount++;
    }else if(increment == 'minus') {
        privateCount--;
    }io.sockets.emit('counter_'+usersOnline[
        socket.id
    ],
    JSON.stringify({
        'privateCount' : privateCount
    }));
}else if (type == 'public') {
    if (increment == 'plus') {
        publicCount++;
    }else if(increment == 'minus') {
        publicCount--;
    }io.sockets.emit('counter_'+usersOnline[
        socket.id
    ],
    JSON.stringify({
        'publicCount' : publicCount
    }));
}else if (type == 'chat'){
    if (increment == 'plus') {
        chatCount++;
    }else if(increment == 'minus') {
        publicCount--;
    }io.sockets.emit('counter_'+usersOnline[
        socket.id
    ],
    JSON.stringify({
        'chatCount' : chatCount
    }));
}
}

http://tools.arantius.com/tabifier  этот сервис тебе помощь))
3
  • Вы хотя бы всматривались в свой листинг? io.sockets.emit('counter_'+usersOnline[ socket.id ], Это нормально?
    – Nofate
    26 фев 2014 в 21:16
  • А что здесь не так?
    – IOleg
    26 фев 2014 в 21:29
  • @IOleg, это не вам, это для @sevakjan
    – Nofate
    26 фев 2014 в 21:54

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

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