Hilfe in c++

abratar

Nathan schrieb:

Oh, stimmt, dein Code ist viel falscher als ich dachte.
Dein Tutorial ist definitiv grottig.
Zeile 14: Was machst du da? Tipp: Es ist undefined behavior.
Und daraus resultierend: Auf welchen Speicher greifst in getKey zu? Tipp: Es ist wieder undefined behavior.
Dass du überhaupt was siehst, ist ein sehr glücklicher Zufall.

Kannst du mir denn nicht sagen was da falsch ist ? und wie ich es verbessern kann .. weil "undefined behavior hilft mir nicht viel Meinst du zeile 14 in main.cpp ?

Nathan

Es bezieht sich auf die Player.cpp.
Beschreib bitte ganz genau, was du da machst und wieso du das so machst, wie du es machst.
Das ist nämlich wieder ein typischer Fall von irgendetwas dranklatschen bis es funktioniert.

Dein Hauptproblem scheint *char pMap[11][15]; zu sein. Du scheinst zu meinen es würde einen Zeiger auf ein char [11][15] sein, ist es aber nicht. Stattdesses ist es ein Array von Arrays von char *. Vielleicht wolltest du auch nur den Compiler ruhig stellen, man kann in C++ so ziemlich alles mit ein paar Sternchen zum kompilieren kriegen, das macht es aber nicht richtig. pMap[0] gibt dir ein uninitialisiertes char *Array und

pMap[11][15] =

    "ÉÍÍÍÍÍ»",
    "º     º",
    "º     º",
    "º     º",
    "º     º",
    "º     º",
    "º     º",
    "º     º",
    "º  @  º",
    "ÈÍÍÍÍÍ¼";.

greift auf den nicht existenten char * pMap[11][15] zu und überschreibt ihn was dir irgendwas kaputt macht.

Eine Möglichkeit das zu reparieren ist das * aus *char pMap[11][15]; zu entfernen und direkt zu initialisieren:

char pMap[11][15] = {
		"ÉÍÍÍÍÍ»",
		"º     º",
		"º     º",
		"º     º",
		"º     º",
		"º     º",
		"º     º",
		"º     º",
		"º  @  º",
		"ÈÍÍÍÍÍ¼"
	};

Dann funktioniert bestimmt auch deine Ausgabe.

oenone

abratar schrieb:

#include <stdlib.h> 
#include "stdlib.h"

Warum zwei mal? Verstehst du den Unterschied? Wenn du C++ machen willst, benutz cstdlib , wenn überhaupt.

abratar

nwp3 schrieb:

Eine Möglichkeit das zu reparieren ist das * aus *char pMap[11][15]; zu entfernen und direkt zu initialisieren:
char pMap[11][15] = {
		"ÉÍÍÍÍÍ»",
		"º     º",
		"º     º",
		"º     º",
		"º     º",
		"º     º",
		"º     º",
		"º     º",
		"º  @  º",
		"ÈÍÍÍÍÍ¼"
	};
Dann funktioniert bestimmt auch deine Ausgabe.

das hatte ich schon versucht .. doch dann weiss ich nicht wie ich auf pMap zugreifen kann .. z.B. "void Player::map()" zeigt mir dann an das es pMap nicht gibt.. wie kann ich das hinbekommen dass er es findet? dachte das geht mit dem * ..

Wutz

Nathan schrieb:

Denn sonst wüsstest du, dass du ein Stringliteral nicht verändern kannst, was du aber in getKey tust.

Der tut kein Stringliteral verändern, wenn den Stern weglässt wie empfohlen.

Wutz

Das ArrayArray kannst du im Konstruktor explizit setzen:

http://ideone.com/KohfMO

abratar

Wutz schrieb:

Das ArrayArray kannst du im Konstruktor explizit setzen:

http://ideone.com/KohfMO

Also erstmal danke für den die hilfe
Aber ehrlich gesagt weiss ich nicht wie ich den code bei mir reinbauen soll

könntest du mir sagen bzw. erklären was ich aus meinem code löschen und was einfügen soll ?

Dein code sieht recht gut aus, nur versteh ich den nicht ganz
was ist z.b. "typedef" ? wieso kann ich da nicht einfach "char" und "struct" ohne "typedef" machen ?

Th69

Hallo,

wenn du einen C++11-fähigen Compiler benutzt, müßte

char map[11][15] =
    {
        "ÉÍÍÍÍÍ»",
        "º     º",
        "º     º",
        "º     º",
        "º     º",
        "º     º",
        "º     º",
        "º     º",
        "º  @  º",
        "ÈÍÍÍÍÍ¼"
    };

im Header (innerhalb deiner Klassendefinition) funktionieren.

Ansonsten wird es etwas komplizierter:

