[PHP]+[JS] Zapytanie SQL w JS, Ciągłe wywoływanie ostatniego zapytania... Opcje

[peja1990]

Witam, piszę skrypt, który ma odpowiadać za zmianę języka witryny dla danego użytkownika serwisu (oparcie o adres IP dla gości) i mam pewien problem, gdyż mam do wyboru 3 języki (PL,EN,DE).
Chcę aby po kliknięciu w jedną z flag wysunęło się okienko w dolnym rogu witryny (pozycja fixed), gdzie pojawia się tekst o wyborze języka, informacja o ładowaniu języka itp.
To zrobiłem i działa bez zarzutu ale problem tkwi w tym, że do każdego skryptu dodałem zapytanie SQL, które ma update'ować rekord danego użytkownika np. gdy jego aktualny język to "pl", a kliknie w flagę angielską to zapytanie ma z'update'ować rekord dla jego IP w komórce "lang" z "pl->en".

Mój kod wygląda tak:

[PHP] pobierz, plaintext 
<script type="text/javascript">
// Język niemiecki
$(document).ready(
function() {
  $(".lang_de").click(
  function() {
    $(".popup").slideDown();
    setInterval(function() {
      $(".popup").slideUp();
    }, 6000);
    var sql = ['<?php mysql_query('UPDATE guests SET lang="de" WHERE ip="'.$_SERVER['REMOTE_ADDR'].'"') or die (mysql_error()); ?>'];
    var contents = ['<?php echo $LANG['popup']['lang']['load']; ?>', '<?php echo $LANG['popup']['lang']['change']['de']; ?>'];
    var current = 0;
    setInterval(function() {
      document.getElementById('load_content').innerHTML = contents[current++];
      document.getElementById('load_sql').innerHTML = sql;
    }, 2000);
    setInterval(function() {
      parent.location='<?php $PHP_SELF; ?>';
    }, 8000);
  });
});
// Język angielski
$(document).ready(
function() {
  $(".lang_en").click(
  function() {
    $(".popup").slideDown();
    setInterval(function() {
      $(".popup").slideUp();
    }, 6000);
    var sql = ['<?php mysql_query('UPDATE guests SET lang="en" WHERE ip="'.$_SERVER['REMOTE_ADDR'].'"') or die (mysql_error()); ?>'];
    var contents = ['<?php echo $LANG['popup']['lang']['load']; ?>', '<?php echo $LANG['popup']['lang']['change']['en']; ?>'];
    var current = 0;
    setInterval(function() {
      document.getElementById('load_content').innerHTML = contents[current++];
      document.getElementById('load_sql').innerHTML = sql;
    }, 2000);
    setInterval(function() {
      parent.location='<?php $PHP_SELF; ?>';
    }, 8000);
  });
});
// Język polski
$(document).ready(
function() {
  $(".lang_pl").click(
  function() {
    $(".popup").slideDown();
    setInterval(function() {
      $(".popup").slideUp();
    }, 6000);
    var sql = ['<?php mysql_query('UPDATE guests SET lang="pl" WHERE ip="'.$_SERVER['REMOTE_ADDR'].'"') or die (mysql_error()); ?>'];
    var contents = ['<?php echo $LANG['popup']['lang']['load']; ?>', '<?php echo $LANG['popup']['lang']['change']['pl']; ?>'];
    var current = 0;
    setInterval(function() {
      document.getElementById('load_content').innerHTML = contents[current++];
      document.getElementById('load_sql').innerHTML = sql;
    }, 2000);
    setInterval(function() {
      parent.location='<?php $PHP_SELF; ?>';
    }, 8000);
  });
});
</script>
<div class="top">
  <span class="line"></span>
  <div class="center">
    <div class="box">
      <ul class="left">
        <li class="log_webmail"><?php echo $LANG['top']['log_webmail']; ?></li>
        <li class="log_system"><?php echo $LANG['top']['log_system']; ?></li>
      </ul>
      <ul class="right">
        <li class="lang_de <?php if($GUEST['lang']=='de') {echo 'active';} ?>"></li>
        <li class="lang_en <?php if($GUEST['lang']=='en') {echo 'active';} ?>"></li>
        <li class="lang_pl <?php if($GUEST['lang']=='pl') {echo 'active';} ?>"></li>
        <li><?php echo $LANG['top']['change_lang']; ?></li>
      </ul>
    </div>
  </div>
</div>
 
<div class="popup"><div class="bg"><?php echo $LANG['popup']['lang']['select'].$br.'<div id="load_content"></div>'.'<div id="load_sql"></div>'; ?></div></div>
[PHP] pobierz, plaintext 

