english for core classes with methods, attributes and (most) comments #54
Conversation
|
Wohooo. How cool is that?! |
There was a problem hiding this comment.
Wow. This looks quite awesome. Thank you for the work.
I have only two notes:
- I found a place where the actual ant API was changed (Angriff -> Angriffsstaerke) which potentially breaks existing ant implementation
- There was a translation for "Last" by "Burdon". but usually we translate this with "Last". I think it's better to be consistent with all the other translations here
You even corrected the wiki links <3
| @@ -444,7 +444,7 @@ private List<PlayerInfo> analyseAssembly(ModuleDefinition module, bool checkRule | |||
| } | |||
| } | |||
|
|
|||
| #region Erkennung �lterer KI-Versionen | |||
| #region Erkennung aelterer KI-Versionen | |||
| internal int H�heI2; | ||
| internal int WidthI; | ||
| internal int WidthI2; | ||
| internal int HightI; |
There was a problem hiding this comment.
This should be "height" I guess
|
|
||
| /// <summary> | ||
| /// Die maximalen Lastwerte aller Kasten des Volkes. | ||
| /// </summary> | ||
| internal readonly int[] Last; | ||
| internal readonly int[] Burden; |
There was a problem hiding this comment.
We usually translate "Last" with "Load" all over the place. To be consistent I would suggest to use "Load" also here.
| } | ||
|
|
||
| /// <summary> | ||
| /// Gibt die Angriffsstärke an | ||
| /// </summary> | ||
| public int Angriff | ||
| public int Angriffstaerke |
There was a problem hiding this comment.
This one will break the API of the Ant. Here we need to stick with "Angriff"
There was a problem hiding this comment.
Danke für die Rückmeldung.
ich bin auch kein Native Speaker aber ich bin beim Compilieren mit Rider auf einige Probleme mit den Umlauten gestoßen, daher der Versuch die Simulation-Classes auf Englisch gerade zu ziehen.
Eine Frage auf die ich beim übersetzten gestoßen bin:
ist Range für Movement Range oder für Smell Range oder für beides?
CasteInfo.cs Zeile 102
// Prüfen, ob der Riechweitemodifikator im Rahmen ist
if (Range < SimulationSettings.Custom.CasteSettings.MinIndex ||
Range > SimulationSettings.Custom.CasteSettings.MaxIndex)
in allen anderen Zusammenhängen, die ich bisher gesehen habe war Range die Bewegungsreichweite und nicht die Riechreichweite
Update obiger Files folgt
There was a problem hiding this comment.
Bei dem ganzen deutschen Quellcode und den Kommentaren brennen mir inzwischen ja auch die Augen :D Deshalb bin ich so froh, dass du dich dieser Sache annimmst.
Was die Range angeht: Es gibt im Grunde ja keine Smell Range. Die Ameise riecht eine Markierung, sobald deren Radius den Kollisionsradius der Ameise kreuzt. Wie weit die Ameise also riechen kann, ist nur von der Markierung abhängig. Daher handelt es sich hier um die maximale Laufreichweite der Ameise.
litte update to ColorFinder.cs division by two for each RGB value in operator + and check for 0 in operator /
…ith english baseAnt because of AntMe.Simulation.RuleViolationException: Ameisen der Version 1.1 oder kleiner werden in dieser Version nicht mehr unterstützt. bei AntMe.Simulation.PlayerAnalysis.analyseAssembly(ModuleDefinition module, Boolean checkRules)
…character uppercase and punctuation marks
|
Huch. Hier ist ja noch richtig was passiert. Lass mich gerne wissen, wenn ihr soweit fertig seid und ich noch mal nen Review machen kann? |
…make it clear where the ant/bug/insect attributes and methods are coming from. Instead of ...Base in most cases ...CoreInsect.
Ja, ich glaube jetzt ein ganz guter Stand erreicht zum Reviewen. |
…sugar hills. SimulationSettings.Custom -> SimulationSettings.Default is a workaround.
…k to CasteIndexCoreInsect again.
No description provided.