Zugriff auf Methode anderer Klasse

[sinoptik]

Hallo Zusammen,

kann mir einer erklären, was ich falsch mache? Und zwar geht es um folgendes. Ich bin gerade dabei ein paar Seleniumtests in Java zu erstellen.
Dabei wiederholen sich einige Methoden. Dies möchte ich verhindern, damit der Code nicht unnötig lang ist.

Ich habe zwei Klassen: class SeleniumMethods (enthält eine Methode, die wiederholt benötigt wird) und class testSearching (enthält Testcode)

package Testcases;

public class testSearching {

	private Selenium selenium;
	
	SeleniumHelper sm = new SeleniumMethods(selenium);


	@BeforeTest
	public void startSelenium() {
		selenium = new SeleniumStarter().getSelenium();
		selenium.start();
	}

	
	@Test
	public void testSearch() throws InterruptedException {
		selenium.open("/");
		selenium.waitForPageToLoad("30000");
		selenium.type("q", "Software");
		selenium.click("//button[@value='submit']");
		
		
		sm.checkTextPresent("Software");
	}	
}

package Testcases;

public class SeleniumMethods {
	
	public Selenium selenium;
	
	public SeleniumMethods (Selenium selenium) {
		this.selenium = selenium;
	}
	
		public void checkTextPresent(String text) throws InterruptedException {
			
			for (int second = 0;; second++) {
				try { selenium.isTextPresent(text); break; } catch (Exception e) {}
				if (second >=30) fail ("timeout"); 
				Thread.sleep(1000);
			}
		}
}

Wenn ich testSearching abspiele kommt folgender Fehler:

FAILED: testSearch
java.lang.AssertionError: timeout
at Testcases.SeleniumMethods.checkTextPresent(SeleniumMethods.java:15)
at Testcases.testSearching.testSearch(testSearching.java:25)


Es ist auch nicht so, dass er diesen Text "Software" nicht findet, denn alles funktioniert bestens, wenn ich diese Methode innerhalb der testSearching-Klasse platziere, also so:

  ......

