Pennergame Flaschensammelbot

[Starflow]

Dann aber ohne node.js oder sonstiger lib. - bin gespannt

Definiere bitte "lib".

Denn Node.js ist eine Plattform basierend auf Googles V8 und irgendwas muss das Script auf der CLI ja laufen lassen.

Außerdem werd ich auch so dreist sein und meine Design Entscheidungen selber treffen.

Hättest du irgendwem die Grundlagen von etwas beibringen wollen, hättest du 'ne HTTP Library schreiben können oder auch
was anderes Grundlegendes, hast du aber nicht.

Du wirst lachen aber Anfangs hatte Ich Username & Password dort stehen, habs dann aber geändert gehabt



Nächstesmal lass ichs stehen, okay?

Ja

Ich hab mein Dropbox (nur 2gb) vollgestopft bis zum Rand, das was ich noch von Reddit druf hatte war in Vergessenheit geraten - das sollte eigendlich garnicht mehr auf der Dropbox sein.
Dropbox nutze ich eigendlich nicht mehr für solche Dinge da ich mittlerweile ne eigene Domain usw habe.. welche ich hier aber sicher nicht Preisgeben werde (nein, nicht cystasy im domainname)^^

Brauchst du nicht, hab ich schon gefunden.
Ich weiß nicht was mich mehr verwirrt, der TK schwachsinn oder das du XHTML 1.0 Strict benutzt. (4 Errors, 34 warning(s))

Ich mein nur, der Code war eigentlich sogar noch "klein" genug um auch einfach hier in nem [CODE] Tag gepostet zu werden.

Touché.
Da gibts wirklich einiges wo ich noch was lernen kann, finde es toll von dir das du dir soviel Mühe gegeben hast wegen mir - Danke dir hierfür.

Kein Problem, ich versuch auch nicht ganz so grob zu sein, du bist ja guten willens.

Von Label´s z.b hatte ich bisher noch nie etwas gehört gehabt (ist das neu hinzugekommen vor einer Weile?).

Hmm, so ungefähr vor 17 Jahre gibt es die schon:

http://www.w3.org/TR/1998/REC-html40-19980424/interact/forms.html#edef-LABEL

:S

Was ich jedoch sagen muss - du hast auch vieles bemängelt das ich nicht wirklich als Fehler / Falsch ansehen würde.
Nur weil ich mich nicht 100% an die Specs halte und "unnötige" Kommentare für mich in den Code einfüge für die Übersichtlichkeit, heißt das ja nicht das es falsch ist.

Über den Nutzwert der Kommentare kann man streiten, über die Spec nicht.
Nein, man muss (und kann auch) nicht immer Valide sein, aber man sollte es nach bestem Wissen und Gewissen anstreben.

Du musst den Boilerplate Code ja nicht mal selber schreiben wenn du die richtigen Tools benutzt.

Ich verstehe das sowas von Bedeutung ist wenn man in ner Gruppe zusammen an etwas arbeitet, aber hier bin nur ich derjenige der an dem Code gearbeitet hatte.

1. Häufiger Irrglaube bei Anfängern, man schreibt Code auch ordentlich für das eigene "spätere Ich"
2. Du hast es Publik gemacht, unleserlichen Code liest aber niemand gern, weil er schwer zu verstehen ist, besonders dann wenn er so gut wie unkommentiert bleibt.

1) Doctype usw fehlt, ja. Ist ja auch nur nen Bot und keine Webseite die ich irgendwo ins Netz stellen würde so das ich dies benötigen würde.

Wie gesagt über die Spec streit ich nicht. Das macht es leserlicher (sowohl für AI Clients als auch für andere Programmierer inkl. deinem späterem Ich) und ist mit den richtigen Tools kaum ein Mehraufwand.

Klar, "gehört es normalerweise dazu", verstehe ich - trotztdem.. das ganze war als Proof of Concept angesehen, und ich habe bewusst Dinge wie Doctype, ne schöne HTML Struktur u.s.w weg gelassen gehabt.

1. Wenn man anderen etwas beibringen will, sollte man so ordentlich und korrekt wie möglich machen.
Quick 'n Dirty ist ok für's prototyping, nicht für Tutorials, durch sowas verbreiten sich nur schlechte Angewohnheiten
und es entsteht so Anspruchsloser Code, wie der den du immer bemängelst.

2. So einen Code würde ich nicht mal für 'n schnelles prototyping schreiben, ich mein Inline-Code, ernsthaft?
Sowas hab ich zuletzt Anfang des Jahrtausends gesehen und selbst da war es schon verpönt.

2) Hatte über eine Helper Function für SetState nachgedacht, habs dann aber weggelassen gehabt.
Nun hast du sie eingefügt.. auch okay^^

Naja ich fand es ziemlich deppert immer wieder das "Status:" erneut mit einzufügen und wo ich grad einmal dran war, hab ich's gleich richtig gemacht.

Ansonsten habe ich mir deine anderen Vorschläge & Verbesserungen die du gemacht hast aber zu Herzen genommen^-^

Find ich gut, hab mittlerweile noch ein paar Änderungen hinzugefügt und den Bug mit den Intervallen gefixxt.

Was mich noch stört:
* Die Status Anzeige ist praktisch geraten (kann man mit dem Ansatz aber auch schlecht machen, weil wegen SOP)
* Kein Error Handling (Ok, das ist häufig so bei Tutorials)
* Denglishe Variablen
* Die Stadt ist immer noch Hardcoded
* Die Zeit beim Bot sollte auch Variable bestimmbar sein, nicht fix auf 10 Min.