Automat » Fehler in der Ertrags-Berechnung

Jop, da hast du Recht. Nervt mich auch ab und an mal, war aber noch nicht so stark, dass ich es angepasst habe

Ich denke, die beste Lösung ist eine neue Spalte im Popup zwischen "Ertrag" und "Punkte" mit dem Titel "Ertrag (netto)" oder so ähnlich, welche dann die von dir beschriebene Kalkulation durchführt.

LG

Zitat von Chris73

---

Hallo zusammen,

auch auf die Gefahr hin, daß ich jetzt gesteinigt werde , aber in der Ertragsberechnung der Queue-Listen steckt ein dicker Fehler.

Z.B. bei Raps wird ein Ertrag von 90 angegeben, was auch richtig ist, wenn man den Raps, der gepflanzt wurde, außen vor läßt. Der Reinertrag ist aber eigentlich nur 60. Bei einmal Pflanzen ist das ja nicht weiter tragisch.

Wenn man aber die gleiche Pflanze mehrfach anpflanzt, um z.B. auf einen bestimmten Stand zu kommen, stimmt die gesamte Rechnung nicht mehr.

Bleiben wir beim Beispiel Raps, von dem man mal angenommen 1.000 Pflanzen braucht. Wenn man nach der Ertragsanzeige der Queue-Liste geht, müßte der Raps dann:

1.000/90=12 mal angepflanzt werden.

Man wird dann aber feststellen, daß immer noch 280 Pflanzen fehlen, weil man als Reinertrag nur:

12*60=720 Pflanzen bekommen hat,

denn nach jeder Ernte setzt man ja 30 Pflanzen wieder ein.

Wenn ich doch einen Denkfehler haben sollte, dann bin ich gerne bereit, mich eines besseren belehren zu lassen.

Ansonsten wäre es vielleicht sinnvoll und für die Programmierer hier wahrscheinlich nur eine Kleinigkeit, die Berechnung entsprechend abzuändern.
Es würe ja schon reichen, bei dem theoretischen ertrag bei der Berechnung jeweils 1 abzuziehen.

Gruß

Christoph

---

Da du mit der Idee kamst, darfst jetzt auch Beta-Testen

Installiere dir den Automaten und Berater, die unter https://github.com/MoeLa/GMscripts_MyFreeFarm/tree/mf zu finden sind (einfach deine aktuellen überschreiben). Fehler hier melden. Wenn's lang genug nix zu meckern gibt, schick' ich dem Linus n Pull-Request, sodass er es in einer neuen Version veröffentlicht.

Du warst schneller als ich. [cool] Mist

Ich fügte eine Änderung in die Funktion updateQueueBox in Zeile 3162 des Automaten hinzu

if (getBuildingTyp(zoneNrL)==1){
	iAmount-=unsafeData.BUILDING_SIZE[getBuildingTyp(zoneNrL)]/unsafeData.prodPlantSize[0][iProd];
}
 
 

Oh, hab da größere Geschütze aufgefahren (neue Spalte und so).

Ist bei dir die Blumenwiese mit dabei?

Zitat von linus--tux

---

Du warst schneller als ich. [cool] Mist

Ich fügte eine Änderung in die Funktion updateQueueBox in Zeile 3162 des Automaten hinzu

1
2
3
4
5
 

if (getBuildingTyp(zoneNrL)==1){
	iAmount-=unsafeData.BUILDING_SIZE[getBuildingTyp(zoneNrL)]/unsafeData.prodPlantSize[0][iProd];
}
 
 

---

Ah, das liegt dann wohl am ProduktTyp der Exoten. Den hab ich dann wohl vergessen... kommt noch

Zitat von Chris73

---

Ich lasse die Beta-versionen jetzt einfach mal bis morgen laufen. Sieht aber auf den ersten Blick super aus. Dann muß man nicht immer rechnen

PS: Ich hab gerade mal die verschiedenen Zonen durchgesehen. Die Änderung ist bei den Farmen 1-4 aktiv sowie auf der Blumenwiese des Bauernmarktes.
Auf Farm 5 ist immer noch die alte Anzeige da.

---

Exotische Produkte: Ja

Blumenwiese: Nein

Zitat von Moe

---

Oh, hab da größere Geschütze aufgefahren (neue Spalte und so).

Ist bei dir die Blumenwiese mit dabei?

Zitat von linus--tux

---

Du warst schneller als ich. [cool] Mist

Ich fügte eine Änderung in die Funktion updateQueueBox in Zeile 3162 des Automaten hinzu

1
2
3
4
5
 

if (getBuildingTyp(zoneNrL)==1){
	iAmount-=unsafeData.BUILDING_SIZE[getBuildingTyp(zoneNrL)]/unsafeData.prodPlantSize[0][iProd];
}
 
 

---

---

Ich hab die exotischen jetzt auch drin. Wer hat denn da das "ex" Kürzel eingeführt?

Zitat von linus--tux

---

Exotische Produkte: Ja

Blumenwiese: Nein

Zitat von Moe

---

Oh, hab da größere Geschütze aufgefahren (neue Spalte und so).

Ist bei dir die Blumenwiese mit dabei?

Zitat von linus--tux

---

Du warst schneller als ich. [cool] Mist

Ich fügte eine Änderung in die Funktion updateQueueBox in Zeile 3162 des Automaten hinzu

1
2
3
4
5
 

if (getBuildingTyp(zoneNrL)==1){
	iAmount-=unsafeData.BUILDING_SIZE[getBuildingTyp(zoneNrL)]/unsafeData.prodPlantSize[0][iProd];
}
 
 

---

---

---

Jop. Was natürlich nicht berücksichtigt wird, ist, wenn man nicht genügend Saatgut im Rack hat. Aber das wurde zuvor auch nicht beachtet.

Zitat von Chris73

---

Zitat von Moe

---

Ich hab die exotischen jetzt auch drin.

---

Na, dann will ich mal testen

Aber ich schätze mal, das läuft genauso problemlos, wie die ersten Änderungen.

---

Das "ex" wurde von mir eingefügt.

Die neue Ertragsberechnung ist in der aktuellen Version des Automaten 2.9.8 vom 25.02.2016 enthalten. Zusehen ist sie dann, wenn du unter Optionen-Automat die Option "Benutze Queue-Listen für die Äcker" aktivierst. In dem Einstellungsfenster der Acker ist dann, wenn du mit der Maus über die eingestellten Pflanzen fährst, eine weitere Spalte "Nettoertrag" sichtbar.

Punkt "manuelles installieren": in dem githup-Bereicht mußt du zuerst das Script anklicken. In der sich öffnenden Seite klickst du dann auf den Button "Raw". Auf der nächsten Seite sollte dann der Button "Script installieren" oder "Download" oder so ähnlich wieder sichtbar sein.