C# Memory leak

  • C#
  • .NET (FX) 4.5–4.8

Es gibt 5 Antworten in diesem Thema. Der letzte Beitrag () ist von Niko Ortner.

    C# Memory leak

    Moin zusammen,

    ich würde gerne mal eure Meinung zu folgender Problemstellung wissen:

    C#-Quellcode

    1. ​namespace ConsoleApp1 {
    2. class Program {
    3. static void Main(string[] args) {
    4. List<Action> actions = new List<Action>();
    6. Task.Run(() => {
    7. byte[] data = new byte[100_000_000];
    8. for (int i = 0; i < data.Length; i++) {
    9. data[i] = 1;
    10. }
    11. actions.Add(() => {
    12. data[44] = 22;
    13. });
    14. });
    16. while (true) {
    17. System.Threading.Thread.Sleep(2000);
    18. if (actions.Count > 0) {
    19. actions[0]();
    20. actions.RemoveAt(0);
    21. Console.WriteLine("Action done!");
    22. }
    23. GC.Collect();
    24. GC.WaitForPendingFinalizers();
    25. GC.Collect();
    26. }
    27. }
    28. }
    29. }


    Also meiner Meinung nach ist das (jetzt nicht gerade sinnvoller aber) valider C# Code. Jedoch bleibt das Array für immer stehen, also eigentlich ein Leck, obwohl ich nach dem entfernen der Action aus der Liste keine Referenzen mehr darauf habe. Ist das ein Leck im System oder habe ich da was übersehen?

    @Bluespide Die Frage ist doch, wo das Array angelegt wird. Für mich sieht das so aus, als ob das Array lokal in der Main()-Prozedur abgelegt ist, in einer Action-Instanz in jedem Falle nicht.
    Wenn dem so ist, wird es natürlich auch erst nach Beenden der Main()-Prozedur zerstört.
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!

    Bluespide schrieb:

    in eine Methode
    Poste mal diesen Code.
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!

    C#-Quellcode

    1. ​namespace ConsoleApp1 {
    2. class Program {
    3. static void Main(string[] args) {
    4. List<Action> actions = new List<Action>();
    5. Test t = new Test();
    6. t.TestMeth(actions);
    7. t = null;
    8. while (true) {
    9. System.Threading.Thread.Sleep(2000);
    10. if (actions.Count > 0) {
    11. actions[0]();
    12. actions.RemoveAt(0);
    13. Console.WriteLine("Action done!");
    14. }
    15. GC.Collect();
    16. GC.WaitForPendingFinalizers();
    17. GC.Collect();
    18. }
    19. }
    20. }
    22. class Test {
    23. public void TestMeth(List<Action> actions) {
    24. Task.Run(() => {
    25. byte[] data = new byte[100_000_000];
    26. for (int i = 0; i < data.Length; i++) {
    27. data[i] = 1;
    28. }
    29. actions.Add(() => {
    30. data[44] = 22;
    31. });
    32. });
    33. }
    34. }
    35. }

    Ich hab hier ein Testprogramm für dich:
    Spoiler anzeigen

    C#-Quellcode

    1. class Program
    2. {
    3. public static void PrintInfo(string Text)
    4. {
    5. Console.WriteLine("[{0}] {1}", System.Threading.Thread.CurrentThread.ManagedThreadId, Text);
    6. }
    8. private static byte[] data2 = null;
    10. static void Main(string[] args)
    11. {
    12. PrintInfo("Start");
    13. List<Action> actions = new List<Action>();
    14. Test t = new Test();
    15. t.TestMeth(actions);
    16. t = null;
    17. while (true)
    18. {
    19. PrintInfo("Waiting for input");
    20. PrintInfo("1: Show Actions");
    21. PrintInfo("2: Perform Action 0");
    22. PrintInfo("3: Collect");
    23. PrintInfo("4: Allocate Data 2 (large)");
    24. PrintInfo("5: Allocate Data 2 (small)");
    25. PrintInfo("6: Clear Data 2");
    26. switch (Console.ReadLine())
    27. {
    28. case "1":
    29. PrintInfo(actions.Count + " Actions to do");
    30. break;
    31. case "2":
    32. if (actions.Count > 0)
    33. {
    34. PrintInfo("Doing Action!");
    35. actions[0]();
    36. actions.RemoveAt(0);
    37. }
    38. else
    39. {
    40. PrintInfo("No Action");
    41. }
    42. break;
    43. case "3":
    44. GC.Collect();
    45. GC.WaitForPendingFinalizers();
    46. GC.Collect();
    47. break;
    48. case "4":
    49. try
    50. {
    51. data2 = new byte[100000000];
    52. for (int i = 0; i < data2.Length; i++)
    53. {
    54. data2[i] = 1;
    55. }
    56. }
    57. catch (OutOfMemoryException)
    58. {
    59. PrintInfo("Out Of Memory");
    60. }
    61. break;
    62. case "5":
    63. try
    64. {
    65. data2 = new byte[1000000000];
    66. for (int i = 0; i < data2.Length; i++)
    67. {
    68. data2[i] = 1;
    69. }
    70. }
    71. catch (OutOfMemoryException)
    72. {
    73. PrintInfo("Out Of Memory");
    74. }
    75. break;
    76. case "6":
    77. data2 = null;
    78. break;
    79. }
    81. }
    82. }
    83. }
    84. class Test
    85. {
    86. public void TestMeth(List<Action> actions)
    87. {
    88. Program.PrintInfo("Start Task");
    89. Task.Run(() => {
    90. Program.PrintInfo("Allocating Array 1");
    91. byte[] data = new byte[1000000000];
    92. Program.PrintInfo("Writing Array 1");
    93. for (int i = 0; i < data.Length; i++)
    94. {
    95. data[i] = 1;
    96. }
    97. Program.PrintInfo("Array 1 Done");
    98. actions.Add(() => {
    99. Program.PrintInfo("Use Array 1");
    100. data[44] = 22;
    101. });
    102. });
    103. }
    104. }

    Daneben den Task-Manager öffnen und im Details-Tab nach Arbeitsspeicher sortieren.
    Beim Debuggen hat Aktion 3 bei mir keinen Effekt, aber wenn ich die Exe im Release-Modus kompiliere und dann vom Explorer starte, dann funktioniert es problemlos.
    "Luckily luh... luckily it wasn't poi-"
    -- Brady in Wonderland, 23. Februar 2015, 1:56
    Desktop Pinner | ApplicationSettings | OnUtils

Facebook
Telefonspende