If Abfrage nur mit bestimmter Initialisierung?

[Slake_821]

Hallo, habe das Problem zwar gelöst, dennoch will ich verstehen oder wissen warum es nicht bzw anders schon funktioniert hat...

char[][] feld=new char[3][3];

public Standardkonstruktor(){

for(int i=0;i<3;i++){
for(int j=0;j<3;j++){

feld[i][j] = 'F';
feld[0][2]= 'X';
}
}

}

Wenn ich über das klassen array folgendes if drüberlaufen lasse will es das if einfach nicht als true anerkennen obwohl es so ist

if((feld[0][2]=='X') && (feld[0][0]=='F'))
System.out.print("true");
else System.out.print("false");

Da kriege ich jetzt immer false raus, es scheint aber mehr an den = 'F' abfragen zu liegen...
Den Fehler habe ich behoben indem ich im Konstruktor es jetzt so geschrieben habe:

feld[0][0]= 'F';
feld[0][1]= 'F';
//usw...

Wäre sehr nett wenn mir das wer erklären könnte warum eine array mit for schleifen nicht "gleich" initialisiert wird wie wenn man es einzeln macht, danke schonmal

 

[krgewb]

Also bei wird true ausgegeben.

public class Standardkonstruktor {

	char[][] feld = new char[3][3];

	public Standardkonstruktor() {

		for (int i = 0; i < 3; i++) {
			for (int j = 0; j < 3; j++) {

				feld[i][j] = 'F';
				feld[0][2] = 'X';
			}
		}

		if ((feld[0][2] == 'X') && (feld[0][0] == 'F'))
			System.out.print("true");
		else
			System.out.print("false");

	}
}

class Hauptprogramm {
	public static void main(String[] args) {
		new Standardkonstruktor();
	}
}

 

[Slake_821]

Kann es ev auch an der IDE liegen? Ich benutze Netbeans 8.2
Das Problem ist nämlich, dass das Array

feld[][]

in weiterer Folge dynamisch sein soll und da kann ich es ja schlecht "manuell" initialisieren

Komisch...danke trotzdem

 

[Robat]

Kann es ev auch an der IDE liegen? Ich benutze Netbeans 8.2

Nein kann es nicht.
Zeig doch mal deinen gesamten Code wie du ihn hast. Vielleicht hat sich ja doch irgendwo ein kleiner Fehler eingeschlichen

 

[Slake_821]

main ist in einer anderen klasse...aufgerufen wird entweder userStep() mit String der form "x:y:O" oder computerStep

public class TicTacToeModel implements InterTTT{

   
    char[][] feld= new char[3][3];
   
    public TicTacToeModel(){
       
        for (int i=0;i<3;i++) {
            for (int j=0;j<3;j++) {
                feld[i][j] = 'F';
            }
        }
    }
   
    @Override
    public String computerStep() {
        checkResult(WinTieLose());
        String move = CalculateComputerStep();
        StoreStep(move);      

        return move;
    }

    @Override
    public String userStep(String fromUser) {
        StoreStep(fromUser);
        return fromUser;
    }
   
    private String CalculateComputerStep()
    {
        String ret;
       
        ret = Winnow();
        if(ret != null)
        {
            return ret;
        }else
        ret = Blocknow();
        if(ret != null)
        {
            return ret;
        }else
        ret = Createfork();
        if(ret != null)
        {
            return ret;
        }else
        ret = Blockfork();
        if(ret != null)
        {
            return ret;
        }else
        ret = DoElse();
        if(ret != null)
        {
            return ret;
        }else
       
            for(int x=0;x<3;x++)
            {
                for(inty=0;y<3;y++)
                {
                    if(feld[x][y] == 'F')
                    {
                        ret = ""+x+":"+y+":X";
                    }
                }
            }
//hier kommt er im code nicht weiter und sagt mir irgendwas mit runtime exception....
        return ret;
    }
   
private void StoreStep(String str) {
        String part[]=str.split(":");
        this.feld[Integer.valueOf(part[0])][Integer.valueOf(part[1])] = part[2].charAt(0);
    }

}

die methoden Winnow,blocknow, createfork, blockfork, doelse sind nur methoden mit vielen if() wie:

if((feld[0][1] == 'O') && (feld[2][0] == 'O') && feld[1][0] == 'F' && feld[0][2] == 'F' && feld[0][0] == 'F'){
            return "0:0:X";
        }else
//if(...)

wenn keine der if richtig ist, geben die methoden null zurück

 

[Robat]

//hier kommt er im code nicht weiter und sagt mir irgendwas mit runtime exception....

Den Stacktrace solltest du hier mal posten...

 

[Slake_821]

Exception in thread "AWT-EventQueue-0" java.lang.RuntimeException: Uncompilable source code
location: class model.TicTacToeModel
at model.TicTacToeModel.CalculateComputerStep(TicTacToeModel.java:70)
...
zeile 70 ist die mit der doppelten for schleife und folgend dem return
...
at model.TicTacToeModel.computerStep(TicTacToeModel.java:26)
at view.TicTacToeView.SetComputerButton(TicTacToeView.java:205)
at view.TicTacToeView.actionPerformed(TicTacToeView.java:199)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2022)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2348)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:252)
at java.awt.Component.processMouseEvent(Component.java:6539)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3324)
at java.awt.Component.processEvent(Component.java:6304)
at java.awt.Container.processEvent(Container.java:2239)
at java.awt.Component.dispatchEventImpl(Component.java:4889)
at java.awt.Container.dispatchEventImpl(Container.java:2297)
at java.awt.Component.dispatchEvent(Component.java:4711)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4904)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4535)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4476)
at java.awt.Container.dispatchEventImpl(Container.java:2283)
at java.awt.Window.dispatchEventImpl(Window.java:2746)
at java.awt.Component.dispatchEvent(Component.java:4711)
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:760)
at java.awt.EventQueue.access$500(EventQueue.java:97)
at java.awt.EventQueue$3.run(EventQueue.java:709)
at java.awt.EventQueue$3.run(EventQueue.java:703)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:74)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:84)
at java.awt.EventQueue$4.run(EventQueue.java:733)
at java.awt.EventQueue$4.run(EventQueue.java:731)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:74)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:730)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:205)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)

 

[Robat]

for(inty=0;y<3;y++)

In der Zeile hast du zB noch ein Syntax-Error

 

[Slake_821]

Ahhh ok ich glaube ich weiß warum...es ist ja tictactoe und ich habe immer den computer anfangen lassen und da ist er in diese for schleife gefallen und wo ich dann umgeschrieben habe von

for (int i = 0; i < 3; i++) {
            for (int j = 0; j < 3; j++) {

                feld[i][j] = 'F';
                feld[0][2] = 'X';
            }
        }

zu

feld[0][0]= 'F';
feld[0][1]= 'F';
//usw...

habe ich angefangen und da ist er gar nicht in die schleife gekommen, da vorher irgendwas true war....und ich habe geglaubt er kommt nur in die for schleife weil er vorher immer nur null als return gekriegt hat, was dann am if liegen würde

sry für die primitivität, ich hab einfach falsch gedacht :/
danke für die hilfe auf jeden fall