algorithm - 这个更有效的版本？

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;
}