Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [AJAX][JavaScript]Zwracanie wartości div'a.
Salur
post
Post #1





Grupa: Zarejestrowani
Postów: 179
Pomógł: 0
Dołączył: 28.12.2014

Ostrzeżenie: (20%)
X----


Witam, proszę o pomoc w wykonaniu czegoś takiego:

http://scr.hu/7bwz/arzux

Sa 2 divy, w jednym będą zdjecia, po kliknieciu np. na zdjecie pierwsze w drugim divie pokaże się to zdjęcie z dodatkowym opisem poniżej.

Bardzo proszę o pomoc.

Ten post edytował Salur 6.07.2015, 10:10:40
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
PrinceOfPersia
post
Post #2





Grupa: Zarejestrowani
Postów: 717
Pomógł: 120
Dołączył: 18.04.2009

Ostrzeżenie: (0%)
-----


No tak, tylko ten kod nawet jeśli działa, to jest wręcz fatalny.
[JAVASCRIPT] pobierz, plaintext
  1. document.getElementById("zdjecia").getElementsByTagName("img")[0].onclick=function()
  2. {
  3. var opis = "Opis do zdjęcia numer 1";
  4. document.getElementById("opis").innerHTML = opis;
  5. }
  6.  
  7. document.getElementById("zdjecia").getElementsByTagName("img")[1].onclick=function()
  8. {
  9. var opis = "Opis do zdjęcia numer 2";
  10. document.getElementById("opis").innerHTML = opis;
  11. }
  12.  
  13. document.getElementById("zdjecia").getElementsByTagName("img")[2].onclick=function()
  14. {
  15. var opis = "Opis do zdjęcia numer 3";
  16. document.getElementById("opis").innerHTML = opis;
  17. }
  18.  
  19. document.getElementById("zdjecia").getElementsByTagName("img")[3].onclick=function()
  20. {
  21. var opis = "Opis do zdjęcia numer 4";
  22. document.getElementById("opis").innerHTML = opis;
  23. }
[JAVASCRIPT] pobierz, plaintext

Wiesz, że napisałeś 4 razy to samo? Rozumiem, że łatwiej jest nacisnąć Ctrl+C a potem trzy razy Ctrl+V niż napisać z sensem - ale uwierz, że ten kod będzie później trudny w utrzymaniu. Za każdym razem kiedy trzeba będzie zmienić jedną rzecz, trzeba będzie ją zmienić w 4 miejscach. Jeśli będzie trzeba zmienić trzy rzeczy (albo dodać nowe 3 ficzery), to trzeba będzie pisać 12 razy (bo 3 ficzery * 4 powtórzenia) to samo. A co jesli obrazkow bedzie 50? Bedziesz 50 razy kopiowal to samo?

Pisanie takiego kodu ma uzasadnienie tylko wtedy, kiedy chcemy coś na szybko sprawdzić i na 90% nie będziemy zaglądać więcej do tego skryptu.

Jeśli Salurowi chodziło o taki "jednostrzałowiec" to okej...

Ale jak to ma być coś, co potem ma trwać i być łatwe do zmienienia to juz predzej coś takiego jak to:
http://jsfiddle.net/sa91ktsp/2/

Zresztą... jakby się zastanowić to napisanie z sensem wcale nie zajmuje więcej czasu niż kopiuj-wklejka. Nie mam pojęcia skąd ta moda na copy-pastę.

Ten post edytował PrinceOfPersia 6.07.2015, 11:41:02
Go to the top of the page
+Quote Post
kafowi
post
Post #3





Grupa: Zarejestrowani
Postów: 75
Pomógł: 20
Dołączył: 13.02.2015

Ostrzeżenie: (0%)
-----


Cytat(PrinceOfPersia @ 6.07.2015, 12:39:15 ) *
Pisanie takiego kodu ma uzasadnienie tylko wtedy, kiedy chcemy coś na szybko sprawdzić i na 90% nie będziemy zaglądać więcej do tego skryptu.
Jeśli Salurowi chodziło o taki "jednostrzałowiec" to okej...

Założyłem, że to na zaliczenie, a one rządzą się swoimi prawami, więc w sumie to twierdzenie jest trafne.

Cytat
Wiesz, że napisałeś 4 razy to samo? Rozumiem, że łatwiej jest nacisnąć Ctrl+C a potem trzy razy Ctrl+V niż napisać z sensem - ale uwierz, że ten kod będzie później trudny w utrzymaniu.

Ja rozumiem, że kod będzie trudny w utrzymaniu i napisałem to samo, odpowiedzi udzieliłem w cytacie pierwszym.
Go to the top of the page
+Quote Post

Posty w temacie


Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 9.10.2025 - 09:34