Minesweeper-App Button setText

TDS95 · 2. Juli 2012

Hallo,

erstelle zurzeit ein kleines Minesweeper Spiel.
Die einzelnen Felder stelle ich mittels Buttons dar und in einem Array speichere ich welcher Button eine Bombe ist und welcher nicht.
Jetzt möchte ich, nachdem ein Button angeklickt wurde und er keine Bombe ist zählen wie viele Bomben um ihn herum versteckt sind
und die Zahl der Bomben als Text auf dem Button ausgeben. Das wollte ich eigentlich so machen:

Code

public void bb0() {
		if (bomb[1] == true) {
			bb0++;
		} else if (bomb[6] == true) {
			bb0++;
		} else if (bomb[7] == true) {
			bb0++;
		}
		String bb0s = Integer.toString(bb0);
		button0.setText(bb0s);
	}

Alles anzeigen

Das mit dem Zählen ist ja kein Problem nur die Zahl dann als Text auf button0 ausgeben zu lassen, damit habe ich Probleme.
Wollte den Integer Wert von bb0 eigentlich mit

Code

String bb0s = Integer.toString(bb0);

in einen String umwandeln und dann mit

Code

button0.setText(bb0s);

Als Text auf dem Button ausgeben lassen. Hier erhalte ich aber den Fehler "button0 cannot be resolved".

Hat jemand eine Lösung für mich?
Vielen Dank im Vorraus

titus · 2. Juli 2012

Hi TDS95,

wie bindest du denn den Button in dein Layout ein?

mfg titus

TDS95 · 2. Juli 2012

Mit XML:

Code

&lt;Button
    	android:id="@+id/button0"
    	android:layout_width="wrap_content"
    	android:layout_height="wrap_content"
    	android:layout_alignBaseline="@+id/button1"
    	android:layout_alignBottom="@+id/button1"
    	android:layout_alignParentLeft="true"
    	android:text="@string/unPressed" /&gt;

und:

Java

final Button button0 = (Button) findViewById(R.id.button0);
    	button0.setOnClickListener(new View.OnClickListener() {
			
			public void onClick(View v) {
				if (bomb[0] == true) {
					lost();
				} else {
					button0.setBackgroundColor(Color.RED);
					// bb0();
				}
			}
		});

Alles anzeigen

in der onCreate()-Methode. (Die Methode bb0() steht nicht in onCreate() sondern außerhalb)

titus · 2. Juli 2012

Hi TDS95;

das Problem ist die Sichtbarkeit der Variabel button0.
Da sie nur in der Methode onCreate bekannt ist(sichtbar) kannst du Sie innerhalb der bb0() nicht nutzen.

Verstehst du?

mfg Titus

TDS95 · 2. Juli 2012

Ja, und wie kann ich das ändern?
Wie kann ich den Button global machen?

Lg

titus · 2. Juli 2012

Hi TDS95,

na in dem du ihn als Klassenvariable setzt.

Also so:

Java

public class deineKlasse extends Activity{
   
   private Button button0;


   @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.dein_layout);
       
         button0 = (Button) findViewById(R.id.button0);




  
    }
}

Alles anzeigen

TDS95 · 2. Juli 2012

Vielen Dank titus, dank deiner Hilfe funktioniert jetzt fast alles.

Mein nächstes Problem:

Java

public void bb0() {
    	if (bomb[1] == true) {
    		bb0++;
    	} else if (bomb[6] == true) {
    		bb0++;
    	} else if (bomb[7] == true) {
    		bb0++;
    	}
    	String bb0s = Integer.toString(bb0);
    	button0 = (Button) findViewById(R.id.button1);
    	button0.setText(bb0s);
    	
    }

Alles anzeigen

Damit möchte ich eigentlich die Bomben um den gedrückten Button herum zählen und als Text ausgeben lassen.
Nur irgendwie ändert sich der Text nicht.
Hat jemand eine Idee?
Kann das überhaupt so funktionieren?

Vielen Dank im Vorraus

block_ · 3. Juli 2012

Das sollte so eigentlich funktionieren, aber die Anzahl der Bomben sollte ja auf dem gedrückten Button stehen, du ladest vorher jedoch einen anderen:

Code

button0 = (Button) findViewById(R.id.button1);

Diese Zeile ist unnötig.

Hast du den Aufruf der bb0-Methode wieder drin? Weiter oben ist sie bei dir nämlich noch auskommentiert.

Zudem solltest du dir vllt überlegen, wie man die Überprüfung dynamisch gestalten kann. Wenn ich mich nicht irre sieht das bei dir bisher so aus:

und dein bb0-Methode ist nur für den Button an Position 0, wo jeweils die benachbarten Felder 1, 6 und 7 auf Bomben geprüft werden. D.h. du hast sehr viele Methoden, für jeden Button jeweils eine Methode, was recht unschön ist.

block_

Minesweeper-App Button setText

Jetzt mitmachen!

Teilen