Memoryleak bei Spiel & Tipps zur Codeoptimierung

KaskadekingDE · 10. August 2015, 19:08

Hey Leute

,

ich schreibe derzeit ein kleines 2D Spiel mit der Sharpex2D Engine. Allerdings mach ich anscheinend beim rendern was ganz böses :>
Da sich jeweils über den Zeitraum von ca. 30 Sekunden der Arbeitsspeicher sich um 100MB erhöht. Wenn das Spiel im Hintergrund läuft bleibt der Speicherverbrauch gleich.
Im Verdacht hab ich dass es an den Dauerzugriff auf die Arrays liegt (welche dafür wahrscheinlich nicht geeignet sind)

2. Problem:
Bei der Klasse WabeLayout die Methode CalculateFieldType hab ich ziemlich viel hardgecoded Zeug drin. Weiß jemand wie man das schöner lösen kann?

Das Projekt hab ich mal angehangen

ErfinderDesRades · 10. August 2015, 21:28

ist zwar mit FW4.5 gelabelt, aber tatsächlich ists wohl VS2015.
Läuft daher bei mir nicht

KaskadekingDE · 10. August 2015, 21:33

Ok das ist doof, dass man keine VS2015 Projekte in älteren Versionen öffnen kann. Hab außerdem eben bemerkt, dass es doch .NET 4.0 ist (habs mal geändert).

Yothrii · 10. August 2015, 21:49

KaskadekingDE schrieb:

dass man keine VS2015 Projekte in älteren Versionen öffnen kann

@ErfinderDesRades Schreib die SLN um. Darin ist eine Versionsnummer. Setz die runter auf deine VS Version die du nutzt und es könnte funktionieren.

nafets · 10. August 2015, 21:55

Oder lösch einfach die SLN. Dann wird eine neue mit der Version deines Studios generiert.

ErfinderDesRades · 10. August 2015, 22:00

nein, 2015 hat neue sprachfeatures, die 2013 nicht kann.
Und die sind scheints auch verwendet.

KaskadekingDE · 10. August 2015, 22:06

@Yothrii
Allerdings nur bis VS2013, da ich C# 6 Features benutze. Da soweit ich weiß erst ab VS2013 die Möglichkeit gibt C# 6 zu benutzen.

EDIT: @ErfinderDesRades VS2013 bietet afaik auch C# 6 Support an.

ErfinderDesRades · 10. August 2015, 22:15

sieht mir nicht so aus, und in eim Thread von stackoverflow erzählen sie was von hinzu-installieren und bla.
Das tu ich mir nicht an, am Ende zerschieß ich mir noch mein VS.

Aber vlt. weiß jmd. genaueres darüber?

Parmaster · 10. August 2015, 22:19

ErfinderDesRades schrieb:

Aber vlt. weiß jmd. genaueres darüber?

Einfach VS2015 installieren. Es können ja verschiedene Versionen gleichzeitig installiert sein.

KaskadekingDE · 10. August 2015, 22:30

Ich hab das Projekt einfach mal nach C# 5 umgeschrieben. Das mit der SLN musste noch selber fixen. Die neue ZIP ist im Startpost

Btw: Das 1. Projekt (ChainReact) muss auf x86 kompiliert werden.

ErfinderDesRades · 10. August 2015, 22:39

ah - X86 - komisch - warum ist das inne Solution nicht verdrahtet?

also jetzt sehe ich ein komisches Gitter und reagieren tut das program auf garnix. Was muss man machen, dass das was macht?

KaskadekingDE · 10. August 2015, 22:49

Das Spiel selber macht noch nichts. Das Problem ist das ich beim rendern einen Memoryleak drin hab welchen den Speicher volllaufen lässt.
Keine Ahnung warum das nicht in der Solution drin ist.

ErfinderDesRades · 10. August 2015, 23:17

jo - ich hab auch nix gefunden. Scheinbar dreht der sich nur brav im Kreis und rendert die Waben halt.
Ich hab nix gefunden, was disposet werden müsste, oder iwelche Listen, dene zugefügt wird, oder Events die abonniert und abonniert werden, oder insbes. Bilder.
Am Ende ists ein Bug inne Sharp2DEngine, weil wenn ich den GameLoop ohne die Zeichen-Befehle laufen lasse, bleibt der Speicher unverändert:

C#-Quellcode

public override void OnRendering(RenderDevice renderer, GameTime gameTime) {
renderer.Begin();
var cut = WabeSize / 3;
foreach (var wabe in _game.Wabes) {
var wabeX = (wabe.X + 1) * WabeSize;
var wabeY = (wabe.Y + 1) * WabeSize;
var rect = new Rectangle(wabeX, wabeY, WabeSize, WabeSize);
renderer.FillRectangle(Color.Black, rect);
for (var x = 0; x <= 2; x++) {
for (var y = 0; y <= 2; y++) {
var field = wabe.ConvertVector2ToWabeField(new Vector2(x, y));
var color = SelectColorFromField(wabe.Owner, field);
var mutltiplicatorX = cut * x;
var mutltiplicatorY = cut * y;
//renderer.FillRectangle(color, new Rectangle((wabeX) + mutltiplicatorX, (wabeY) + mutltiplicatorY, cut, cut));
//renderer.DrawRectangle(new Pen(Color.White, 3), new Rectangle((wabeX) + mutltiplicatorX, (wabeY) + mutltiplicatorY, cut, cut));
}
}
//renderer.DrawRectangle(new Pen(Color.Yellow, 3), rect);
}
renderer.End();
}

