Diskuze – Lekce 10 - Dokončení objektového diáře v JavaScriptu
ZpětUpozorňujeme, že diskuze pod našimi online kurzy jsou nemoderované a primárně slouží k získávání zpětné vazby pro budoucí vylepšení kurzů. Pro studenty našich rekvalifikačních kurzů nabízíme možnost přímého kontaktu s lektory a studijním referentem pro osobní konzultace a podporu v rámci jejich studia. Toto je exkluzivní služba, která zajišťuje kvalitní a cílenou pomoc v případě jakýchkoli dotazů nebo projektů.


Martin Ko:16.3.2019 9:05
mě nepomohlo ani opravit podle tebe . button smazat záznam nereaguje ikdyž jsem zkoušel localStorage.removeItem()
Tadeáš:3.4.2019 19:08
Mohl by jsi říct proč je to potřeba a nefunguje to dle tutoriálu?
Přesně důvod popsat nedokáži. Předpokládám, že to bude souviset s přepisováním proměnných. Každému tlačítku se přiřazuje funkce. S každým cyklem se přiřadí jen k poslednímu. Možná pro pochopení může pomoci tento krátký skript.
for (var i=0; i<5; i++) {
var btn = document.createElement('button');
btn.innerHTML = i;
btn.onclick = function() {alert(i)}
document.body.appendChild(btn);
}
var br = document.createElement("br");
document.body.appendChild(br);
for (var i=0; i<5; i++) {
var btn = document.createElement('button');
btn.innerHTML = i;
const j = i;
btn.onclick = function() {alert(j)}
document.body.appendChild(btn);
}
První cyklus vypíše 5 tlačítek a po stisknutí se objeví varovné okno s indexem i. V tomto případě u každého tlačítka je hodnota 5. V druhém cyklu je index i převeden na konstantu j, která je neměnná a nelze ji dále přepisovat jinými hodnotami. V tomto případě každé tlačítko vrátí svůj příslušný index. Tak jak se očekává.
x.listo:3.4.2019 21:01
Chyba může být kdekoliv, možná i jen v překlepu. Můžu ti pomoci jedině tak, že si svůj kód porovnáš s mým.
class Diar {
constructor(jazyk = "cs-CZ") {
const zaznamyZeStorage = localStorage.getItem("zaznamy");
this.zaznamy = zaznamyZeStorage ? JSON.parse(zaznamyZeStorage) : [];
this.jazyk = jazyk;
this.dt = null;
this.nazevInput = document.getElementById("nazev");
this.datumInput = document.getElementById("datum");
this.potvrditButton = document.getElementById("potvrdit");
this.vypisElement = document.getElementById("seznam-ukolu");
this._nastavUdalosti();
}
_nastavUdalosti() {
this.potvrditButton.onclick = () => { // this zůstane nyní stále this
if (this.datumInput.value !== "") {
const zaznam = new Zaznam(this.nazevInput.value, this.datumInput.value);
this.zaznamy.push(zaznam);
this.ulozZaznamy();
this.vypisZaznamy();
} else
alert("Musíte vyplnit datum!");
};
}
seradZaznamy() {
this.zaznamy.sort(function (zaznam1, zaznam2) {
return (new Date(zaznam1.datum) - new Date(zaznam2.datum));
});
}
vypisZaznamy() {
this.seradZaznamy();
this.vypisElement.innerHTML = "";
let posledniDatum = null;
for (const zaznam of this.zaznamy) {
if (zaznam.datum !== posledniDatum) {
const datum = new Date(zaznam.datum).toLocaleDateString(this.jazyk, {
weekday: "long",
day: "numeric",
month: "short",
year: "numeric"
});
//this.vypisElement.innerHTML += "<h3>${datum}</h3>";
this.dt = document.createElement("div");
this.dt.innerHTML = `<h3>${datum}</h3>`;
this.dt.className = "datum";
this.vypisElement.appendChild(this.dt);
}
posledniDatum = zaznam.datum;
const ukol = document.createElement("div");
ukol.innerHTML = `<br><strong>${zaznam.nazev}</strong><br>kdy: ${zaznam.datum}<br>splněno: ${!zaznam.splneno ? "ne" : ""}splněn`;
ukol.className = "ukol";
this.dt.appendChild(ukol);
//this.vypisElement.innerHTML += `<h4>${zaznam.nazev}</h4>
//<br>úkol ${!zaznam.splneno ? "ne" : ""}splněn`;
this._pridejTlacitko("Smazat", () => {
if (confirm("Opravdu si přejete odstranit úkol?")) {
this.zaznamy = this.zaznamy.filter(z => z !== zaznam); // Ponechá vše co není rovné proměnné zaznam
this.ulozZaznamy();
this.vypisZaznamy();
}
});
this._pridejTlacitko("Označit jako " + (zaznam.splneno ? "nesplněný" : "splněný"), () => {
zaznam.splneno = !zaznam.splneno;
this.ulozZaznamy();
this.vypisZaznamy();
});
//this.vypisElement.innerHtml = `<div class="ukol">` + this.vypisElement.innerHtml + `</div>`;
}
}
_pridejTlacitko(titulek, callback) {
const button = document.createElement("button");
button.onclick = callback;
button.innerText = titulek;
this.vypisElement.appendChild(button);
}
ulozZaznamy() {
localStorage.setItem("zaznamy", JSON.stringify(this.zaznamy));
}
}
cau, chyba je v samotnem pridani obsahu pomocí innerHTML do elementu, který už má obsah s eventem. Takže když se použije takto: element.innerHTML += obsah; každou další iterací se bohužel přemaže původní event.
Jinak jsem rád, že tu někdo rozvíji dál JS
cau, napadlo me pridat jeste tlacitko pro vymazani celeho diar (v pripade, ze byste meli spoustu poloezek, nebo proste chteli smaznout historii), tak jsem v html vytvoril button element navazal ho v kontruktoru a do metody nastavUdalosti, pripsal tuto metodu:
this.vycistitButton.onclick = () =>{
if (alert("Opravdu si prejete vycistit Diar?")){
localStorage.clear();
this.ulozZaznamy();
this.vypisZaznamy();
}
}
Ovsem po kliknuti nic. Alert vyskoci, ale po "ok" jsou zaznamy porad k nalezeni, nenapada nekoho proc to nefunguje? (na netu jsem hledat funkci localStorage.clear() a nemela by mit zadny parametr, takze myslim ze tam chyba neni, potom me ale nenapada proc to nefunguje.
Patrik Pastor:14.4.2019 9:46
zkusil jsem a lze to mazat z prostredka, nicmene to rozbilo tim DIVem cele css. Jde o to ze ted, pokud mas k jednomu datu vice zaznamu, pak tlacitka jednotlivych zaznamu jsou NAKONCI tohoto divu. Jinymi slovy tlacitka uz nejsou u kazde polozky jednotlive, ale nevzhledne rozhazeny v dusledku pridani divu. Jak jsi to teda vyresil s tlacitky, aby zustaly u kazde polozky?
x.listo:14.4.2019 14:03
Výborně.
To jsem přehlédl. Řešení může být přidání dalšího parametru do
_pridejTlacitko, který bude reprezentovat konkrétní úkol.
_pridejTlacitko(titulek, callback, ukol) {
const button = document.createElement("button");
button.onclick = callback;
button.innerText = titulek;
ukol.appendChild(button);
}
Potom už stačí jen doplnit požadovaný element jako parametr do obou volání.
this._pridejTlacitko("Smazat", () => {
if (confirm("Opravdu si přejete odstranit úkol?")) {
this.zaznamy = this.zaznamy.filter(z => z !== zaznam); // Ponechá vše co není rovné proměnné zaznam
this.ulozZaznamy();
this.vypisZaznamy();
}
}, ukol);
this._pridejTlacitko("Označit jako " + (zaznam.splneno ? "nesplněný" : "splněný"), () => {
zaznam.splneno = !zaznam.splneno;
this.ulozZaznamy();
this.vypisZaznamy();
}, ukol);
Patrik Pastor:14.4.2019 17:43
Diky, ted uz je vsechno vporadku (skoda ze sem na to sam neprisel, ale tak delam to poprve), proto jsem si chtel pridat prave tlacitko pro odstraneni celeho zaznamu (pri vytvareni jsem omylem naklikal jeden zaznam hodnekrat, a kdyz to konecne najelo, mel jsem tam ukoly uplne vsechny). Metoda pro tlacitko byla "localStorage.clear()" - jak jsem na sel na stackoverflow, ale stejne mi nejede, nevis proc?
_nastavUdalosti(){
// => znaci zmenu klicoveho slova this na puvodni kontext (tedy "this" plati
pro klasu "Diar")
this.potvrditButton.onclick = () => {
if (this.datumInput.value !== ''){
const zaznam = new Zaznam(this.nazevInput.value,
this.datumInput.value);
this.zaznamy.push(zaznam);
this.ulozZaznamy();
this.vypisZaznamy();
} else {
alert('Musite vyplnit datum');
}
};
// TATO METODA MA VYCISTIT CELY DIAR
this.vycistitButton.onclick = () =>{
if (alert("Opravdu si prejete vycistit Diar?")){
window.localStorage.clear();
this.ulozZaznamy();
this.vypisZaznamy();
}
}
}
vycistitButton je navazane tacitko, ktere jsem v html napsal pod
potvrdit:
<div>
<button id='potvrdit'>Uložit úkol</button>
<button id='vycistit'>Vycistit diar</button>
</div>
Zobrazeno 10 zpráv z 29.