Vorbei an Scala-Map als argument an eine Funktion zu viel Zeit kostet

Ich habe ein scala-companion-Objekt mit einer Methode, die annimmt, eine Karte als parameter. Dann geht diese Karte an eine andere Funktion in einem anderen companion-Objekt ohne Veränderungen. Und die eigentliche Methode aufrufen dauert zu viel Zeit, wenn die Methode, die Ausführung ist schnell (habe ich gemessen alles). Wenn ich don ' T-pass-eine Karte (mit null statt) und es wirkt schnell, aber mit der übergabe als argument, die tatsächlichen Aufruf der Methode ist sehr langsam.

Bin ich etwas fehlt, und die Karte wird neu erstellt und nicht nur eine Referenz übergeben wird?

object ContentElementParser {
    def parse(node: Node, assets: Map[String, Asset]): Option[ContentElement] = {
        //Some logic here
        AssetParser.getAsset(subNode, assets) //this call is too slow because of assets map
    }
}

object AssetParser {
    def getAsset(node: Node, assetMap: Map[String, Asset]): Asset = {
        //logic
    }
}

InformationsquelleAutor makados | 2013-07-23

4

Wird durch Verweis übergeben. Etwas anderes Los ist-du bist die Messung das erste mal, wenn Sie die Karte verwenden, die erfordert einige Klasse be auch (nachfolgende Aufrufe wäre viel schneller), oder Sie tun viel mehr Arbeit, wenn Sie übergeben eine Karte im Gegensatz zu null oder Sie sind sehr knapp aus Speicher und Sie sind mess-garbage-collection-Zeit statt.

Könnte es kopiert werden, wenn es eine implizite Konvertierung im Rahmen, aber wenn der Typ Signatur ist genau das gleiche in beiden Orten, das wäre nicht ein Problem, da "keine Konvertierung" hat immer Priorität.

Hier ist der bytecode für die parse call (mit einem content Methode Hinzugefügt Asset so erzeugt es eine Option[ContentElement], und ein sub Methode Hinzugefügt Node zu füllen subNode):
```
def parse(node: Node, assets: Map[String, Asset]): Option[ContentElement] =
      AssetParser.getAsset(node.sub, assets).content


public scala.Option parse(Node, scala.collection.immutable.Map);
  Code:
   0:  getstatic     #19; //Field AssetParser$.MODULE$:LAssetParser$;
   3:  aload_1
   4:  invokevirtual #25; //Method Node.sub:()LNode;
   7:  aload_2
   8:  invokevirtual #29; //Method AssetParser$.getAsset:
                            (LNode;Lscala/collection/immutable/Map;)LAsset;
   11: invokevirtual #35; //Method Asset.content:()Lscala/Some;
   14: areturn
```
Sehen? Keine Karte kopieren. aload_2 ist die Karte übergeben wird. Nichts passiert, außer, dass es an getAsset über invokevirtual.
- Vermögens-map erstellt, in einem anderen Ort, und alle, die es ausgeführt wird innerhalb einer for-Schleife. Der erste Anruf wird länger - 2,5 sec, wenn alle subsequend sind rund 1,3 Sekunden. getAsset Methode Körpers ausgeführt, in einer Millisekunde.
- Ich habe Hinzugefügt, th Vorschlag es wirklich die garbage-collection. (Versuchen Sie -verbose:gc -XX:+PrintGCTimeStamps -XX:+PrintGCDetails auf der Kommandozeile, vielleicht?)
- durch "geben Sie die Signatur ist genau das gleiche", meinen wir, unveränderlich.Karte in beiden Methoden? Auch verdächtig, dass Ihre großen Zeiten variieren um den Faktor 2.
- geben Sie Signaturen, sind genau dieselben - ursprünglich war es collection.Anzeigen-Merkmal, sondern suchen Wege aus der es, ich habe es geändert unveränderlich Karte, was nichts geändert.
- Post ein minimiertes Beispiel, und dann vielleicht können Sie einige helfen. Jetzt haben Sie geschrieben, eine Skizze von etwas, das sollte gut funktionieren, und sind der Suche nach Problemen, die innerhalb es, wenn keines vorhanden ist. Es ist woanders. Da Sie nicht zeigen, die "anders", es ist ziemlich schwer zu helfen, finden Sie das Problem.
- Sie vermeiden, Objekt-zu-Objekt ruft nicht, warum es funktioniert viel schneller, aber wenigstens umgestalten losgeworden, etwas anderes, das war tatsächlich das problem.
InformationsquelleAutor Rex Kerr
1

Also habe ich versucht zu reproduzieren, dies in einem eigenständigen Projekt-und scheiterte, es funktioniert gut. Aber in einem Projekt war es langsam so offensichtlich etwas anderes vor sich ging.

Ich landete refactoring den code und loszuwerden von Objekt zu Objekt fordert. Erstellt eine übliche Klasse, die akzeptiert assetsMap als Konstruktor val und jetzt funktioniert es viel schneller

InformationsquelleAutor makados

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.