Przeglądarka zdjęć w JavaScript na zaliczenie

1

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.

2

Ja dodam, że id elementów typu ppp albo ddd są beznadziejne. O czym to mówi komuś kto sprawdza ten kod?

0

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

1 użytkowników online, w tym zalogowanych: 0, gości: 1