search

phylll / mychs-macro-magic

A simple, sane, and friendly little scripting language for your Roll20 macros.
MIT License
0 stars 0 forks source link

meleeAttack & rangedAttack: Dokumentation der Modifikatoren reparieren & Tooltip ergänzen #6

Closed phylll closed 3 years ago

phylll commented 3 years ago

Angriffsmodifikatoren neben der Privattabelle für den kontrollierenden Spieler und GM auch im Tooltip dokumentieren.

michael-buschbeck commented 3 years ago

Die Privattabelle hat (bei mir) offenbar auch einen Bug und zeigt viel mehr an als das, was tatsächlich in die Modifikatoren eingeflossen ist:

image

Erzeugt bei einem Angriff von hinten.

phylll commented 3 years ago

Spannend. Bei meinen Tests ist das nicht passiert, muss ich mal im Testspiel anschauen. Was ich da mache, ist im Dropdown die verschiedenen Kombinationen von Boni/Mali als Produkt von Primzahlen zu codieren. Das wird dann ausgewertet mit !mmm if semiAutoModCode % 19 == 0 usw. -- und in deinem Fall kommt ein Wert raus, der offenbar durch alle Primzahlen zwischen 19 und 2 sauber teilbar ist?!

image

michael-buschbeck commented 3 years ago

Hah, clever!

0 wäre so ein Wert – oder halt irgendetwas, was MMM nicht als Zahl interpretieren kann und das deswegen beim Modulo-Operator zu 0 wird.

Die Tabelle passt tatsächlich auch zum berechneten Endergebnis: Die Summe aller Einträge ist +2.

Ich glaube, ich habe bei den Drop-downs überall den Default genommen.

phylll commented 3 years ago

Tatsache. Da steht eine 0 im Code, wo lt. der Exceltabelle eben eine 1 stehen sollte (für einen normalen Angriff ohne irgendeinen dieser Modifikatoren aus den Dropdown-Boxen). Keine Ahnung wann und wie die da reingerutscht ist, nachdem ich das getestet hatte. Ich bau noch den Tooltip.

Vielen Dank!

michael-buschbeck commented 3 years ago

Die Tabelle ist natürlich toll – ich hatte nur nicht realisiert, dass sie schon die effektiven Modifikatoren anzeigt. Tooltip ist dann das Tüpfelchen auf dem I :)

phylll commented 3 years ago

Tooltip war einfacher als ich dachte, habe ich heute früh auch noch schnell gebaut für alle drei Skripte. Muss ich nur nochmal ein bisschen testen, bevor der PR kommt. Viel copy & paste, da kann ja immer mal was passieren...