innerHTML i warunki

pytanie zadane 28 lutego 2018 w JavaScript przez drrak Nowicjusz (140 p.)

Witam, próbuję zrobić grę tekstową i zastanawiam się czy można jakoś skrócić ten rozlazły kawałek skryptu:

if(x== 0 && y== 0){cs(0,0,1,0);
if(u1[4]==0 && u1[3]==0){txt.innerHTML += '<p>Jesteś w ciemnym pokoju. Rozglądasz się uważnie i zwracasz uwagę na spinacz i kubek.</p>';}
else if(u1[4]==0 && u1[3]==1){txt.innerHTML += '<p>Jesteś w ciemnym pokoju. Rozglądasz się uważnie i zwracasz uwagę na kubek.</p>';}
else if(u1[4]==1 && u1[3]==0){txt.innerHTML += '<p>Jesteś w ciemnym pokoju. Rozglądasz się uważnie i zwracasz uwagę na spinacz.</p>';}
else{txt.innerHTML += '<p>Jesteś w ciemnym pokoju. Rozglądasz się uważnie, ale nie widzisz nic ciekawego.</p>';}
}

Dziękuję za pomoc.

PS. Na IE11 nie można wybrać kategorii przy zadawaniu pytania/zakładaniu nowego tematu. Musiałem skorzystać z Edge.

komentarz 28 lutego 2018 przez Jedras Maniak (54,860 p.)

To kod pisany dla samego siebie? Jeśli masz zamiar to gdzieś pokazać światu to lepiej zrób porządek z tymi tablicami w if'ach. Chyba, że gdzieś to sensownie udokumentujesz, ale nadal to dość marny pomysł.

komentarz 28 lutego 2018 przez drrak Nowicjusz (140 p.)

Kod jest tylko dla mnie, co nie znaczy, że ma być zły. Nie bardzo rozumiem co innego mógłbym zrobić z tymi tablicami. Jestem tylko amatorem javasrcript i nieco zardzewiałem. Chętnie poprawie mój kod, tylko proszę dokładnie mi wyjaśnić co i gdzie można ulepszyć.

1 odpowiedź

odpowiedź 28 lutego 2018 przez Scypyon Gaduła (3,450 p.)
edycja 28 lutego 2018 przez Scypyon

const txt = "Rozglądasz się uważnie i zwracasz uwagę";

const txt1 = "Jesteś w ciemnym pokoju"

if(x== 0 && y== 0){cs(0,0,1,0);
if(u1[4]==0 && u1[3]==0){txt.innerHTML +=  txt1+" "+ txt+" na spinacz i kubek.";}
else if(u1[4]==0 && u1[3]==1){txt.innerHTML += txt1+" "+ txt+" na kubek.";}
else if(u1[4]==1 && u1[3]==0){txt.innerHTML += txt1+" "+ txt+" na spinacz.";}
else{txt.innerHTML += txt1+" " + "Rozglądasz się uważnie, ale nie widzisz nic ciekawego.";}
}

O takie coś ci chodziło? :) ps. pisane na szybko.

komentarz 28 lutego 2018 przez drrak Nowicjusz (140 p.)

Dziękuję za zainteresowanie i odpowiedź. Niewątpliwie jest krócej, tylko nie jestem pewien czy na dłuższą metę to dobre/czytelne rozwiązanie. W tym jednym przypadku ma to sens, ale może zrobić się niezły bałagan, gdy pokoi będzie więcej i każdy będzie mieć inny opis.

komentarz 28 lutego 2018 przez Scypyon Gaduła (3,450 p.)

Oczywiście ,że ma to sens, zamiast pisać ten sam tekst 4 razy piszesz go raz, jeżeli pokazałbyś mi całość, to mógłbym ci bardziej wszystko zoptymalizoać, m.in tablice(spratański sposób) ale to już według własnego uznania, jeżeli pomogłem to liczę na naj :)

komentarz 1 marca 2018 przez drrak Nowicjusz (140 p.)

Nie wiem jak zrobić to samo bez tablic. Tablice są mi potrzebne do takiego jakby zapisu stanu gry, a dokładniej wygenerowania odnośnika m.in. z przedmiotami i współrzędnymi.

komentarz 3 marca 2018 przez Scypyon Gaduła (3,450 p.)

wiesz, żeby ci pomóc, musiałbym zobaczyć całość

komentarz 7 marca 2018 przez drrak Nowicjusz (140 p.)

Wiem, wiem, ale musiałbym przygotować fragment skryptu i zrobiłby się mały offtop. Znaczy fragment do tablic, bo do głównego pytania nie mam co dopisywać. Poza tym chwilowo straciłem wenę twórczą :)