nie ma polskich znaków (jak będziesz dorabiał, to przy okazji warto walnąć <!DOCTYPE html>
na początku oraz <meta charset="utf-8">
)
Oce
- tam nie powinno być Olce
?
var photo=new Array()
var tresc=new Array()
var cena=new Array()
nie trzeba new Array, można pusty literał tablicowy:
var photo = [];
var tresc = [];
var cena = [];
cena[1] = '2000 PLN'
cena[2] = '700 PLN'
W JS tablice zaczynają się od zera, więc wypadałoby tak zrobić
cena[0] = '2000 PLN';
cena[1] = '700 PLN';
albo z użyciem literałów tablicowych:
var cena = ['2000 PLN', '700 PLN'];
<td valign=top>
valign
jest już przestarzały, możesz użyć vertical-align
z CSS:
https://rocketvalidator.com/html-validation/the-valign-attribute-on-the-td-element-is-obsolete-use-css-instead
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/td#attr-valign
Poza tym tabelki są do robienia tabelek, a nie do layoutu, tak się robiło z kilkanaście lat temu. Możesz się zainteresować pozycjonowaniem elementów w CSS np. za pomocą flexa https://css-tricks.com/snippets/css/a-guide-to-flexbox/
No i możesz użyć const
i let
zamiast var
. W tym przypadku to nie ma większego znaczenia, ale jak będziesz robił coś większego, to będzie mieć:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/const
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/let
No i jak klikam kilka razy Next to oczekiwałbym, że powróci do początku, a nie powraca.
Ja dodam, że id elementów typu ppp
albo ddd
są beznadziejne. O czym to mówi komuś kto sprawdza ten kod?
Hej, też kiedyś byłem na entry levelu i takie zadanko to był dla mnie killer.
Niestety nie widzę nigdzie treści zadania, więc mogę się domyślać że chodziło o stworzenie "galerii" i wyświetlenie ceny, waluty oraz krótkiego opisu.
Zrobiłem szybki refaktor, rzuć okiem jak to powinno wyglądać (chociaż pewnie znalazłoby się w moim kodzie wiele rzeczy do usprawnienia).
Jakbyś miał pytania jak co działa i dlaczego to dawaj znać.
Tylko prośba - nie przeklejaj kodu i nie oddawaj go na zaliczenie jako swojego, nie w tym rzecz.
https://jsfiddle.net/u7z0m8hx/1/
Pozdro