// im Header (innerhalb deiner Klasse)
char map[11][15];

// im Konstruktor
static const char init_map[11][15] =
    {
        "ÉÍÍÍÍÍ»",
        "º     º",
        "º     º",
        "º     º",
        "º     º",
        "º     º",
        "º     º",
        "º     º",
        "º  @  º",
        "ÈÍÍÍÍÍ¼"
    };

// Array kopieren
memcpy(&map[0][0], &init_map[0][0], sizeof(map));

PS: Warum eigentlich [11][15]? Deine Daten sind doch [10][7] groß?

abratar

Th69 schrieb:

// Array kopieren
memcpy(&map[0][0], &init_map[0][0], sizeof(map));
[/cpp]

PS: Warum eigentlich [11][15]? Deine Daten sind doch [10][7] groß?

1. Ich habe Visual Studio 2010.
2. Was genau macht "Array kopieren" ? also was wird da kopiert und wohin ? ist das die ausgabe oder wie?
3. weil der compiler mir sagt dass wenn ich da [11][8] rein tu dass es nicht ausreicht für eine 15... keine ahnung .. sonst kompiliert der auch nicht.. wahrscheinlich zählen die zeichen bei meiner map mehr als nur 1 zeichen oder so ..

Wutz

Gewöhne dir beim Programmdesign an, verschiedene Funktionalitäten in verschiedene Programmteile (bei C++ sinnvollerweise Klassen) zu kapseln, hier also z.B. die Nutzeraktionen und die Ausgabe, und presse nicht alles in eine Klasse.
Dazu ist natürlich ein kompletter Überblick über die Dinge notwendig.
Dann definierst du die Schnittstellen deiner Klassen, instanziierst die Objekte (das geschieht sinnvollerweise in main oder zumindest in ein und derselben Aufrufebene) und arbeitest vorrangig mit den Objekten und deren Schnittstellenmethoden und eben nicht mit Funktionen (Sinn der OOP).

#include <iostream>
#include <cstdlib>
#include <windows.h>

using namespace std;

struct Brett {
  Brett::Brett() {
    x = 3; y = 8; char z[10][8] = {
      "ÉÍÍÍÍÍ»",
      "º     º",
      "º     º",
      "º     º",
      "º     º",
      "º     º",
      "º     º",
      "º     º",
      "º  @  º",
      "ÈÍÍÍÍÍ¼" };

    memcpy(pMap,z,sizeof z);
  }
  char pMap[10][8];
  int x, y;
  void left(){  if (pMap[y][x-1] == ' ')  pMap[y][x--] = ' ', pMap[y][x] = '@'; }
  void right(){ if (pMap[y][x+1] == ' ')  pMap[y][x++] = ' ', pMap[y][x] = '@'; }
  void up(){    if (pMap[y-1][x] == ' ')  pMap[y--][x] = ' ', pMap[y][x] = '@'; }
  void down(){  if (pMap[y+1][x] == ' ')  pMap[y++][x] = ' ', pMap[y][x] = '@'; }

  void sp(int choosecolor)
  {
    SetConsoleTextAttribute(GetStdHandle(STD_OUTPUT_HANDLE), choosecolor);
  }
  void s()
  {
    SetConsoleTextAttribute(GetStdHandle(STD_OUTPUT_HANDLE), 7);
  }

  void print()
  {
    for (int i = 0; i < 10; i++)
    {
      sp(697);
      cout << pMap[i] << "\n";
      s();
    }
    system("pause>nul");
  }
};

class Player
{
  Brett *brett;
  void getKey();
  bool game_running;

  void clearscreen();

public:
  Player(Brett&b){
    game_running = true; brett = &b;
  }

  void run();
};

void Player::run()
{
  while (game_running == true)
  {
    clearscreen();
    brett->print();
    getKey();

  }
}

void Player::getKey()
{
  if (GetAsyncKeyState(VK_LEFT))
  {
    brett->left();
  }
  else
  if (GetAsyncKeyState(VK_RIGHT))
  {
    brett->right();
  }
  else
  if (GetAsyncKeyState(VK_UP))
  {
    brett->up();
  }
  else
  if (GetAsyncKeyState(VK_DOWN))
  {
    brett->down();
  }
  else
  if (GetAsyncKeyState(VK_ESCAPE))
  {
    game_running = false;

    exit(0);
  }

}

void Player::clearscreen()
{
  HANDLE hOut;
  COORD Position;

  hOut = GetStdHandle(STD_OUTPUT_HANDLE);

  Position.X = 0;
  Position.Y = 0;
  SetConsoleCursorPosition(hOut, Position);
}

int main()
{
  Brett brett;
  Player player(brett);
  player.run();
}