[MySQL][PHP]Błąd przy wyświetlaniu danych Opcje

[warland]

Witam.
Proszę o pomoc.
Skrypt pobiera dane z bazy i wyświetla w infowindow.
Działają tylko te które w bazie mają liczby całkowite a te z dziesiętnymi już nie.
Przykład:
gdy w kolumnie 1, 2 i 3 są liczby całkowite (23,34 - 23 - 34) to ok . W chwili gdy w jednak pojawiają się liczby dziesiętne (23.5,34 - 23.5 - 34) pojawia się problem:

Kod

Query error: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 11 - Invalid query:
        SELECT
        SUM(modify_effect.population),
        SUM(modify_effect.culture),
        SUM(modify_effect.gdp),
        SUM(modify_effect.treasury),
        SUM(modify_effect.military),
        SUM(modify_effect.support)
        FROM modify_effect
            LEFT JOIN land_modifier
            ON modify_effect.id = land_modifier.modify_effect_key
        WHERE land_modifier.land_key =;

większy wycinek kodu:

[PHP] pobierz, plaintext 
function get_count_of_account_land($account_key)
 {
    $this->db->select('COUNT(*) as count');
    $this->db->from('land');
    $this->db->where('account_key', $account_key);
    $query = $this->db->get();
    $result = $query->result_array();
    return isset($result[0]['count']) ? $result[0]['count'] : 0;
 }
 // Get all modify effects
 function get_sum_effects_of_land($land_key)
 {
    $land_key = mysqli_real_escape_string(get_mysqli(), $land_key);
    $query = $this->db->query("
        SELECT 
        SUM(modify_effect.population) as population,
        SUM(modify_effect.culture) as culture,
        SUM(modify_effect.gdp) as gdp,
        SUM(modify_effect.treasury) as treasury,
        SUM(modify_effect.military) as military,
        SUM(modify_effect.support) as support 
        FROM modify_effect 
            LEFT JOIN land_modifier 
            ON modify_effect.id = land_modifier.modify_effect_key 
        WHERE land_modifier.land_key = " . $land_key . ";
    ");
    $result = $query->result_array();
    if ( isset($result[0]) ) {
        return $result[0];
    }
    return [];
 }
 // Get count of modifiers
 function get_sum_modifiers_for_land($land_key)
 {
    $land_key = mysqli_real_escape_string(get_mysqli(), $land_key);
    $query = $this->db->query("
        SELECT 
        modify_effect.id,
        modify_effect.name,
        COUNT(modify_effect.id) AS count
        FROM modify_effect 
            LEFT JOIN land_modifier 
            ON modify_effect.id = land_modifier.modify_effect_key 
        WHERE land_modifier.land_key = " . $land_key . "
        GROUP BY modify_effect.id;
    ");
    $result = $query->result_array();
    if ( isset($result[0]) ) {
        return $result;
    }
    return [];
 }
 // Get sum of effect values
 function get_sum_effects_for_account($account_key)
 {
    $account_key = mysqli_real_escape_string(get_mysqli(), $account_key);
    $query = $this->db->query("
        SELECT 
        SUM(modify_effect.population) as population,
        SUM(modify_effect.culture) as culture,
        SUM(modify_effect.gdp) as gdp,
        SUM(modify_effect.treasury) as treasury,
        SUM(modify_effect.military) as military,
        SUM(modify_effect.support) as support, 
        (
            SELECT COUNT(*) as land_count
            FROM land
            WHERE account_key = " . $account_key . "
        ) as land_count
        FROM modify_effect 
            LEFT JOIN land_modifier 
            ON modify_effect.id = land_modifier.modify_effect_key
        WHERE land_modifier.land_key IN 
        (
            SELECT id
            FROM land
            WHERE account_key = " . $account_key . "
        )
    ");
    $result = $query->result_array();
    if ( isset($result[0]) ) {
        return $result[0];
    }
    return [];
 }
 // Get all modify of land
 function get_all_modify_effects()
 {
    $this->db->select('*');
    $this->db->from('modify_effect');
    $this->db->order_by('sort_order', 'asc');
    $this->db->where('sort_order !=', 0);
    $query = $this->db->get();
    return $query->result_array();
 }
[PHP] pobierz, plaintext 

Czy ktoś może mi pomóc jak to rozwiązać? Gdzie szukać by to poprawić?

[SmokAnalog]

Pokaż jak wywołujesz get_sum_effects_of_land(...).

[warland]

[PHP] pobierz, plaintext 
// Get Land Square
        $land_square = $this->game_model->get_single_land($world_key, $coord_slug);
        $land_square['sum_effects'] = $this->game_model->get_sum_effects_of_land($land_square['id']);
        $land_square['sum_modifiers'] = $this->game_model->get_sum_modifiers_for_land($land_square['id']);
        $land_square['ambasada_list'] = false;
        if ($land_square['kapitol']) {
            $land_square['ambasada_list'] = $this->game_model->get_ambasadas_of_land($land_square['id']);
        }
        $account = false;
        if ($land_square['account_key'] != 0) {
            $account = $land_square['account'] = $this->user_model->get_account_by_id($land_square['account_key']);
            // This shouldn't happen, but it does
            // This is a workaround to ensure issue is not noticed
            if (!$account) {
                // Unclaim land
                $this->game_model->update_land_data($land_square['id'], 0, '', '', 1, '#000000');
                $land_square = $this->game_model->get_single_land($world_key, $coord_slug);
                $land_square['sum_effects'] = $this->game_model->get_sum_effects_of_land($land_square['id']);
                $land_square['sum_modifiers'] = $this->game_model->get_sum_modifiers_for_land($land_square['id']);
                $this->game_model->update_land_kapitol_status($land_square['id'], $kapitol = 0);
            } 
            else {
                $account = $land_square['account'] = $this->get_full_account($account);
            }
[PHP] pobierz, plaintext 

[SmokAnalog]

W którymś momencie $land_square['id'] zawiera pusty string, false albo innego czorta.

[warland]

kłopot w tym że nie wiem..
tak to wygląda w rzeczywistości (po kliknięciu w pola z koordynatami pełnymi (liczby całkowite) nie ma kłopotu.. Problem pojawia się gdy klikamy w pol z koordynatami z liczbami dziesiętnymi..
https://cvinto.pl/land/world/7?land=-49.5,-26

Nie wiem czy nie odpowiada za to ta część kodu (może z tabeli tylko zatwierdza wyniki z liczbami całkowitymi. Chociaż na przykładzie powyższym sama lokalizacja pola 49.5,-26 działa bez zarzutu a jedynie nie chce ładować się infowindow:

[PHP] pobierz, plaintext 
// For rounding land coords
  function round_down(n) {
    if (n > 0) {
      return Math.ceil(n / land_size) * land_size;
    } else if (n < 0) {
      return Math.ceil(n / land_size) * land_size;
    } else {
      return 0;
    }
  }
 
  // Uppercase words
  function ucwords(str) {
    return (str + '').replace(/^([a-z])|\s+([a-z])/g, function($1) {
      return $1.toUpperCase();
    });
  }
 
  // For number formatting
  function number_format(nStr) {
    if (!nStr) {
      return 0;
    }
    nStr += '';
    x = nStr.split('.');
    x1 = x[0];
    x2 = x.length > 1 ? '.' + x[1] : '';
    var rgx = /(\d+)(\d{3})/;
    while (rgx.test(x1)) {
      x1 = x1.replace(rgx, '$1' + ',' + '$2');
    }
    return x1 + x2;
  }
[PHP] pobierz, plaintext 

infowindow :

[PHP] pobierz, plaintext 
// Set land window
  function set_window(event) {
    // Set Parameters
    // Not sure why subtracting land_size on lat makes this work, but results in correct behavior
    var lat = round_down(event.latLng.lat()) - land_size;
    var lng = round_down(event.latLng.lng());
    var coord_slug = lat + ',' + lng;
[PHP] pobierz, plaintext 

Ten post edytował warland 30.01.2018, 23:54:16

[SmokAnalog]

Nie za bardzo mam ochotę analizować taki kod spaghetti, ale sprawdź po prostu czy nie robisz czegoś dziwnego po drodze z tą liczbą. Wypisuj sobie dla testu wartości w różnych miejscach kodu i zobacz co jest nie tak.

[warland]

tak też robię ale na razie bez rezultatu. Dziękuję za próbę pomocy.

[SmokAnalog]

A czy problemem nie jest przypadkiem to, że po prostu operujesz na nieistniejącej krainie?

[PHP] pobierz, plaintext 
$land_square = $this->game_model->get_single_land($world_key, $coord_slug);
[PHP] pobierz, plaintext 

Co to zwraca, kiedy kraina nie zostaje znaleziona?

Włącz sobie wyświetlanie wszystkich błędów, łącznie z notice. To naprawdę pomaga.

[warland]

Cały problem tkwi w tym kodzie. round_down zaokrągla wszystkie wartości dziesiętne przez co pozycje z kolumny które mają taką wartość dziesiętną są zaokrąglane i pokrywają się z innymi przez co powstaje konflikt.
Przykład:
wynik w kolumnie 55 skrypt odczytuje 55 czyli poprawnie
wynik w kolumnie 55.4 skrypt odczytuje też jako 55 czyli nie poprawnie i wywołuje konflikt.

Jak to teraz zapisać by zaokrąglało do 0.5 (lub jednym miejscu po przecinku)? w // For rounding land coords i wczytywało do // Set land window ?
Proszę o pomoc (próbowałem na wiele sposobów ale mi nie idzie za nic). Dotyczy to liczb dodatnich i ujemnych
Czy na to wszystko może też mieć wpływ kod w // For number formatting iż skrypt nie wyświetla pozycji które są z liczbą dziesiętną?

[PHP] pobierz, plaintext 
// For rounding land coords
  function round_down(n) {
    if (n > 0) {
      return Math.ceil(n / land_size) * land_size;
    } else if (n < 0) {
      return Math.ceil(n / land_size) * land_size;
    } else {
      return 0;
    }
  }
 
  // Uppercase words
  function ucwords(str) {
    return (str + '').replace(/^([a-z])|\s+([a-z])/g, function($1) {
      return $1.toUpperCase();
    });
  }
 
  // For number formatting
  function number_format(nStr) {
    if (!nStr) {
      return 0;
    }
    nStr += '';
    x = nStr.split('.');
    x1 = x[0];
    x2 = x.length > 1 ? '.' + x[1] : '';
    var rgx = /(\d+)(\d{3})/;
    while (rgx.test(x1)) {
      x1 = x1.replace(rgx, '$1' + ',' + '$2');
    }
    return x1 + x2;
  }
 
 
  // Set land window
  function set_window(event) {
    // Set Parameters
    // Not sure why subtracting land_size on lat makes this work, but results in correct behavior
    var lat = round_down(event.latLng.lat()) - land_size;
    var lng = round_down(event.latLng.lng());
    var coord_slug = lat + ',' + lng;
[PHP] pobierz, plaintext 

Ten post edytował warland 31.01.2018, 09:57:19

[nospor]

Porzadkuj kod.
Przeciez te dwa bloki IF/ELSE
if (n > 0) {
return Math.ceil(n / land_size) * land_size;
} else if (n < 0) {
return Math.ceil(n / land_size) * land_size;
}

Robia dokladnie to samo.

Co do zaokraglania to
https://stackoverflow.com/questions/6137986...-to-nearest-0-5

[warland]

bez rezultatu
możliwe że nie potrafię tego odpowiednio podmienić w kodzie.
Wszelkie próby kończą się tym że set_window się nie wyświetla dla pola z liczbą dziesiętną

lat-szerokość geograficzna
lng-długość geograficzna
gdy lat i lng ma np. 55.0 i 48.0 to set_window wyświetla się dla 55 i 48 bez liczby po przecinku
gdy lat i lng ma np. 55.5 i 48.5 to set_window nie wyświetla się dla tego pola ponieważ te kordy odczytywane są jako 55 i 48 czyli duplikują się z tymi z powyższego pola.

za nic nie wiem czemu nie chce czytać z bazy lat i lng jako wartości dziesiętne.
Pewnie bez kogoś kto nie przerobi mi tego kodu ostatniego co podałem nie poradzę sobie (IMG:style_emoticons/default/sad.gif)

[nospor]

Ty mi nie pisz, ze bez rezultatu, tylko pokaz kod na jaki zmieniles. Skad mam wiedziec co masz nadal zle.
I generalnei wez sie troche do roboty, opisuj problem dokladniej, sprawdzaj jakie wartosci sa na kazdym etapie, np tu:

var coord_slug = lat + ',' + lng;
Masz zrobic:
var coord_slug = lat + ',' + lng;
alert(coord_slug);

by zobaczyc co dokladnie ci sie generuje

[warland]

Udało się. Dziękuję wszystkim na naprowadzenie mnie na właściwą ścieżkę która rozwiązała problem.