KaskadekingDE · 10. August 2015, 23:23

Da haste anscheinend recht. Ich erwähn mal @ThuCommix hier. Vielleicht weiß er bescheid

nafets · 10. August 2015, 23:55

Wie wäre es, wenn du einfach jeden Befehl (d.h. auch Sharpex2D) durchsteppst und schaust, was verdächtig aussieht?

ThuCommix · 11. August 2015, 09:04

Du könntest mal probieren den Pen außerhalb des OnRendering zu deklarieren und nicht in jeden Frame - eventuell könnte das schon die Lösung des Problems sein. Welche Version von Sharpex2D verwendest du?

KaskadekingDE · 11. August 2015, 10:04

Es hat teilweise geholfen. Jedenfalls steigt der Arbeitsspeicher langsamer d.h es bleibt trotzdem der Memoryleak.
Ich verwende die Version 1.2.3 (die von NuGet).

So siehsts jetzt aus:

Spoiler anzeigen

C#-Quellcode

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Xml.Serialization;
using ChainReact.Core.Game;
using ChainReact.Core.Game.Field;
using ChainReact.Core.Game.Objects;
using Sharpex2D;
using Sharpex2D.GameService;
using Sharpex2D.Math;
using Sharpex2D.Rendering;
namespace ChainReact
{
public class MainGame : Game
{
public const int WabeSize = 64;
private ChainReactGame _game;
private Color _unusedFields;
private Color _spheresUnpowered;
private Color _spheresPowered;
private Color _unowned;
private Pen _whiteBorder;
private Pen _yellowBorder;
public override EngineConfiguration OnInitialize(LaunchParameters launchParameters)
{
var resultDevice = DeviceSelector.Select(new RenderDevice[]
{
new Sharpex2D.Rendering.DirectX9.DirectXRenderDevice(),
new Sharpex2D.Rendering.DirectX10.DirectXRenderDevice(),
new Sharpex2D.Rendering.DirectX11.DirectXRenderDevice()
}, DeviceSelector.SelectorMode.Highest, true);
return new EngineConfiguration(resultDevice, null);
}
public override void OnLoadContent()
{
// TODO Add player configuration
var players = new List<Player> {new Player(1, "Player1", Color.Green), new Player(2, "Player2", Color.Red)};
_game = new ChainReactGame(players);
_unusedFields = Color.Gray;
_spheresUnpowered = Color.LightGray;
_spheresPowered = Color.Black;
_unowned = Color.White;
_whiteBorder = new Pen(Color.White, 3);
_yellowBorder = new Pen(Color.Yellow, 3);
}
public override void OnUpdate(GameTime time)
{
}
public override void OnRendering(RenderDevice renderer, GameTime gameTime)
{
renderer.Begin();
var cut = WabeSize / 3;
foreach (var wabe in _game.Wabes)
{
var wabeX = (wabe.X + 1)* WabeSize;
var wabeY = (wabe.Y + 1) * WabeSize;
var rect = new Rectangle(wabeX, wabeY, WabeSize, WabeSize);
renderer.FillRectangle(Color.Black, rect);
for (var x = 0; x <= 2; x++)
{
for (var y = 0; y <= 2; y++)
{
var field = wabe.ConvertVector2ToWabeField(new Vector2(x, y));
var color = SelectColorFromField(wabe.Owner, field);
var mutltiplicatorX = cut * x;
var mutltiplicatorY = cut*y;
renderer.FillRectangle(color, new Rectangle((wabeX) + mutltiplicatorX, (wabeY) + mutltiplicatorY, cut, cut));
renderer.DrawRectangle(_whiteBorder, new Rectangle((wabeX) + mutltiplicatorX, (wabeY) + mutltiplicatorY, cut, cut));
}
}
renderer.DrawRectangle(_yellowBorder, rect);
}
renderer.End();
}
private Color SelectColorFromField(Player player, WabeField field)
{
switch (field.Type)
{
case WabeFieldType.Unused:
return _unusedFields;
case WabeFieldType.Unpowered:
return _spheresUnpowered;
case WabeFieldType.Powered:
return _spheresPowered;
case WabeFieldType.Center:
return player?.Color ?? _unowned;
default:
return Color.Transparent;
}
}
}
}

@nafets
Werd ich machen

ThuCommix · 11. August 2015, 10:56

Bist du sicher das der GC nicht irgendwann alles wegräumt? In den Draw Calls ist nichts mehr was irgendwas im RAM erzeugen würde. Eventuell steigt der RAM Verbrauch weil du neue Rectangles erzeugst, aber das dürfte der GC dann irgendwann wegräumen.

KaskadekingDE · 11. August 2015, 11:06

Der GC arbeitet zwar mit aber wegräumen tut er nichts (oder halt wenig). Hab jetzt mal die neueren Version der Lib probiert und da scheint es das Problem nicht zu geben

ThuCommix · 11. August 2015, 11:07

Klingt ja schon mal so, als hätt ich da tatsächlich was verbessert :D.

Memoryleak bei Spiel & Tipps zur Codeoptimierung

Memoryleak bei Spiel & Tipps zur Codeoptimierung

KaskadekingDE schrieb:

ErfinderDesRades schrieb:

C#-Quellcode

C#-Quellcode

Ähnliche Themen

7 Benutzer haben hier geschrieben