@Test
	public void testSearch() throws InterruptedException {
		selenium.open("/");
		selenium.waitForPageToLoad("30000");
		selenium.type("q", "Software");
		selenium.click("//button[@value='submit']");
		
		for (int second = 0;; second++) { 
			try { selenium.isTextPresent("Software"); break; } catch (Exception e) {}
			if (second >=10) fail ("timeout");
			Thread.sleep(1000);
		}

Für Ratschläge bedanke ich mich im Voraus.
vg sinoptik

 

[Andi_CH]

SeleniumHelper.java:15

Daher kommt IMO die Exception - was steht denn da?

 

[nrg]

SeleniumHelper.java:15

Daher kommt IMO die Exception - was steht denn da?

if (second >=30) fail ("timeout"); vielleicht ???:L

> was steht denn in fail? (edit: wobei das auch selbsterklärend ist)

 

[nrg]

@TO: ziemlich schwer lesbarer Code..
Nur um auszuschließen, dass du SeleniumMethods nicht mit dem falschen Objekt instanziierst, probier mal:

	 @Test
	    public void testSearch() throws InterruptedException {
	        selenium.open("/");
	        selenium.waitForPageToLoad("30000");
	        selenium.type("q", "Software");
	        selenium.click("//button[@value='submit']");
	        
	        sm = new SeleniumMethods(selenium);
	        sm.checkTextPresent("Software");
	    }[/Java]

 

[sinoptik]

@nrg: Vielen Dank, das ist es. Jetzt funktioniert es.
Noch ne Frage: muss ich jetzt in jeder Methode

 "sm = new SeleniumMethods(selenium)"

benutzen?

 

[nrg]

    private Selenium selenium;
    
    SeleniumHelper sm = new SeleniumMethods(selenium);
 
    @BeforeTest
    public void startSelenium() {
        selenium = new SeleniumStarter().getSelenium();

du deklarierst selenium und sm als Attribute und übergibst dem Konstruktor von SeleniumMethods sofort dein selenium. selenium kann aber zu diesem Zeitpunkt nur null sein. Das Verhalten war mir jetzt auch ungetestet etwas unklar, deshalb meinte ich auch, dass der Code schwer lesbar ist. Hätte auch eher eine NPE erwartet...

ich würde es dann mal so probieren:

    private Selenium selenium;
    private SeleniumHelper sm;

    @BeforeTest
    public void startSelenium() {
        selenium = new SeleniumStarter().getSelenium();
        sm = new SeleniumMethods(selenium)
        selenium.start();
    }
...

edit: aber ein bisschen Fachliteratur zu OO würde ich mir noch gönnen. Wie schon paar mal gesagt, ist der Code imho ein graus. Nicht übel nehmen

 

[new@java]

Hallo!

Wir haben ein ganz ähnliches Problem. Wir wollen, dass in der Klasse Spiel.java eine Methode der Klasse Willie.java aufgerufen wird. Zum Schluss soll in einem JLabel (JLabels befinden sich in der Klasse Willie) die verstrichene Zeit angezeigt werden. Das Problem ist nun: Wenn die Methode aufgerufen wird, bewegt sich der Wurm nicht mehr. Woran liegt das? Und wie kriegt man das hin, dass die Zeit in dem JLabel angezeigt wird? Was fehlt uns? Wir haben echt schon alles Mögliche ausprobiert! Hier unser Quellcode:

 import java.awt.Color;
import java.awt.Font;
import java.awt.FontMetrics;
import java.awt.Graphics;
import java.awt.Image;
import java.awt.Toolkit;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;

import javax.swing.ImageIcon;
import javax.swing.JPanel;
import javax.swing.Timer;

public class Spiel extends JPanel implements ActionListener
{
	Willie zzz;
	
	private final int WIDTH = 300;
	private final int HEIGHT = 300;
	private final int DOT_SIZE = 10;
	private final int ALL_DOTS = 900;
	private final int RAND_POS = 29;
	private final int DELAY = 100;

	private int x[] = new int[ALL_DOTS];
	private int y[] = new int[ALL_DOTS];

	public int score = 0;
	private int dots;
	private int apple_x;
	private int apple_y;
	private int birne_x;
	private int birne_y;
	public int zeit;
	
	
	private long time = 0;

	private boolean left = false;
	private boolean right = true;
	private boolean up = false;
	private boolean down = false;
	private boolean inGame = true;

	private Timer timer;
	private Image ball;
	private Image apple;
	private Image birne;
	private Image head;

	public Spiel(Willie z)
	{
		zzz = z;
		addKeyListener(new TAdapter());
		// Hintergrundfarbe des Spielfelds
		setBackground(Color.YELLOW); 
		// Bilder einbinden, sodass spŠter €pfel, Birnen und der Wurm auf dem Spielfeld erscheinen  
		ImageIcon iid = new ImageIcon(this.getClass().getResource("koerper.png"));
		ball = iid.getImage();

		ImageIcon iia = new ImageIcon(this.getClass().getResource("apfel.png"));
		apple = iia.getImage();

		ImageIcon iib = new ImageIcon(this.getClass().getResource("birne.png"));
		birne = iib.getImage();

		ImageIcon iih = new ImageIcon(this.getClass().getResource("kopf_hoch.png"));
		head = iih.getImage();

		setFocusable(true);
		iniGame(); // ruft die Methode iniGame auf

	}

	public void iniGame()
	{

		dots = 3; //Willie hat zu Beginn 3 Kšrperteile

		for (int z = 0; z < dots; z++)
		{
			x[z] = 50 - z * 10;
			y[z] = 50;
		}

		locateApple();
		locateBirne();

		timer = new Timer(DELAY, this);
		timer.start();
	}


	public void paint1(Graphics g)
	{
		super.paint(g);

		if (inGame)
		{

			g.drawImage(birne, birne_x, birne_y, this);

			for (int z = 0; z < dots; z++)
			{
				if (z == 0)
					g.drawImage(head, x[z], y[z], this);
				else
					g.drawImage(ball, x[z], y[z], this);
			}

			Toolkit.getDefaultToolkit().sync();
			g.dispose();

		} 
		else
		{
			gameOver(g);
		}
	}

	public void paint(Graphics g)
	{
		super.paint(g);

		if (inGame)
		{

			g.drawImage(apple, apple_x, apple_y, this);

			for (int z = 0; z < dots; z++)
			{
				if (z == 0)
					g.drawImage(head, x[z], y[z], this);
				else
					g.drawImage(ball, x[z], y[z], this);
			}

			Toolkit.getDefaultToolkit().sync();
			g.dispose();

		} else
		{
			gameOver(g);
		}
	}

	public void gameOver(Graphics g)
	{
		String msg = "Game Over";
		Font small = new Font("Helvetica", Font.BOLD, 20);
		FontMetrics metr = this.getFontMetrics(small);

		g.setColor(Color.black); // Farbe der Schrift
		g.setFont(small);
		g.drawString(msg, (WIDTH - metr.stringWidth(msg)) / 2, HEIGHT / 2);
	}

	public void checkApple()
	{

		if ((x[0] == apple_x) && (y[0] == apple_y))
		{
			dots++;
			//score += 10;
			locateApple();
		}
	}
	
	//public void Punkte(Snake t)
	//{
	//	Snake ttt;
	//	ttt = t;
		//System.out.println(score);
	
//	}

	public void checkBirne()
	{

		if ((x[0] == birne_x) && (y[0] == birne_y))
		{
			dots++;
			locateBirne();
		}
	}

	public void move()
	{
		
		for (int z = dots; z > 0; z--)
		{
			x[z] = x[(z - 1)];
			y[z] = y[(z - 1)];
		}

		if (left)
		{
			x[0] -= DOT_SIZE;
		}

		if (right)
		{
			x[0] += DOT_SIZE;
		}

		if (up)
		{
			y[0] -= DOT_SIZE;
		}

		if (down)
		{
			y[0] += DOT_SIZE;
		}
	}

	public void checkCollision()
	{

		for (int z = dots; z > 0; z--)
		{

			if ((z > 4) && (x[0] == x[z]) && (y[0] == y[z]))
			{
				inGame = false;
			}
		}

		if (y[0] > HEIGHT)
		{
			inGame = false;
		}

		if (y[0] < 0)
		{
			inGame = false;
		}

		if (x[0] > WIDTH)
		{
			inGame = false;
		}

		if (x[0] < 0)
		{
			inGame = false;
		}
	}

	public void locateApple()
	{
		int r = (int) (Math.random() * RAND_POS);
		apple_x = ((r * DOT_SIZE));
		r = (int) (Math.random() * RAND_POS);
		apple_y = ((r * DOT_SIZE));
	}

	public void locateBirne()
	{
		int r = (int) (Math.random() * RAND_POS);
		birne_x = ((r * DOT_SIZE));
		r = (int) (Math.random() * RAND_POS);
		birne_y = ((r * DOT_SIZE));
	}

	public void actionPerformed(ActionEvent e)
	{

		if (inGame)
		{
			
			
			checkApple();
			checkBirne();
			checkCollision();
			move();
			zeit += DELAY/1000;
			Willie.zeitAktualisieren(zeit);
		}

		repaint();
	}
	//public void zeitAktualisieren(int zeit)
	//{

		//zeit = DELAY / 1000;
		//System.out.println();

	//}
	private class TAdapter extends KeyAdapter
	{

		public void keyPressed(KeyEvent e)
		{

			int key = e.getKeyCode();

			// Wenn auf Tastatur Pfeil nach links gedrŸckt wird und nicht nach rechts, 
			//lŠuft Willi auch nur nach links und nicht nach oben oder nach unten; 
			//und nach rechts schon gar nicht.
			if ((key == KeyEvent.VK_LEFT) && (!right))
			{
				left = true;
				up = false;
				down = false;
			}
			
			// Wenn auf Tastatur Pfeil nach rechts gedrŸckt wird und nicht nach links, 
			//lŠuft Willi auch nur nach rechts und nicht nach oben oder nach unten; 
			//und nach links schon gar nicht.
			if ((key == KeyEvent.VK_RIGHT) && (!left))
			{
				right = true;
				up = false;
				down = false;
			}
			
			// Wenn auf Tastatur Pfeil nach oben gedrŸckt wird und nicht nach unten, 
			//lŠuft Willi auch nur nach oben und nicht nach links oder nach rechts; 
			//und nach unten schon gar nicht.
			if ((key == KeyEvent.VK_UP) && (!down))
			{
				up = true;
				right = false;
				left = false;
			}
			// Wenn auf Tastatur Pfeil nach unten gedrŸckt wird und nicht nach
			// oben, lŠuft Willi auch nur nach unten und nicht nach links oder nach
			// rechts; und nach oben schon gar nicht.
			if ((key == KeyEvent.VK_DOWN) && (!up))
			{
				down = true;
				right = false;
				left = false;
			}
		}
	}
}

 import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.GridLayout;

import javax.swing.JLabel;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class Willie extends JFrame
{
	
	public static JLabel Zeitanzeige;
	
	public Willie()
	{
		
		
		// Neues GridLayout erzeugen
		GridLayout gl = new GridLayout(1, 3);
		// Ein JPanel wird erzeugt und das GridLayout zugewiesen
		JPanel info = new JPanel(gl);
		// Das JPanel wird im BorderLayout platziert
		getContentPane().add(BorderLayout.NORTH, info);
		// Dem JPanel eine Farbe zuweisen
		info.setBackground(Color.white);

		// Neues JLabel fŸr Anzeigen erzeugen
		JLabel Zeitanzeige = new JLabel("Zeit:");
		JLabel Leben = new JLabel("Leben:");
		JLabel Punkte = new JLabel("Punkte:" + "" );
		
		// Dem JPanel die einzelnen JLabel hinzufŸgen
		info.add(Zeitanzeige);
		info.add(Leben);
		info.add(Punkte);

		// Das Spiel in der Mitte platzieren
		getContentPane().add(BorderLayout.CENTER, new Spiel(this));
		setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
		setSize(320, 340);
		setLocationRelativeTo(null);
		setTitle("Willie Wurm");

		setResizable(false);
		setVisible(true);
		
	}
	public static void zeitAktualisieren(int zeit)
	{
		Zeitanzeige.setText(Integer.toString(zeit));
	}
	public static void main(String[] args)
	{
		new Willie();
	}
	
}

Wäre super, wenn uns jemand helfen könnte!!
Danke im Voraus.