4Gewinnt: Spielsteine setzen

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • 4Gewinnt: Spielsteine setzen

    Hi,

    hab folgendes Problem bei meinem 4Gewinnt Programm.
    Ich setze einen Spielstein über die Horizontale und dieser soll rein theoretisch soweit in die Vertikale voranschreiten bis er das Ende des Spielfeldes oder auf einen anderen Spielstein trifft.
    Nun ist es aber, dass wenn er auf einen anderen Spielstein trifft, dass dieser ersetzt wird, was aber nicht sein soll.

    Quellcode

    1. int CGameField::checkVerticalLines(int j){
    2. int i=0;
    3. while(i != 9){
    4. if(GameField[j][i] == 0){
    5. ++i;
    6. }
    7. else{
    8. return --i;
    9. }
    10. };
    11. return i;
    12. }
    Alles anzeigen

    Die freien Spielfelder sind mit 0 gekenntzeichnet.
    Hab ich irgendwo einen Denkfehler?

    cheers
    nobody

    Edit://
    Ich habe herausgefunden, das der Wert der zurückgeben wird, zwar in GameField gespeichert wird, sobald aber die Methode nochmals aufgerufen wird, ist GameField nur mit dem Standardwert 0 belegt und das auch nur in dieser Methode.
    Woran könnte das liegen, das sich die Werte, die sich innerhalb und außerhalb der Methode befinden, unterscheiden?
    "Irren ist menschlich. Aber wer richtigen Mist bauen will, braucht einen Computer."

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von nobody special ()

  • GameField ist in der Headerdatei als private Member deklariert.


    ich hatte nochmal zwischen zeitliche leichte Änderungen vorgenommen.
    MIttlerweile sieht der Code nun so aus.

    Quellcode

    1. int CGameField::checkVerticalLines(int *j){
    2. int i = 0;
    4. while(i != 9)
    5. {
    6. if( GameField[i][*j] == 0){
    7. ++i;
    8. }
    9. else
    10. {
    11. --i;
    12. break;
    13. }
    14. }
    15. return i;
    16. }
    Alles anzeigen


    Das ist die Methode die checkVerticalLines aufruft und das ergebniss in GameField speichert.

    Quellcode

    1. bool CGameField::setChip(int X, int Player){
    3. int Y = checkVerticalLines(&X);
    5. GameField[Y][X] = Player;


    Die Adressen innerhalb der Methode und außerhalb sind identisch.
    "Irren ist menschlich. Aber wer richtigen Mist bauen will, braucht einen Computer."

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von nobody special ()

  • Das Problemliegt an:
    while(i != 9)

    Wenn noch kein Stein gelegt sind kriegst du 9 zurück, dein Feld ist jedoch 8*8 (0..7) groß.
    Du erhälst neben einem falschen Index dann auch eine Speicherverletzung.

    PS: Bennen mal deine Zählvariablen gleich (X=i? Y=j? oder doch nicht Oo)

    Quellcode

    1. int checkVerticalLines(const int j){
    2. int i=0;
    3. for (; i<8; i++)
    4. {
    5. if (GameField[j][i] != 0)
    6. break;
    7. }
    9. return i-1;
    10. }
    12. bool setChip(int j, int Player)
    13. {
    14. int i = checkVerticalLines(j);
    15. if (i>=0)
    16. GameField[j][i] = Player;
    17. else
    18. return false;
    19. return true;
    20. }
    Alles anzeigen


    Kannst ja versuchen das auf dein Algorithmus zu übertragen.


    Mfg Rushh0ur

  • So simpel ist es leider auch nicht....
    Mein 2dArray hat die Größe von 10x10.
    Wenn es 8x8 hätte würde er die unterste Reihe mit steinen ja nicht anzeigen.
    Außerdem würde sicherlich doch ne Fehlermeldung oder Warnung kommen wenn ich
    außerhalb des gültigen Speicherbereiches versuchen würde einen Schreibzugriff zumachen oder?
    Und das hät ich dann auch bemerkt....

    Ausschnitt aus Header

    Quellcode

    1. private: static const int Max = 10;
    2. const int Gamemode;
    3. int Counter;
    4. int GameField[Max][Max];


    Das mit den Zählvariablen muss ich mal ändern.. sieht wirklich nicht schön aus und verwirrt.
    "Irren ist menschlich. Aber wer richtigen Mist bauen will, braucht einen Computer."

  • Hallo,

    ich war in letzter Zeit ziemlich beschäftigt und hatte keine Zeit um mich weiter um mein Problem, welches immer noch besteht, zu kümmern.
    Wie gesagt das selbe Problem wie seit Wochen. Meine Spielsteine auf der untersten Ebene werden von den neuen Spielsteinen ersetzt, anstatt das diese neuen Steine eine Ebene weiter oben platziert werden.
    Ich werde jetzt einfach mal die gesamte Klasse Gamefield posten. Vielleicht versteckt sich ja der Fehler auch irgendwo anderest.
    Ich hoffe ihr könnt mir helfen.

    Quellcode

    1. #include "CGameField.h"
    2. #include <iostream>
    3. CGameField::CGameField() : Gamemode(4),Counter(0)
    4. {}
    6. CGameField::~CGameField()
    7. {
    8. }
    9. CGameField::CGameField(int Mode) : Gamemode(Mode),Counter(0)
    10. {
    11. }
    13. void CGameField::initializeGamefield(){
    15. for (int i = 0; i < Max; ++i){
    16. for (int j = 0; j < Max; ++j){
    17. GameField[i][j] = 0;
    18. }
    19. }
    21. }
    23. int CGameField::checkVerticalLines(int *j){
    24. int i = 0;
    26. while(i != 9)
    27. {
    28. if( GameField[i][*j] == 0){
    29. ++i;
    30. }
    31. else
    32. {
    33. --i;
    34. break;
    35. }
    36. }
    37. return i;
    38. }
    40. bool CGameField::setChip(int j, int Player){
    42. int i = checkVerticalLines(&j);
    44. GameField[i][j] = Player;
    46. ++Counter;
    48. if(Counter == 99){
    49. std::cout<<"Unentschieden\n";
    50. printGamefield();
    51. return true;
    52. }
    53. if(checkHor(&j,&i,&Player)){
    54. std::cout<<"Spieler "<<Player<<" hat gewonnen\n";
    55. printGamefield();
    56. return true;
    57. }
    58. if(checkVer(&j,&i,&Player)){
    59. std::cout<<"Spieler "<<Player<<" hat gewonnen\n";
    60. printGamefield();
    61. return true;
    62. }
    63. if(checkDia(&j,&i,&Player)){
    64. std::cout<<"Spieler "<<Player<<" hat gewonnen\n";
    65. printGamefield();
    66. return true;
    67. }
    68. return false;
    69. }
    73. bool CGameField::checkHor(int *X, int *Y, int *Player){
    75. int Counter = 0;
    76. int n=0;
    78. while(GameField[*Y][*X+n]==*Player){
    79. ++Counter;
    80. ++n;
    81. }
    82. n=1;
    83. while(GameField[*Y][*X-n]==*Player){
    84. ++Counter;
    85. ++n;
    86. }
    87. if(Counter >= Gamemode){return true;}
    88. return false;
    89. }
    91. bool CGameField::checkVer(int *X, int *Y, int *Player){
    93. int Counter = 0;
    94. int n=0;
    96. while(GameField[*Y+n][*X]==*Player){
    97. ++Counter;
    98. ++n;
    99. }
    100. n=1;
    101. while(GameField[*Y-n][*X]==*Player){
    102. ++Counter;
    103. ++n;
    104. }
    105. if(Counter >= Gamemode){return true;}
    106. return false;
    107. }
    109. bool CGameField::checkDia(int *X, int *Y, int *Player){
    111. int Counter = 0;
    112. int n=0;
    114. while(GameField[*Y+n][*X+n]==*Player){
    115. ++Counter;
    116. ++n;
    117. }
    118. n=1;
    119. while(GameField[*Y-n][*X-n]==*Player){
    120. ++Counter;
    121. ++n;
    122. }
    123. if(Counter >= Gamemode){return true;}
    124. return false;
    125. }
    128. void CGameField::printGamefield(){
    130. std::cout<<"0123456789";
    131. std::cout<<"\n------------\n";
    132. for(int i = 0; i< 10;++i){
    133. for (int j = 0; j< 10; ++j){
    134. std::cout<<GameField[i][j];
    135. }
    136. std::cout<<"\n";
    137. }
    138. std::cout<<"\n";
    140. }
    142. int* CGameField::getGameField(){
    144. return *GameField;
    145. }
    Alles anzeigen


    Greetz
    nodody
    "Irren ist menschlich. Aber wer richtigen Mist bauen will, braucht einen Computer."