Wszystko działa ładnie, lecz nie ważne którą flagę kliknę, zawsze wywoływane jest ostatnie zapytanie mySQL, w tym przypadku dla języka PL.
Dla przykładu: mój aktualny język to EN, klikam w DE, a skrypt i tak wywołuje zapytanie dla PL i zmienia mi język na PL a nie na DE.

Próbowałem to robić na wiele sposobów, lecz jeszcze nie udało mi się znaleźć odpowiedniego, dlatego piszę na forum mając nadzieję, że ktoś z Was mi pomoże

Za pomoc będę niezmiernie wdzięczny.
Za każdą odpowiedź daję "POMÓGŁ" !

Serdecznie pozdrawiam.
Kamil Dunaj.

[phpion]

Za każdą odpowiedź daję "POMÓGŁ" !

Uważaj lepiej z tym - łatwo możesz zarobić warna za takie praktyki. Oznaczenie "Pomógł" służy do oznaczania faktycznie pomocnych odpowiedzi, a nie każdych odpowiedzi.

[peja1990]

Dobrze mistrzu
A jesteś w stanie mi pomóc ?

[nospor]

Ogólnie zachęcanie POMOGŁ jest żałosne.

Co do tematu:
1) Tworzysz 3 razy zmienną sql to pewnie ci się za każdym razem nadpisuje
2) Tworzysz w kodzie html zmienną sql, którą potem na serwerze wykonujesz... ałć... bez problemu mogę wykonać dowolne zapytanie na Twojej bazie

Do serwera powinieneś przekazywać tylko aktualnie kliknięty język. To wszystko. Zapytanie sql masz budować już po stronie serwera przed samym wykonaniem.

3) Bez sensu tworzysz x razy ten sam kod. Napisz jedną funkcję, która przyjmie parametry język i po sprawie.

[peja1990]

Ogólnie zachęcanie POMOGŁ jest żałosne.

Co do tematu:
1) Tworzysz 3 razy zmienną sql to pewnie ci się za każdym razem nadpisuje
2) Tworzysz w kodzie html zmienną sql, którą potem na serwerze wykonujesz... ałć... bez problemu mogę wykonać dowolne zapytanie na Twojej bazie

Do serwera powinieneś przekazywać tylko aktualnie kliknięty język. To wszystko. Zapytanie sql masz budować już po stronie serwera przed samym wykonaniem.

3) Bez sensu tworzysz x razy ten sam kod. Napisz jedną funkcję, która przyjmie parametry język i po sprawie.

Dzięki, a jesteś w stanie przedstawić swoją propozycję tego skryptu ?

[sunpietro]

Od siebie tylko dodam, że mieszanie kodu JS i PHP jest złą praktyką. Powinieneś mieć osobne pliki z obsługą JS i PHP.

[peja1990]

Super, a potrafisz to zrobić tak jak należy i żeby działało tak jak to opisałem ?

Może napisze ktoś przynajmniej jak zrobić z tego jedną funkcję i jak zainicjować wartość ID dla kodu php w JS.

Np: var sql = ['<?php mysql_query('UPDATE guests SET lang="TUTAJ WARTOŚĆ ID" WHERE ip="'.$_SERVER['REMOTE_ADDR'].'"') or die (mysql_error()); ?>'];

Ten post edytował peja1990 22.09.2011, 14:17:58

[Kużdo]

Nadal nie zrozumiałeś. Po stronie przeglądarki nie powinno być żadnych zapytań mysql, kodów PHP itp, bo jest to niebezpieczne dla Twojego skryptu jak i całego serwera. Przez JS masz przesłać tylko i wyłącznie zmienną która informuje jakąś flagę kliknął user. Nic więcej. Reszta ma odbywać się po stronie serwera, tam gdzie nie ma nikt inny dostępu oprócz admina. Dodatkowo powinieneś filtrować tę zmienną czy aby na pewno zawiera to co planowałeś by zawierała. U ciebie są to dwuliterowe oznaczenia języków, więc możesz stworzyć tablicę zawierającą wszystkie dostępne języki i sprawdzać, czy przesłana zmienna znajduje się w tej tablicy, jeśli tak, to wszystko jest ok.

[kpt_lucek]

Ja osobiście zrobiłbym to tak:

[HTML] pobierz, plaintext 
<ul class="right">
 <li class="lang_de <?php if($GUEST['lang']=='de') {echo 'active';} ?>" title="flaga"></li>
 <li class="lang_en <?php if($GUEST['lang']=='en') {echo 'active';} ?>" title="flaga"></li>
 <li class="lang_pl <?php if($GUEST['lang']=='pl') {echo 'active';} ?>" title="flaga"></li>
 <li><?php echo $LANG['top']['change_lang']; ?></li>
