Proszę o szybką pomoc php, ajax, codeigniter

Question

Mam w folderze helpers stworzony plik helpers/response.php który wygląda tak : 

<?php
function response($message){
  if(isset($_GET['format']) && !empty($_GET['format'])){
    switch($_GET['format']){
        case 'json':
          print_r(json_encode($message));
          break;
        default:
          print_r($message);
          break;
    }
  } else {
    print_r($message);
  }
}

ma on robić tak ze jeżeli wpiszemy jakąś ścieżkę np. localhost/np/np/1?format=json to wyświetli bazę danych w js. zamiast w json wyświetla w html'u albo wgl co jest źle?

Comments

if(isset($_GET['format']) && !empty($_GET['format'])){

do

if(!empty($_GET['format'])){

Na jedno wyjdzie. Bo jak nie jest pusty, to jest ustawiony. I nie, nie powoduje wyswietlenia notice/warninga.

   print_r($message);

do

   echo $message;

Bo print_r zjada więcej zasobów :P

Ja bym to napisał tak:

<?php
//php 7
function response($message){
    switch($_GET['format'] ?? null){
        case 'json':
          echo json_encode($message);
          break;
        default:
          echo $message;
          break;
    }
}

//php 5.6

function response($message){
    switch(isset($_GET['format']) ? $_GET['format'] : null){
        case 'json':
          echo json_encode($message);
          break;
        default:
          echo $message;
          break;
    }
}

 

---

Pomimo tego dalej wyświetla jak html a nie przetwaza na js

Answer 1

Co dostajesz, jak wejdziesz w /get/1?format=json?

Odpal devtools przeglądarki (f12) z zaznaczoną zakładką networks, przeladuj, zaznacz na liscie /get/1?format=json i pokaz screenshota

Comments

Dodam ci wszystkie pliki jakie mam na gitcie ok?

---

https://github.com/Wonderpol/Eng2/ jak chcesz to moge dać jeszcze bazę MySQL

---

nie bedzie potrzebna

---

$this->input 

Nie działa ci, jak należy. Mozesz robić to za pomocą zmiennych środowiskowych ($_GET, $_POST), ale to nie jest zgodne ze sztuką i odradzam.

W ogóle zapomnij o używaniu print_r... zamień to na echo

Twoja funkcja (metoda) get "powinna" wyglądać:

    public function get($_id = null)
    {
        if (!$_id || !($zadania = $this->Components_model->get($_id))) {
            http_response_code(404);

            return;
        }

        if ($this->input->get('format') == 'json') {
            echo json_encode($zadania);
        } else {
            http_response_code(400);
            die('Not Supported Format');
        }
    }

 

Dodaj mój przykład i zamień $this->input->get('format') na ($_GET['format'] ?? ''), i sprawdź, czy ci zadziała.

W ogóle nie podoba mi się ten framework. Nie widzę nawet HttpNotFoundException, a robienie wszystkiego http_response_code mija się z celem. Jeśli dopiero co zaczałeś, możesz przenieść projekt na Yii2, Symfony. 

---

Dzięki ziomek zrobione i w json'ie działa ładnie już a framwork'a raczej nie będę zmieniał mimo że ma wad kilka

Answer 2

Po prosimy kod JS.

Comments

          <script>
                    $(document).on('click' , '.info-button', function(){
                        var id = $(this).data('id');

                      $.ajax({
                        url: 'get/' + id + '?format=json',
                        type: 'json',
                        succes: function(respone){

                        }
                      });
                          });



                </script>
a błąd w konsoli 404

---

Jak błąd 404, to sprawdź, czy istnieje taka strona :P

I pamiętaj że: 'get/' + id + '?format=json' to nie to samo, co '/get/' + id + '?format=json', po moze ci chodzilo o to 2.