более эффективная версия этого?

Question

у меня есть эта штуковина здесь:

function numOfPackets(bufferSize, packetSize) { 
    if (bufferSize <= 0 || packetSize > bufferSize) return 0; 
    if (packetSize < 0) throw Error(); 
    var out = 0; 


    for(;;){ 
    out++; 
    bufferSize = bufferSize - packetSize; 
    if(packetSize > bufferSize) break; 
    } 

    return out; 
} 

, который я бегу на часто, может у меня более сверхвысоком вариант этого?

Answer 1

Я предполагаю, что вы пишете в JavaScript или что-то подобное, и в этом случае использование:

function numOfPackets(bufferSize, packetSize) { 
    if (bufferSize <= 0) return 0; 
    if (packetSize <= 0) throw Error(); 
    return Math.floor(bufferSize/packetSize); 
} 

Также обратите внимание, что исходный код входит в бесконечный цикл при packetSize = 0;

Эквивалентное в C++:

int numOfPackets(int bufferSize, int packetSize) { 
    if (bufferSize <= 0) return 0; 
    if (packetSize <= 0) throw Error(); 
    return bufferSize/packetSize; 
}