</ul>
[HTML] pobierz, plaintext 

JQuery:

[HTML] pobierz, plaintext 
$('li').click(function(){
var a = $(this).attr("class");
var b = $(this).attr("title");
if(b=="flaga")
{
 $.ajax({
	type: "POST",
	url: "zmiana_flag.php",
	data: "a=" + a,
	success: function(odpowiedz){ 
		alert(odpowiedz);
	},
	error: function(){
		alert("Błąd!");
	}
 });
}
});
[HTML] pobierz, plaintext 

zmiana_flag.php:

[PHP] pobierz, plaintext 
<?
require("dane_do_bazy_danych.php");
//łączenie do mysql...
$flaga = $_POST[a];
$sql=mysql_query(...) or die(mysql_error());
 
if($sql){
 echo "ok";
}
// else nie potrzebny, w przypadku błędu zwróci go mysql_error()
?>
[PHP] pobierz, plaintext 

Ten post edytował kpt_lucek 22.09.2011, 19:51:10

[Kużdo]

Po prywatnej korespondencji z peja1990 wiem, że tam wcale nie trzeba żadnego AJAXa. Po kliknięciu w flagę strona po kilku sek odświeżała się, więc zmianę języka można obsłużyć przez zwykłego GETa/POSTa. Nie wiem nawet czy kod, który przedstawiłem pomoże temu userowi, bo sprawdzając jego stronę już na starcie raziły błędy HTML m.in. cała sekcja head znajdowała się body i to nie tak, że <head>(...)</head> było w body, tylko head było puste, a to co miało tam być, było w body...

[kpt_lucek]

Po prywatnej korespondencji z peja1990 wiem, że tam wcale nie trzeba żadnego AJAXa. Po kliknięciu w flagę strona po kilku sek odświeżała się, więc zmianę języka można obsłużyć przez zwykłego GETa/POSTa. Nie wiem nawet czy kod, który przedstawiłem pomoże temu userowi, bo sprawdzając jego stronę już na starcie raziły błędy HTML m.in. cała sekcja head znajdowała się body i to nie tak, że <head>(...)</head> było w body, tylko head było puste, a to co miało tam być, było w body...

Ok, ale sposób przedstawiony przeze mnie zajmuje zdecydowanie mniej miejsca i komunikacja między MySQL odbywa się za pomocą odrębnego pliku php.

[peja1990]

Ja osobiście zrobiłbym to tak:

[HTML] pobierz, plaintext 
<ul class="right">
 <li class="lang_de <?php if($GUEST['lang']=='de') {echo 'active';} ?>" title="flaga"></li>
 <li class="lang_en <?php if($GUEST['lang']=='en') {echo 'active';} ?>" title="flaga"></li>
 <li class="lang_pl <?php if($GUEST['lang']=='pl') {echo 'active';} ?>" title="flaga"></li>
 <li><?php echo $LANG['top']['change_lang']; ?></li>
</ul>
[HTML] pobierz, plaintext 

JQuery:

[HTML] pobierz, plaintext 
$('li').click(function(){
var a = $(this).attr("class");
var b = $(this).attr("title");
if(b=="flaga")
{
 $.ajax({
	type: "POST",
	url: "zmiana_flag.php",
	data: "a=" + a,
	success: function(odpowiedz){ 
		alert(odpowiedz);
	},
	error: function(){
		alert("Błąd!");
	}
 });
}
});
[HTML] pobierz, plaintext 

zmiana_flag.php:

[PHP] pobierz, plaintext 
<?
require("dane_do_bazy_danych.php");
//łączenie do mysql...
$flaga = $_POST[a];
$sql=mysql_query(...) or die(mysql_error());
 
if($sql){
 echo "ok";
}
// else nie potrzebny, w przypadku błędu zwróci go mysql_error()
?>
[PHP] pobierz, plaintext 

Super mistrzu, lecz po kliknięciu w flagę teraz nic się nie dzieje, nie chodzi o animację bo to inna sprawa lecz, nie aktualizuje się rekord co jest tutaj najważniejsze.

[nospor]

Poprawka:

<li lang="de" class="lang_de <?php if($GUEST['lang']=='de') {echo 'active';} ?>" title="flaga"></li>
analogicznie reszta

var a = $(this).attr("lang");

No i tu:
$sql=mysql_query(...) or die(mysql_error());
wstawiłeś zapytanie?

[peja1990]

Poprawka:

