View Issue Details

IDProjectCategoryView StatusLast Update
0002145EresseaKampfpublic2017-12-05 18:50
ReporterCTD Assigned ToMrTurner  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version3.7.0 
Target Version3.7.0Fixed in Version3.7.0 
Summary0002145: Armbrüste halbieren die Rüstung nicht
Description

Ich bin im Code über das hier gestolpert:

if CHANGED_CROSSBOWS

if (awtype && fval(awtype, WTF_ARMORPIERCING)) {
    /* crossbows */
    ar /= 2;
    an /= 2;
}

endif

Da CHANGED_CROSSBOWS in der settingh.h fix als 0 definiert wird und ich auch keine andere Stelle im Code gefunden habe wo das brücksichtigt wird scheint es kein WTF_ARMORPIERCING zu geben, oder habe ich da was übersehen?

TagsNo tags attached.
Factionnone
GameN/A
Report0

Activities

Enno

Enno

2015-10-10 09:09

administrator   ~0006169

Sieht so aus. Ich würde das #define da entfernen, wenn die Anleitung sagt, dass Armbrüste die Rüstung halbieren, oder den Code komplett löschen, wenn sie das nicht sagt. Dementsrprechend entweder als Bugfix ankündigen, oder als Code-Vereinfachung verschweigen.

Mit #define zu konfigurieren ist ein code smell, das machen wir heute nicht mehr. Früher (Vinyambar, glaube ich) gab es mal verschiedene Executables für verschiedene Spiele, mit getrennten settings.h Dateien, heute benutzen alle Spiele den selben Code mit verschiedenen Konfigurationen (ini/json/xml).

CTD

CTD

2015-10-12 19:45

developer   ~0006173

Ja, im Wiki steht unter er Waffentabelle:

Bei einem Armbrusttreffer wird der Rüstungsschutz des Gegners halbiert und abgerundet.

Ich würde das ganze auch noch etwas weiter nach "hinten" verschieben, nachdem RüstungsRS und NatürlicherRS schon zusammengelegt sind, sonst ist z.B. ein Dämon mit PlattenPanzer in E3 (RS3 +1/2 Natürliche von 2 = RS 4) nicht wie erwartet bei RS 2 sondern bei RS 1 (3/2 = 1 , 2/2 = 1, 1+ 1/2 von 1 = 1,5 abgrundet zu 1).

MrTurner

MrTurner

2015-11-03 12:08

updater   ~0006192

Der panzerbrechende Effekt der Armbrüste wird nun tatsächlich angewendet und zwar auf den Verbund aus Rüstung und natürlicher Rüstung.

Issue History

Date Modified Username Field Change
2015-10-08 07:02 CTD New Issue
2015-10-10 09:09 Enno Note Added: 0006169
2015-10-12 19:45 CTD Note Added: 0006173
2015-10-31 16:51 MrTurner Assigned To => MrTurner
2015-10-31 16:51 MrTurner Status new => assigned
2015-11-03 12:08 MrTurner Note Added: 0006192
2015-11-03 12:08 MrTurner Status assigned => resolved
2015-11-03 12:08 MrTurner Resolution open => fixed
2015-11-03 12:15 MrTurner Status resolved => feedback
2015-11-03 12:15 MrTurner Resolution fixed => reopened
2015-11-03 12:15 MrTurner Status feedback => resolved
2015-11-03 12:15 MrTurner Fixed in Version => 3.7.0
2015-11-03 12:15 MrTurner Resolution reopened => fixed
2015-11-03 12:16 Enno Target Version => 3.7.0
2017-12-05 18:50 Enno Status resolved => closed
2023-05-28 12:25 Enno Category ATTACKIERE => Kampf