Devolver número aleatorio pero no 2

Question

¿Por qué esto a veces devuelve 2?

function pickServer(){ 
    $varr = rand(1,4); 
    if($varr==2){ 
     pickServer(); 
    } 
    return $varr; 
} 

Answer 1

Porque no se detiene la función allí. la cuarta línea debería decir:

return pickServer(); 

Answer 2

Porque cuando el valor es 2 no devuelve pickserver. Y la función continúa devolviendo $ varr.

Answer 3

Llama a la función recursivamente para obtener otro número que no es 2, pero no hace nada con ese resultado.

Intente cambiar pickServer() a return pickServer().

Mejor aún, escribir la función iterativa para que solo un bucle hasta que el valor devuelto no es 2.

Answer 4

function pickServer(){ 
    $varr = rand(1,4); 
    if($varr==2){ 
     return pickServer(); //leave here 
    } 
    return $varr; 
} 

Answer 5

Lo que probablemente quiere es

function pickServer(){ 
    $varr = rand(1,4); 
    if($varr==2){ 
    $varr = pickServer(); 
    } 
    return $varr; 
} 

- pero tenga en cuenta que no hay no hay garantía de que esto no vaya a una recursividad demasiado larga. Tal vez debería más bien hacer algo como esto:

function pickServer(){ 
    $varr = rand(1,3); 
    if($varr > 1){ 
    $varr = $varr + 1; 
    } 
    return $varr; 
} 

Answer 6

que se olvidó de devolver el valor ...

function pickServer(){ 
$varr = rand(1,4); 
if($varr==2){ 
    return pickServer(); 
} 
return $varr; 
} 

Answer 7

Otra manera de hacerlo es utilizando do … while:

function pickServer() { 
    do { 
     $varr = rand(1,4); 
    } while ($varr == 2); 
    return $varr; 
} 

Answer 8

La respuesta a su pregunta, como han señalado otros, es que su código se cae sin regresar. Si 2 es devuelto por la llamada a rand() en el primer intento y el segundo intento (hay una probabilidad 1/16 de que esto ocurra), obtendrá 2 como resultado.

Pero su enfoque para resolver el problema podría ser mejor.

Tanto la recursividad como el bucle son molestos para este problema. Este es un problema de asignación de , no es un problema de aleatoriedad . (Se parece algo de aleatoriedad común de codificación problemas de entrevistas que pueden ser manejados con mayor facilidad en un bucle de rechazo, pero en realidad no es un problema de esa clase.)

¿Quieres uno de los tres resultados, no cuatro . (1, 3 y 4.) Eso significa que debe generar un rango de tres números aleatorios, no cuatro. Puede reasignar con una matriz o usar un if. Ambas posibilidades se muestran a continuación. Avíseme si tengo una sintaxis incorrecta: mi PHPfu es débil esta mañana.

/* array remapping */ 
function pickServer() { 
    $remap = array(1, 3, 4); 
    return $remap[rand(1,3)]; 
} 

/* if remapping */ 
function pickServer() { 
    $server = rand(1,3); 
    if ($server==2) { 
     $server=4; 
    } 
    return $server; 
} 

No me di cuenta antes, pero balpha anticipó mi respuesta. Remató con un if en su segundo ejemplo. En lugar de reasignar 2 a 4, acaba de agregar uno a cualquier respuesta por encima de 1, que es una solución equivalente.

Answer 9

yo por lo menos así:

function pickServer() 
{ 
$servers = array(1,3,4); 
return $servers[rand(1,count($servers))]; 
} 

Answer 10

se puede quitar la recursividad y volver a asignar un seleccionados al azar 2. limitan a disminuir el alcance y el mapa los principios de la gama (en este caso 2) a 1.

function pickServer(){ 
    $varr = rand(2,4); 
    if($varr==2){ 
     return 1; 
    } 
    return $varr; 
}