[MySQL][PHP]Problem z edytowaniem rekordów w bazie mysql z wykorzystaniem mysqli, Nie moge edytować pojdynczego rekordu w bazie. Opcje

[Misza71]

Dzien dobry.
Próbuje zrobić swój pierwszy rozbudowany projekt. Wszystko szło okey do momentu w którym chciałbym edytować pojedynczy rekord w bazie. Wstawie kod będzie łatwiej wytłumaczyć. Wyskakuje mi wiadomość z catcha mimo tego że żadne throw nie zostaje uruchomiony a sama edycja pominięta jeżeli ktoś by mi wytłumaczył gdzie robię błąd byłbym wdzięczny.

[PHP] pobierz, plaintext 
 
   <?php
 
   require_once "TryToConnect.php"; // Jest tam funkcja magicConnect 
   session_start();
 
 
    try
   {
 
   $connection = new magicConnect;
   $resultOfMagicConnect=$connection->tryToConnect();
 
   if($resultOfMagicConnect==false)
   {
      echo "Pierwszy throw"; 
       throw new Exception(mysqli_connect_errno());  
   }
 
   $comic_id =  $_GET["id"];
 
    $rowResults = $resultOfMagicConnect->query("SELECT * from registration where id_registration = $comic_id"); // Wyswietla dane w formularzu
 
    $results=$rowResults->fetch_assoc();
 
    $editCategory =$results["category"];
    $editPriority = $results["priority"];
    $editPayment = $results['payment'];
 
 
 
           if(!$rowResults)
           {
            echo "Drugi throw";
               throw new Exception($connect->error); 
           }
 
               if($rowResults)
               {
 
 
 
                if(isset($_POST['yourTitle']))
                {
                    $newTitle =$_POST['yourTitle'];
 
                    $updateQuery=("UPDATE registration SET title ='$newTitle' WHERE id_registration = '$comic_id' ");
                    echo "Drugi "; // Kod działa tylko do tego momentu
                    $edit=mysqli_query($resultOfMagicConnect, $updateQuery);
 
                    echo "Drugi ";
                    if(mysqli_query($resultOfMagicConnect, $updateQuery)) // Nie wchodzi do tego ifa
                    {
                        echo "Drugi ";
                        echo "Records were updated successfully.";
                    }
                    echo "Drugi "; // te echo też omija
 
 
 
 
                  }
 
 
                }
 
       }
 
 
 
           catch(Exception $e)
           {
               echo '"Błąd serwera! Spróbuj innym razem."'; // Wyswietla sie htrow ppo wszystkim
           }
 
           ?>
[PHP] pobierz, plaintext 

Tutaj funcja magic Connect

[PHP] pobierz, plaintext 
<?php
 
class magicConnect
{
    public function tryToConnect()
    {
    $result = new mysqli('localhost','root','','systemzglaszaniaawarii');
    if ($result->connect_errno ===0)
    {
        return $result;
    }
    else
    {
        return false;
    }
}
}
 
?>
 
[PHP] pobierz, plaintext 

Z góry dziekuję i pozdrawiam

[nospor]

Przeciez obiekt $e zawiera dokladna wiadomosc co i gdzie jest zle. Zamiast latac po forach, wpierw wyswietl to sobie a sie dowiesz co ci rzuca wyjatkiem

[Misza71]

Dzięki za opowiedz. Nie pomyślałem o tym
Dobra stary dzięki wielkie mam problem ze zmienną $comic_id/ Gdy zmieniłem na numer rekordu 32 normalnie zmienia tytuł w bazie.
Treść błędu to
"mysqli_sql_exception: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '32''' at line 1 in C:\xampp\htdocs\SystemZglaszaniaMarkNaDomene\editTitle.php:50 Stack trace: #0 C:\xampp\htdocs\SystemZglaszaniaMarkNaDomene\editTitle.php(50): mysqli_query(Object(mysqli), 'UPDATE registra...') #1 {main}

Wiesz może czy zmienną którą wprowadzam w instrukcji update musze konwertować na jakis konkretny typ?

Działa musiałem skonwertować na inta dziękuję za podpowiedz

Ten post edytował Misza71 12.07.2022, 13:47:56

[nospor]

Skoro uzywasz mysqli poczytaj o BINDowaniu danych do zapytania. W tej chwili twoj kod jest dziurawy jak ser szwajcarski

[Misza71]

Zabezpiecze przed sqlinjection jak będzie działac. Jeszcze musze edytować całą bązę danych. No i teraz jest kolejny problem bo po konwersji wartośc mojej zmiennej to zero...

[nospor]

Watpie ze 0 jest wynikiem kowersji. Poprostu ta zmienna od poczatku jest kijowa

Zaprzyjaznij sie z VAR_DUMP i stosuj zawsze gdzie popadnie


