Minimo comune multiplo in php

[TheMaker94]

Salve, sono ancora io
Vorrei calcolare l'mcm di 2 numeri secondo l'algoritmo di Euclide:


Quindi ho impostato il codice come segue:

<?php
$n1 = 47;
$n2 = 24;
//funzione del MCD
function MCD($n1, $n2){
    while ($n1 != $n2){
        if ($n1 > $n2){
            $n1 = $n1 - $n2;
        } else {
            $n2 = $n2 - $n1;
        }
    }
    return $n1;
}

$mcd = MCD($n1, $n2);

//funzione del mcm
function mcm($n1, $n2){
    ($n1*$n2)/$mcd;
return $n1;
}

    $mcm = mcm($n1, $n2);
    echo "Il mcm è $mcm.";
?>

Per il calcolo del'MCD ho visto su internet, mentre l'mcm l'ho calcolato con l'algoritmo di Euclide.
Dato che mi dà una serie di errori:

Notice: Undefined variable: mcd in c:\programmi\easyphp1-8\www\prove.php on line 20

Warning: Division by zero in c:\programmi\easyphp1-8\www\prove.php on line 20
Il mcm è 47.

ho tentato di trovare qualcosa sul web e ho notato che il codice lo impostano diversamente..
Ora la mia domanda è.. perchè non è possibile farlo seguendo l'algoritmo di euclide? (sarò sicuramente io che sbaglio, quindi illuminatemi )

[milejo]

ciao senti così non potra mai funzionare perchè il valore di $mcd nella funzione mcm sarà sempre indefinito. semmai potrà funzionare così...

<?php
$n1 = 47; 
$n2 = 24; 
//funzione del MCD 
function MCD($n1, $n2){ 
    while ($n1 != $n2){ 
        if ($n1 > $n2){ 
            $n1 = $n1 - $n2; 
        } else { 
            $n2 = $n2 - $n1; 
        } 
    } 
    return $n1; 
} 


//funzione del mcm 
function mcm($n1, $n2, $MCD){ 
         
    ($n1*$n2)/$MCD; 
return $n1; 
} 

    $mcd = MCD($n1, $n2);  
    $mcm = mcm($n1, $n2,$mcd);
    
    echo "Il mcm è $mcm."; 
?>

[TheMaker94]

Grazie per la risposta
Ora non mi segnala errori, solo che la funzione non funziona (scusa il gioco di parole ) perchè mi restituisce $n1

[dav22]

<?php
$num1 = 47; 
$num2 = 24; 

//funzione del MCD 
function MCD($n1, $n2){ 
    while ($n1 != $n2){ 
        if ($n1 > $n2){ 
            $n1 = $n1 - $n2; 
        } else { 
            $n2 = $n2 - $n1; 
        } 
    } 
    return $n1; 
} 

//funzione del mcm 
function mcm($n1, $n2, $nMCD){ 
         
    ($n1*$n2)/$nMCD; 
    return $n1; 
} 

    $m1 = MCD($num1, $num2);  
    $m2 = mcm($num1, $num2, $m1);
    
    echo "MCD: $m1 - mcm: $m2";
?>

ok ammetto di non averlo provato e di averlo modificato al volo.. ma così funziona? Spero di non aver scritto delle cavolate..

[TheMaker94]

Ancora niente
dav22 potresti spiegarmi gentilmente che modifiche hai apportato?

[mr.click]

<?php
$n1 = 47;
$n2 = 24;
//funzione del MCD
function MCD($n1, $n2){
    while ($n1 != $n2){
        if ($n1 > $n2){
            $n1 = $n1 - $n2;
        } else {
            $n2 = $n2 - $n1;
        }
    }
    return $n1;
}

//funzione del mcm
function mcm($n1, $n2){
        
    $mcd = MCD($n1, $n2)
    $mcm = ($n1*$n2)/$mcd;
    return $mcm;
}
 
$mcm = mcm($n1, $n2);
echo "Il mcm è ".$mcm;
?>

prova cosi....dovrebbe andare

[dav22]

Originariamente inviato da TheMaker94
Ancora niente
dav22 potresti spiegarmi gentilmente che modifiche hai apportato?

ho fatto la modifica ma non ho controllato l'algoritmo.. e infatti il tuo errore è sempre stato che in mcm fai si il calcolo ma poi te restituisci sempre e solo n1

//funzione del mcm 
function mcm($n1, $n2){ 
    ($n1*$n2)/$mcd; 
return $n1; 
}

basta che lo cambi con (almeno nel mio esempio)

//funzione del mcm 
function mcm($n1, $n2){ 
   return ($n1*$n2)/$mcd; 

}

oppure usi direttamente quello di mr.click

[TheMaker94]

Grazie mille a tutti per l'aiuto, quello di mr.click funziona benissimo ^^