View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0002137EresseaMACHEpublic2015-09-12 14:252017-12-05 18:50
ReporterEnno Assigned ToEnno  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version3.6 
Target Version3.7.0Fixed in Version3.7.0 
Summary0002137: learning-by-doing respektiert die Anzahl der Arbeiter nicht
Description

In die Funktion produceexp wird ein Parameter n übergeben, der die Anzahl der Arbeiter angibt. Wenn der kleiner als u->number ist, sollte sich die Lernchance entsprechend verringern, tut sie aber nicht. Der Parameter wird überhaupt nicht benutzt.

int produceexp(struct unit *u, skill_t sk, int n) { if (global.producexpchance > 0.0F) { if (n == 0 || !playerrace(u_race(u))) return 0; learn_skill(u, sk, global.producexpchance); } return 0; }

Steps To Reproduce

Dieser Code hat keine Tests, aber sobald er welche hat, wird das offensichtlich sein.

TagsNo tags attached.
Faction0
GameE2
Report944

Activities

Enno

Enno

2015-09-12 14:51

administrator   ~0006133

Fixed in https://github.com/eressea/server/pull/296
CTD

CTD

2015-10-12 19:56

developer   ~0006174

Enno, du hast grade einwandfrei funktionierenden Code Kaputt gemacht! Vom logischen, die Personen die nicht Arbeiten können sich in der Zeit weiterbilden, wenn überhaupt müssen die Lerntage bei nichtauslaustung steigen. Vom Praktischen: Extreme Erhöhung des mikromanagments. Von den Regeln: Wendet sie das Talent lediglich an, ist die Chance in eine neue Stufe zu kommen auf 1/3 der Chance beim Lernen reduziert, man steigt dadurch also etwa mit 1/3 der Geschwindigkeit des exklusiven Lernens auf. Das ist einfach und klar, nach der neuen Regelung ist das auch schwer zu erklären. Bitte Orginalzustand wieder herstellen!
Enno

Enno

2015-10-13 09:22

administrator   ~0006175

Ich glaube, du missverstehst die Intention des Codes (und die Regeln). Ich habe hier lediglich den Zustand wieder hergestellt, der schon seit Ewigkeiten galt. Diskussion über mögliche ander Regelungen ist besser auf eressea-dev aufgehoben, als in einem Bugreport.

Issue History

Date Modified Username Field Change
2015-09-12 14:25 Enno New Issue
2015-09-12 14:25 Enno Status new => assigned
2015-09-12 14:25 Enno Assigned To => Enno
2015-09-12 14:51 Enno Note Added: 0006133
2015-09-12 14:51 Enno Status assigned => resolved
2015-09-12 14:51 Enno Fixed in Version => 3.7.0
2015-09-12 14:51 Enno Resolution open => fixed
2015-10-12 19:56 CTD Note Added: 0006174
2015-10-12 19:56 CTD Status resolved => feedback
2015-10-12 19:56 CTD Resolution fixed => reopened
2015-10-13 09:22 Enno Note Added: 0006175
2015-10-13 09:22 Enno Status feedback => assigned
2015-10-13 09:22 Enno Status assigned => resolved
2015-10-13 09:22 Enno Resolution reopened => fixed
2017-12-05 18:50 Enno Status resolved => closed