var_dump($_GET);
i zobacz dokladnie co siedzi w get a nie polegasz na jakis zalozeniach

[Misza71]

nospor dzięki za wskazówki i pomoc. Bez Ciebie siedziałbym jeszcze pół dnia albo zmieniał cały kod. Pozdrawiam

Za chwilkę sprawdze co robi Var_dump bo myślałem badziej o ctype_alnum i sprawdzaniu o długości. W zmiennej siedział string zle zabrałem się za konwersje .

Bindowanie działa też przy użyciu mysqli? czy tylko w pdo?

[nospor]

mysqli tez ma bindowanie. Temu napisalem ze skoro uzywasz mysqli to zainteresuj sie bindowaniem

[Misza71]

Wstawie mój skończony kod. Zrobiłem te binodwanie ale nie działa nie wyskakuje żaden błąd ale dane nie są aktualizowane.

[PHP] pobierz, plaintext 
 
   <?php
 
   require_once "TryToConnect.php"; // Jest tam funkcja magicConnect 
   session_start();
 
 
 
    try
   {
 
   $connection = new magicConnect;
   $resultOfMagicConnect=$connection->tryToConnect();
 
   if($resultOfMagicConnect==false)
   {
 
       throw new Exception(mysqli_connect_errno());  
   }
 
   $comic_id =  $_GET["id"];
 
    $rowResults = $resultOfMagicConnect->query("SELECT * from registration where id_registration = $comic_id"); // Wyswietla dane w formularzu
 
    $results=$rowResults->fetch_assoc();
 
    $editCategory =$results["category"];
    $editPriority = $results["priority"];
    $editPayment = $results['payment'];
    $check=true;
 
 
 
           if(!$rowResults)
           {
 
               throw new Exception($connect->error); 
           }
 
               if($rowResults)
               {
 
 
 
                if(isset($_POST['yourTitle']))
                {
                    $newTitle =$_POST['yourTitle'];
                    if(strlen($newTitle)<=3 || strlen($newTitle)>30) //imie musi skladac sie od 3 do 20 liter
                    {
                       $check=false;
                       echo "Tytuł musi składać się od 3 do 30 liter.";
 
                    }
                    if(ctype_alnum($newTitle)==false) // Sprawdza czy ciąg znaków składa się z cyfr i liczb 
                    {
                       $check=false;
                        echo "Tyutł może składać się tylko z liter (bez polskich znaków).";
 
                    }
 
                    if($check ==true)
                    {
                        htmlentities($newTitle,ENT_QUOTES, "UTF-8");
 
                  intval($comic_id);
 
                    $updateQuery=("UPDATE registration SET title ='$newTitle' WHERE id_registration = $comic_id");
 
//TUTAJ
 
                 $stmt =$resultOfMagicConnect->prepare("UPDATE registration SET title =? WHERE id_registration = ?");
 
                 $stmt->bind_param("si",$newTitle, $comic_id);
                        echo "Cos";
                        $stmt->execute();
 
                 $edit=mysqli_query($resultOfMagicConnect, $updateQuery);
 
                    header ('Location:accident.php');
 
 
                    }
 
                  }
 
 
                }
 
       }
 
 
 
           catch(Exception $e)
           {
               echo '"Błąd serwera! Spróbuj innym razem."'; // Wyswietla sie htrow ppo wszystkim
                echo $e;
           }
           $resultOfMagicConnect->close(); 
           ?>
[PHP] pobierz, plaintext 

Prosze jak znajdziesz chwilę rzuć okiem bo nie mam pomysłu. To już ostatnia prośba pozdrawiam

Ten post edytował Misza71 13.07.2022, 10:18:18

[nospor]

1) Sie zdecyduj i albo robisz bindowanie albo nie. Teraz to samo zapytanie raz odpalasz z bindowaniem a raz bez

2) VAR_DUMP, pamietasz?

Pokaz co zwraca
var_dump($newTitle);
var_dump($comic_id)

[Misza71]

Wiem że miałem dwa sposoby edycji ale nie działało bindowanie i po prostu robiłem dalej projekt.
Zawartośc zmiennych:
newTitle = string(9) "Tytul"
comic_id =string(4) "'32'"

Ten sam problem z tym że nie wiem jak konwertować na inta. Wcześniej w kodzie odbywa się taka konwersja

[nospor]

no jak ci to ma dzialac, jak twoja liczba przekazana do bindowania zawiera apostrofy... Teraz masz '32' a masz miec 32. Do bindowania przekazuje sie normalne dane a nie w ciapkach

Prosilem wczesniej zrobic
var_dump($_GET);
Bo teraz to wyglada jakbys te dane w ciapkach przekazywal juz GET i POSTem. No chyba ze gdzies tam w odchlani twojego kodu sam dodajesz te ciapki. Nie wiem, z kodu co pokazales tutaj nic takiego nie wynika. Wiec albo GETem slesz od razu w ciapkach albo sprawdz czy sam gdzies nie dodajesz.