<li lang="de" class="lang_de <?php if($GUEST['lang']=='de') {echo 'active';} ?>" title="flaga"></li>
analogicznie reszta

var a = $(this).attr("lang");

No i tu:
$sql=mysql_query(...) or die(mysql_error());
wstawiłeś zapytanie?

To już sobie poprawiłem lecz zamiast atrybutu lang dałem id, tylko nie to że nie podmienia na poste pole czy coś tylko w ogóle nie podmienia.
A zapytanie oczywiście, że dałem
Gdzie może być błąd ?

Do tego w pliku PHP zmieniłem z $_POST[a] na $_POST['a'] i nic

Ten post edytował peja1990 22.09.2011, 20:38:13

[nospor]

Odpal firebuga dla FIrefox i zobacz, czy ci ten ajax idzie, co zawiera itp.

[peja1990]

Odpal firebuga dla FIrefox i zobacz, czy ci ten ajax idzie, co zawiera itp.

Ok, ale najpierw muszę zainstalować "lisa" bo ogólnie to nie korzystam z tej przeglądarki

[nospor]

Naprawdę nie musisz mnie/nas informować o tym. Poprostu zainstaluj i powiedz co ci pokaże.

[peja1990]

Naprawdę nie musisz mnie/nas informować o tym. Poprostu zainstaluj i powiedz co ci pokaże.

Nic mi nie pokazuje, tak jakby to był zwykły obrazek z hoverem... Poprostu jedno wielkie NIC ;/

Zaraz Ci pokażę to Ty też sprawdzisz, ok ?

Łapcie, sprawdzajcie: http://detronic.pl/test/pl/start.html

Ten post edytował peja1990 22.09.2011, 20:54:35

[nospor]

