Programm "frisst" zu viel CPU

  • VB.NET
  • .NET (FX) 4.5–4.8

Es gibt 16 Antworten in diesem Thema. Der letzte Beitrag () ist von Ferb300.

    Programm "frisst" zu viel CPU

    Hi community!
    Ich hab ein teil von einem Programm erstellt, das eine Welt für ein Jump and Run Game erstellt.
    Wenn auf den Button gedrückt wird, soll die Welt erstellt werden.
    So hab ich das gemacht:
    Spoiler anzeigen

    VB.NET-Quellcode

    1. Public Class Form1
    3. Dim ziel_X_erreicht As Boolean = False
    5. Dim ziel_Y_erreicht As Boolean = False
    7. Dim Blocksize As New Size(16, 16)
    9. Dim BlockPositionY As Decimal = 200
    11. Dim BlockPositionX As Integer = 0
    14. Dim Art As Blocktype = Blocktype.Grass
    16. Private Sub Form1_Load(sender As Object, e As EventArgs) Handles MyBase.Load
    18. End Sub
    20. Private Sub Button1_Click(sender As Object, e As EventArgs) Handles Button1.Click
    22. Do Until ziel_Y_erreicht = True
    24. If BlockPositionY < 1005 Then
    26. Do Until ziel_X_erreicht = True
    28. If BlockPositionX < 965 Then
    30. Dim Block As New PictureBox
    32. Block.Size = Blocksize
    34. Block.Location = New Point(BlockPositionX, BlockPositionY)
    36. Select Case Art
    38. Case Blocktype.Grass
    40. Block.BackgroundImage = My.Resources.grass_block_minecraft
    42. Case Blocktype.Dirt
    44. Block.BackgroundImage = My.Resources.Dirt_Block_Minecraft
    46. End Select
    48. '
    50. BlockPositionX = BlockPositionX + 16
    52. Me.Controls.Add(Block)
    54. Else
    56. ziel_X_erreicht = True
    58. End If
    60. Loop
    62. BlockPositionY -= 16
    64. ziel_X_erreicht = False
    66. Art = Blocktype.Dirt
    68. Else
    70. ziel_Y_erreicht = True
    72. End If
    74. Loop
    76. End Sub
    78. End Class


    sry für die ganzen freizeilen, das ist einfach so passiert

    Wenn man jetzt auf den Button drückt hängt das ganze Programm und ein Blick in den Task-Manager verrät, dass das Programm stolze 30% CPU auslastet! Was kann ich dagegen tun?
    umfassender HTML-Editor mit allem drum und dran: 40%

    Die Picturebox ist ein Control um ein einzelnes Bild darzustellen, nicht um dafür missbraucht zu werden eine Platform in nem Spiel darzustellen. Was du können willst/musst ist es GDI/+ zu beherrschen. Damit zeichnest du auf eine einziges Panel alles was du brauchst.

    @simpelSoft keine Alternativen zu nennen ist natürlich auch schön:
    Dev1: "Ist kacke, machs anders"
    Dev2: "okay, wie?"
    Dev1: "Machs anders."

    Ich habs auch ohne (das Tutorial zu) GDI+ geschafft, nämlich so:
    (keine Ahnung, warum es jetzt funktioniert)
    Spoiler anzeigen

    VB.NET-Quellcode

    1. Public Class Form1
    3. Dim ziel_X_erreicht As Boolean = False
    5. Dim ziel_Y_erreicht As Boolean = False
    7. Dim Blocksize As New Size(16, 16)
    9. Dim BlockPositionY As Decimal = 200
    11. Dim BlockPositionX As Integer = 0
    13. Enum Blocktype
    15. Dirt = 0
    17. Grass = 1
    19. End Enum
    21. Dim Art As Blocktype = Blocktype.Grass
    23. Private Sub Form1_Load(sender As Object, e As EventArgs) Handles MyBase.Load
    25. SetWorldAgain()
    27. End Sub
    29. Sub SetWorldAgain()
    31. BlockPositionX = BlockPositionY = 0
    33. Do Until ziel_Y_erreicht = True
    35. If BlockPositionY < Me.Height Then
    37. Do Until ziel_X_erreicht = True
    39. If BlockPositionX < Me.Width Then
    41. Dim Block As New PictureBox
    43. Block.Size = Blocksize
    45. Block.Location = New Point(BlockPositionX, BlockPositionY)
    47. Select Case Art
    49. Case Blocktype.Grass
    51. Block.BackgroundImage = My.Resources.grass_block_minecraft
    53. Case Blocktype.Dirt
    55. Block.BackgroundImage = My.Resources.Dirt_Block_Minecraft
    57. End Select
    59. '
    61. BlockPositionX = BlockPositionX + 16
    63. Me.Controls.Add(Block)
    65. Else
    67. ziel_X_erreicht = True
    69. End If
    71. Loop
    73. BlockPositionY += 16
    75. BlockPositionX = 0
    77. ziel_X_erreicht = False
    79. Art = Blocktype.Dirt
    81. Else
    83. ziel_Y_erreicht = True
    85. End If
    87. Loop
    89. End Sub
    91. 　
    93. End Class


    sry (mal wieder) für die ganzen freizeilen
    Kann mir einer sagen, warum es jetzt hinhaut?
    umfassender HTML-Editor mit allem drum und dran: 40%

    Ich vermute mal deswegen, vergleiche die beiden.

    Ferb300 schrieb:

    VB.NET-Quellcode

    1. Do Until ziel_Y_erreicht = True
    2. If BlockPositionY < 1005 Then
    3. BlockPositionY -= 16 ' ein -=
    4. Else
    5. ziel_Y_erreicht = True
    6. End If
    7. Loop

    Ferb300 schrieb:


    VB.NET-Quellcode

    1. Do Until ziel_Y_erreicht = True
    2. If BlockPositionY < Me.Height Then
    3. BlockPositionY += 16 ' ein +=
    4. Else
    5. ziel_Y_erreicht = True
    6. End If
    7. Loop

    Wenn du genau hinsiehst, müsstest du erkennen, dass im Startpost die Bedingung BlockPositionY < 1005 NIE eintreten kann, da du den Wert ständig verringerst.

    Dieser Beitrag wurde bereits 4 mal editiert, zuletzt von „EaranMaleasi“ ()

    Danke, ich wusste zwar das es an diesen beiden zeilen liegt aber nicht, warum.
    Jetz weiß ich es! :thumbsup:
    umfassender HTML-Editor mit allem drum und dran: 40%

    Irrelevant. Auch wenn es jetzt so funktionieren mag.
    Guck dir Bitte Gdi+ an und das OOP konzept. Du kannst nämlich alles schön in Klassen sortieren für Tiles/Plattformen für deine Map, Player, Items, NPC.
    Und später kann man auch noch mit Vererbung arbeiten.
    Glaube mir es macht nicht nur den Code wesentlich übersichtlicher und besser wartbar, sondern Erlaubt dir auch viel größere Projekte relativ einfach zu meistern
    Ich wollte auch mal ne total überflüssige Signatur:
    ---Leer---

    jvbsl schrieb:

    Guck dir Bitte Gdi+ an und das OOP konzept.


    Bei GDI+ check ich zwar überhauptnichts, aber das OOP-Konzept beherrsche ich im schlaf. Zum Beispiel werden jetzt nicht lauter pictureboxen erstellt sonder Objekte der klasse Block (selbstgeschrieben!), die von PictureBox erbt und Methoden und Funktionen hat, zum zerstören von blöcken, wiederhinbauen usw... Wenn das spiel fertig ist, tu ichs in den Showroom und poste hier einen link
    umfassender HTML-Editor mit allem drum und dran: 40%

    Nein du wirst nicht von PictureBox erben^^
    Kannst dir nen Interface Drawable mit ner Draw Methode machen und davon erben.
    In deiner Klasse Block speicherst du dann alle nötigen Informationen, die man zum zeichnen braucht. Implementierst die Draw Methode in welcher du auf das gegebene Graphics Objekt zeichnest.
    Dann kannst dir eine Map machen, die auch Drawable implementiert und alle Blöcke ebthält und deren draw Methoden in der eigenen aufruft.
    Aber auch da könnte man noch eine viel bessere Architektur machen.
    Ich wollte auch mal ne total überflüssige Signatur:
    ---Leer---

    Ferb300 schrieb:

    jvbsl schrieb:

    Guck dir Bitte Gdi+ an und das OOP konzept.


    Bei GDI+ check ich zwar überhauptnichts, aber das OOP-Konzept beherrsche ich im schlaf. Zum Beispiel werden jetzt nicht lauter pictureboxen erstellt sonder Objekte der klasse Block (selbstgeschrieben!), die von PictureBox erbt und Methoden und Funktionen hat, zum zerstören von blöcken, wiederhinbauen usw... Wenn das spiel fertig ist, tu ichs in den Showroom und poste hier einen link


    Wenn du von der Picturebox erbst, ist deine neue Klasse auch eine voll implementierte PictureBox, was deiner Aussage widerspricht, nicht lauter PictureBoxen zu erstellen.
    Vererbung nochmal nachlesen, wenn du wach bist und nicht im Schlaf lernen :thumbsup:

Facebook
Telefonspende