effizientere Version von diesem?

Question

Ich habe dies hier THINGY:

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

, die ich bei oft laufen, kann u mich mehr efficent Variante davon geben?

Answer 1

Ich denke, dass Sie in JavaScript oder etwas ähnliches, wobei in diesem Fall die Verwendung schreiben:

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

Beachten Sie auch, dass Ihr Original-Code tritt in eine Endlosschleife, wenn packetSize = 0;

Gleichwertig in C++:

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