[Misza71]

Miałes rację w get przekazywałem dodatkowe ''. Byłem świecie przekonany że gdy używasz instrukcji echo to apostrofy drukują nazwę zmiennej za to średniki jej wartośc.

[PHP] pobierz, plaintext 
  <table>
   <thead>
      <tr>
         <th>Tytuł</th> <th>Opis</th> <th>Priorytet</th><th>Kategoria</th><th>Platnosc</th>
      </tr>
   </thead>
   <tbody>
      <tr>
 
 
      <?php 
 
     echo "<td><a href=editTitle.php?id=$comic_id>"
      ?> <?php echo $editTitle?></a></td> 
      <td><a href="#"> OpisKtoregoNIEma</td> 
      <?php 
 
     echo "<td><a href=editPriory.php?id=$comic_id>"
      ?> 
      <?php echo $editPriority?></td>
 
      <?php 
 
         echo "<td><a href=editCategory.php?id=$comic_id>"
          ?> 
        <?php echo $editCategory?></td> 
       <?php 
 
         echo "<td><a href=editPayment.php?id=$comic_id>"
          ?>
           <?php echo $editPayment?></td>
      </tr>
 
      </tr>
   </tbody>
[PHP] pobierz, plaintext 

To jest kod który wysyła geta z strony poprzedzającej edycje tytułu. Dzięki za poświęcony czas i pomoc, najprawdopodobniej odpuściłbym bindowanie i zrobił to tradycyjną metodą. Pozdrawaim ten php wcale nie jest porsoty jak się wydaje

Ten post edytował Misza71 13.07.2022, 14:26:42

[nospor]

ten php wcale nie jest porsoty jak się wydaje

Jak ktos robi glupie bledy to wszystko wydaje sie trudne

Byłem świecie przekonany że gdy używasz instrukcji echo to apostrofy drukują nazwę zmiennej za to średniki jej wartośc.

Nie boj sie robic eksperymentow by doskonalic swoja wiedze. NIe robisz przeciez aktywnego reaktora jadrowego. Nie rozwalisz pol swiata:

[PHP] pobierz, plaintext 
$zmienna ="Ala ma kota z apostrofem'";
echo "$zm";
echo $zm;
echo '$zm';
[PHP] pobierz, plaintext 

[Misza71]

Dobra pomieszało mi się jej na odwrót Jest jakaś zasada kiedy używać cudzysłowów a kiedy nie? W sensie zmienna zapisana normalnie daje ten sam efekt. Wiem że gdy pisałem różne instrukcje np SELECT bazie musiałem operować na cudzysłowiach i

[nospor]

Cudzyslow uzywasz z grubsza gdy masz dlugi tekst do ktorego wkladasz zmienna a jestes na tyle leniwy ze nie chce ci sie laczyc tekstow np:

[PHP] pobierz, plaintext 
 
$zm ='blabla';
echo "Wersja $zm dla leniwych";
echo 'Wersja ' . $zm .' dla laczycych teksty';
[PHP] pobierz, plaintext 

Oczywiscie zartuje z ta wersja dla "leniwych"

[Misza71]

Dobra na teraz już sobie odpuszczam. Kurczę nie wiem nawet jak mam Ci podziękować bo pisanie tego po raz 10 to już trochę cringe. Mam nadzieję ża nauka będzie owocna bo chciałbym sie po tym projekcie postarać gdzieś o praktyki

Lewniwi obracją świat

[nospor]

Swoja droga w ostatnim kodzie co pokazales, bez sensu w oogle ze wchodziles w php. Skoro wiekszosc kodu to html to powinno to wygladac tak:

[PHP] pobierz, plaintext 
      <table>
       <thead>
          <tr>
             <th>Tytuł</th> <th>Opis</th> <th>Priorytet</th><th>Kategoria</th><th>Platnosc</th>
          </tr>
       </thead>
       <tbody>
          <tr>
 
 
 
         <td><a href=editTitle.php?id=<?= $comic_id ?>>
          <?= $editTitle?></a></td> 
          <td><a href="#"> OpisKtoregoNIEma</td> 
 
         <td><a href=editPriory.php?id=<?= $comic_id?>>
          <?= $editPriority?></td>
 
 
             <td><a href=editCategory.php?id=<?= $comic_id?>>"
            <?= $editCategory?></td> 
             <td><a href=editPayment.php?id=<?= $comic_id?>>"
               <?= $editPayment?></td>
          </tr>
 
          </tr>
       </tbody>
[PHP] pobierz, plaintext 

[Misza71]

Okey zmienię dzieki.