version plus efficace de cela?

Question

j'ai ce machin ici:

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

que je dirige à souvent, peut me donner variante plus efficent de celui-ci?

Answer 1

Je suppose que vous écrivez en JavaScript ou quelque chose de similaire, dans lequel cas d'utilisation:

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

Notez également que votre code d'origine entre dans une boucle infinie lorsque packetSize = 0;

Équivalemment en C++:

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