TEn kod:
$('li').click(function(){ ....
.....
ma być odpalony po załadowaniu strony, czyli
$(document).ready( .........
...........
co już stosowałeś wcześniej. Myśl trochę.

[Kużdo]

Nadal jestem zdania, że jest to zbędne, ponieważ po kliknięciu w flagę po kilku sekundach strona wczytuje się ponownie, więc znacznie prościej jest to zrobić używając parametru w GETcie, do tego odpada dodatkowy JS, który w tym przypadku jest naprawdę zbędny...

[peja1990]

TEn kod:
$('li').click(function(){ ....
.....
ma być odpalony po załadowaniu strony, czyli
$(document).ready( .........
...........
co już stosowałeś wcześniej. Myśl trochę.

Masz rację...
Teraz jest reakcja na click ale aktualizuje rekord na PUSTY ;/
Dodam, że tutaj zmieniłem na: url: "http://detronic.pl/test/zmiana_flag.php", z pewnych względów, wiecie może dlaczego nie aktualizuje poprawnie rekordu ?

Podgląd: http://detronic.pl/test/pl/start.html

Nadal jestem zdania, że jest to zbędne, ponieważ po kliknięciu w flagę po kilku sekundach strona wczytuje się ponownie, więc znacznie prościej jest to zrobić używając parametru w GETcie, do tego odpada dodatkowy JS, który w tym przypadku jest naprawdę zbędny...

Kolego zobacz na to ".../pl/start.html" <- rozszerzenie nie pozwala na użycie metody GET, a nie zmienię na php



DZIAŁA !!!!
DZIĘKI ZA POMOC !!!!!!!!!!!!!

Ten post edytował peja1990 22.09.2011, 21:12:34

[Kużdo]

Kolego zobacz na to ".../pl/start.html" <- rozszerzenie nie pozwala na użycie metody GET, a nie zmienię na php

Kolega nie zna nawet HTMLa, więc sorry, ale postarałbyś się chociaż przeczytać te linki, które Ci podesłałem...

To, że coś ma takie rozszerzenie to nie znaczy, że nie da się już tego wykonać. Skoro jest to HTML, to zmianę języka można obsłużyć w zewnętrznym pliku PHP, który na koniec przekieruje Cię znów na start.html, a jak dla mnie, to ta Twoja strona nadal nie działa, wyskakuje okienko, ale przekierowania późniejszego zero, więc nie wiem jaki pożytek z tego co zrobiłeś... Chyba, że jakoś to przekształcisz... Tak czy siak spójrz na to z innej strony. Teraz masz HTML + JS + zewnętrzny plik PHP do zmiany języka, a tak miałbyś tylko HTML + PHP. Ale jak kto woli...

Linki w postaci:

[HTML] pobierz, plaintext 
<a href="http://adres.do/pliku.php?lang=pl"><img src="adres/do/obrazka_pl.png" /></a>
<a href="http://adres.do/pliku.php?lang=en"><img src="adres/do/obrazka_en.png" /></a>
<a href="http://adres.do/pliku.php?lang=de"><img src="adres/do/obrazka_de.png" /></a>
[HTML] pobierz, plaintext 

Plik php:

[PHP] pobierz, plaintext 
<?php
//polaczenie z baza
$dostepne_jezyki = array('pl', 'en', 'de');
 
if (isset ($_GET['lang']) && in_array($_GET['lang'], $dostepne_jezyki))
{
    $query = "UPDATE guests SET lang = '{$_GET['lang']}' WHERE ip = '{$_SERVER['REMOTE_ADDR']}'";
    $result = mysql_query($query);
    header('Location: hxxp://adres.do.strony/test'.$_GET['lang'].'/start.html');
}
?>
[PHP] pobierz, plaintext 

Musiałem w headerze dać hxxp zamiast http, bo forum dziwnie to konwertowało.

Te dwa fragmenty zaktualizują Twój język w bazie + przekierują Cię na odpowiednie linki:

Kod

http://adres.do.strony/pl/start.html
http://adres.do.strony/en/start.html
http://adres.do.strony/de/start.html

Ten post edytował Kużdo 22.09.2011, 21:31:44

[peja1990]

Kolega nie zna nawet HTMLa, więc sorry, ale postarałbyś się chociaż przeczytać te linki, które Ci podesłałem...

To, że coś ma takie rozszerzenie to nie znaczy, że nie da się już tego wykonać. Skoro jest to HTML, to zmianę języka można obsłużyć w zewnętrznym pliku PHP, który na koniec przekieruje Cię znów na start.html, a jak dla mnie, to ta Twoja strona nadal nie działa, wyskakuje okienko, ale przekierowania późniejszego zero, więc nie wiem jaki pożytek z tego co zrobiłeś... Chyba, że jakoś to przekształcisz... Tak czy siak spójrz na to z innej strony. Teraz masz HTML + JS + zewnętrzny plik PHP do zmiany języka, a tak miałbyś tylko HTML + PHP. Ale jak kto woli...

Linki w postaci:

[HTML] pobierz, plaintext 
<a href="http://adres.do/pliku.php?lang=pl"><img src="adres/do/obrazka_pl.png" /></a>
<a href="http://adres.do/pliku.php?lang=en"><img src="adres/do/obrazka_en.png" /></a>
<a href="http://adres.do/pliku.php?lang=de"><img src="adres/do/obrazka_de.png" /></a>
[HTML] pobierz, plaintext 

Plik php:

[PHP] pobierz, plaintext 
<?php
//polaczenie z baza
$dostepne_jezyki = array('pl', 'en', 'de');
 
if (isset ($_GET['lang']) && in_array($_GET['lang'], $dostepne_jezyki))
{
    $query = "UPDATE guests SET lang = '{$_GET['lang']}' WHERE ip = '{$_SERVER['REMOTE_ADDR']}'";
    $result = mysql_query($query);
    header('Location: hxxp://adres.do.strony/test'.$_GET['lang'].'/start.html');
}
?>
[PHP] pobierz, plaintext 

Musiałem w headerze dać hxxp zamiast http, bo forum dziwnie to konwertowało.

Te dwa fragmenty zaktualizują Twój język w bazie + przekierują Cię na odpowiednie linki:

Kod

http://adres.do.strony/pl/start.html
http://adres.do.strony/en/start.html
http://adres.do.strony/de/start.html

Nie mogłeś tak wcześniej ?
Czemu Ty się tak czepiłeś tego html'a ? Haha, gdzie Ty w moim kodzie widzisz coś pomieszanego w sekcji head i body ? Może zainwestuj w okularki ?
Przyjrzyj się jeszcze raz kodowi strony Pozatym to tylko szkielet.
I uwierz mi, że to co mam teraz z JS'em jest na 100% lepsze od zwykłego HTML+PHP.
Przekierowania nie potrzebuję na .../en/... itd. bo po co ? Mogę to zrobić i w aktualnym kodzie ale to bez sensu bo i tak wszystkie linki automatycznie wraz ze zmianą języka są updateowane

WIELKIE DZIĘKI WSZYSTKIM ZA POMOC Uratowaliście mnie !
Za to dostaliście kilka "pomógł", za każdą pomocną wypowiedź <- to tak nawiasem dla moderatorów

Serdecznie pozdrawiam i jeszcze raz dziękuję
Szczególnie użytkownikowi "nospor" jak i użytkownikowi "Kużdo" za poświęcenie sporo swojego czasu