Dodawanie liczby do zmiennej

pytanie zadane 22 marca 2019 w JavaScript przez Jokii Nowicjusz (240 p.)
edycja 23 marca 2019 przez ScriptyChris

Dzień dobry mam problem z takim skrypcikiem:

<?php
  session_start();
?>
<html lang="pl">
<head>
    <meta http-equiv="content-type" content="text/html; charset=utf-8">
    <title>Strona Główna</title>
    <link rel="Stylesheet" href="style.css">

<script>
 var Money = '<?php echo $_SESSION['money']; ?>';
 function addmoney()
 {
    Money = Money + 1;
    document.getElementById("moneyzdobyte").innerHTML = Money;
 }
</script>

</head>
    
<body>
<?php
 
        if (!isset ($_SESSION['zalogowany']))
        {
        echo "<div id='container'>";
        echo $_SESSION['login']." [<a href='logout.php'> Wyloguj się!</a>]<br>";
        echo "Zarobiłeś: <div id='moneyzdobyte'>0</div>";
        echo "<button onclick='addmoney()'>Zarabjaj</button>";
        } else header("Location: index.php");
?>
</body>
</html>

Otóż zamiast wyniku załóżmy 250 + 1 = 251 to otrzymuje 2501... i mogę dodawać te jedynki w nieskończoność gdzie jest problem?

komentarz 22 marca 2019 przez kalczur Gaduła (4,320 p.)

Dzieje się tak ponieważ traktuję to jak tekst, a więc przed dodaniem do tego jakiejś liczby daj
Money = parseFloat(Money); jeśli chcesz zmiennoprzecinkowy a jeśli całkowite to parseInt()

No i co właściwie przypisujesz w 10 linijce do Money?

1	komentarz 22 marca 2019 przez lateM Pasjonat (17,660 p.) Kod w bloczek

komentarz 22 marca 2019 przez Jokii Nowicjusz (240 p.)

@kalczur, zmienną sesyjną z php z innego pliku

komentarz 22 marca 2019 przez kalczur Gaduła (4,320 p.)

Zrób tak jak pisałem i daj znać czy działa

komentarz 22 marca 2019 przez Jokii Nowicjusz (240 p.)

Dodałem do skryptu to co kazałeś i nadal nie działa

jakieś inne sugestie?

1 odpowiedź

odpowiedź 23 marca 2019 przez MrxCI Dyskutant (8,260 p.)

//Money = Money + 1;


function addMoney(money,int){

if(typeof money == `object` || typeof int == `object`){throw `param cannot be object`;}

money = typeof money == `number` ? money : parseInt(money);
int = typeof int == `number` ? int : parseInt(int);

return money+=int;

}

// usage  
// Money = addMoney(Money,20);

Taka funkcja na szybko, myślę że powinna spełnić podstawowe oczekiwania

Założyłem ze zmienna money nie może być liczbą zmiennoprzecinkową.