UPDATE PHP MYSQL aktualizowanie rekordów.

Question

    if($_SERVER['REQUEST_METHOD'] == "POST" and isset($_POST['roll']))
    {
        func();
    } else {
      unset($_POST['roll']);
    }
    function func()
    {
        $rol = rand(1,99998);
        echo $rol, number_format($num2)."\n"; 
        if ($rol > "500000") {
          $db = mysqli_connect($server, $user, $password, $dbapp);
          $sql = "UPDATE users SET points='9299' WHERE id = 1";
          mysqli_query($db, $sql);
          echo "sukces";
          unset($_POST['roll']);
        } else {
          echo "<50000";
          unset($_POST['roll']);
        }
    };

No co mam mówić nie działa, wcześniej zrobiłem jakoś że działało ale wywalił się program bez zapisu i nie mogę cofnąć ustawień.
Znaczy dokładniej nie działa mi dodanie rekordu do bazy danych.

Answer 1

Zamien nic nie warte

mysqli_query($db, $sql);
echo "sukces";

na:

if (mysqli_query($db, $sql) === TRUE) {
    echo 'success';
} else {
    printf("Error: %s\n", $mysqli->error);
}

unset($_POST['roll']);

 

Daj znac o wyjsciu (output) funkcji "func".

Answer 2

Pierwszy błąd:

$rol = rand(1,99998);
    echo $rol, number_format($num2)."\n"; 
    if ($rol > "500000") {

zmienna $rol jest liczbą a w warunku sprawdzasz string  if ($rol > "500000")  "500000"

$rol = rand(1,99998);
    echo $rol, number_format($num2)."\n"; 
    if ($rol > 500000) {

Drugi $rol = rand(1,99998); ten zapis nigdy nie spełni tego warunku:
 if ($rol > 500000) za dużo o jedno 0 "zero" wnioskując z zapisu poniżej:
 echo "<50000";
i skąd się wzięła zmienna $num2, number_format($num2), zmienna gobalna?

Comments

Czy spełni warunek czy go nie spełni i tak nie nadpisuje mi rekordu
Co do $num2 jest to nie istotnę, testowałem coś.

---

Jak nie ma znaczenia czy się kiedykolwiek spełni warunek. 

UPDATE w Twoim kodzie zostaje "wywołany" gdy w zmiennej $rol znajdzie się liczba większa od 500000 if ($rol > 500000) w podanym przez Ciebie kodzie nigdy nie zostaje spełniony warunek, bo rand(1,99998) zwróci największą wartość to 99998, w dodatku rand  zwraca wartość typu intiger (liczbę), a ty porównujesz ją z wartością w cudzysłowach (typ string) więc jak może się wykonać kod między If a else

    $rol = rand(1,99998)!;
    ... 
    if ($rol > "500000"!) {
      $db = mysqli_connect($server, $user, $password, $dbapp);
      $sql = "UPDATE users SET points='9299' WHERE id = 1";
      mysqli_query($db, $sql);
      echo "sukces";
      unset($_POST['roll']);
    } else {
      ...
    }

Powiedzmy, że php z tym sobie poradzi i zwróci wartość true (niejawna konwersja typów), ale nie zmienia to faktu, że to jest zła praktyka programowania.

<?php
    $rol = 500001;
    echo $rol."\n"; 
    if ($rol > "500000") {
      echo "sukces";
    } else {
      echo "<50000";
    }
?>

Tu warunek zawsze będzie false

<?php
    $rol = 99998; // rand(1,99998)
    echo $rol."\n"; 
    if ($rol > "500000") {
      echo "sukces";
    } else {
      echo "<50000";
    }
?>

Tu jest jak za pomocą UPDATE ... SET  zwiększyć wartość (increase), która już jest zapisana o wartość z SET.
SQL how to increase or decrease one for a int column in one command

Można też używać "stałych", żeby zapobiec takim błędom: 

function func()
{
    const MAX_ROL = 50000;
    // define("MAX_ROL", 50000);

    $rol = rand(1,99998);
    echo $rol, number_format($num2)."\n"; 
    if ($rol > MAX_ROL) {
      $db = mysqli_connect($server, $user, $password, $dbapp);
      $sql = "UPDATE users SET points='9299' WHERE id = 1";
      mysqli_query($db, $sql);
      echo "sukces";
      unset($_POST['roll']);
    } else {
      echo "<".MAX_ROL;
      unset($_POST['roll']);
    }
};

 

Answer 3

1. Poprawiłbym nazwy funkcji i zmiennych. Coś takiego jak func albo rol nic nie wnosi. Skąd osoba to czytająca ma wiedzieć co masz na myśli?

Znaczy dokładniej nie działa mi dodanie rekordu do bazy danych

No nic dziwnego, że nie działa dodanie rekordu skoro Ty zmieniasz już istniejący.

2.Po co jest to? ->

unset($_POST['roll']);

 

Comments

1) func - funkcja
roll - moja wina nie dodałem drugiego l, ma losować mi liczbę
Oraz chodziło mi że nie zmienia mi istniejący się rekord, ciągle jest taki sam w tej bazie danych.
2) Aby po wejściu na stronę nie losowało się to automatycznie tylko po kliknieciu buttona,
jedno kliknięcie i unsetuje roll, bo 1 if sprawdza czy jest jakis roll

---

1) To rozumiem, że funkcja :D Chodzi o to aby nadać taką nazwę aby było wiadomo co ona robi :D

Roll to po angielsku rolka, zwój - chyba nie o to Ci chodziło? Może bardziej random, jak już?

2) Przecież jak wchodzisz na stronę to nie używasz metody POST tylko GET. Ten pierwszy warunek w takiej sytuacji Ci nie wejdzie, funkcja się nie wywoła. Poza tym o ile mi wiadomo to parametry w POST nie żyją dłużej niż jeden request.