Java IndexOutOfBoundsException

@ProgrammeringElev: Om du ännu har problem så tänk på att både listor och strings använder nollindexering (d.v.s. precis som array så är först värdet på plats 0, inte 1).

Säg att din lista har två värden: "han", "näsa"
Då finns "han" på plats 0, och "näsa" på plats 1
list.size() returnerar 2.
Försöker du ta list.get(list.size()) får du IndexOutOfBounds, för inget element finns på plats 2. Du bör därför ta list.get(list.size()-1) för att få sista. Precis samma sak gäller för en String, someString.charAt(someString.length()-1) för att få sista.

Vet inte om metoden getLast() du pratar om hör till List eller String, men den löser säkert problemet för den åtminstone.

Vet inte om du fått svar men här är mina spontana kommentarer(ursäkta om jag låter vresig ):

Du undrade varför dina breaks inte fungerade? Det är för att du försöker bryta dig ur if statementet efter att du redan visat dialogen. Break fyller ingen funktion här.

I dina if's kollar du även alltid om i>0 och ja det är det ju alltid så länge loopen körs, så det behöver du inte kolla.

Du har en konstruktor som aldrig kallas vad vi kan se, så den kan du ta bort.

Jag förstår inte vad for loopen är till för, den loopar 1000 gånger? Till vilken mening?

IndexOutOfBounds kastas när du, mycket riktigt, försöker använda ett index som inte ligger i intervallet. Tänk på nollindexeringen.

Förstår inte riktigt varför du ska lagra input strängen i en lista, blir det inte onödigt komplext när du egentligen bara ska jämföra två strängar åt gången?

Rent allmänt är koden ganska svårläst, försök skriv om den så blir det enklare att felsöka. Gör inte för mycket på samma rad, har tydligare variabel namn etc.

Här är mitt försök:

import javax.swing.JFrame;
import javax.swing.JOptionPane;
import javax.swing.JTextArea;

public class WordGame extends JFrame {
	public static JTextArea tA = new JTextArea(100, 100);

	public static void main(String[] args) {
		String first = "";
		String second = "";
		while (true) {
			// Input
			first = JOptionPane.showInputDialog("Skriv första ordet:");
			second = JOptionPane.showInputDialog("Skriv andra ordet:");

			if (first == null || second == null) {
				JOptionPane.showMessageDialog(tA, "Du måste ange två ord, spelet är slut!");
				break;
			}

			// Error checking
			if (!first.matches("[a-zA-Z]+") || !second.matches("[a-zA-Z]+"))
				JOptionPane.showMessageDialog(tA, "Ord kan bestå bara av bokstäver!");

			else if (first.equals(second))
				JOptionPane.showMessageDialog(tA, "Du kan inte mata in samma ord två gånger!");

			else if (first.charAt(first.length() - 1) != second.charAt(0))
				JOptionPane.showMessageDialog(tA,
						"Ordets första bokstav måste ha samma bokstav som förra ordets sista bokstav");
			else
				JOptionPane.showMessageDialog(tA,
						"Yay, ordet " + first + " sista bokstav matchar ordet " + second + " första bokstav!");
		}
	}
}

Har inte testat så noga, men tror den gör det du vill.

@ProgrammeringElev: Som nämnts tidigare i tråden är det säkert för att du använder s.length. Är du säkert på att det är s du skall använda?

Break-statements har ingen effekt på if-satser, han försöker bryta sig ur sin for-loop vilket han också gör, men eftersom han hade en onödig yttre while-loop i början så fortsatte programmet ändå.

i > 0 är falskt under första iterationen, och gör så att ingen jämförelse görs då, så det fyller sin funktion.

Han nämner i inlägget att det skall spelas tills man har matat in 1000 ord.

Det stämmer att du behöver spara undan använda ord i någon sorts lista om ett ord ska vara tillåtet en gång som mest.

Du måste tänka efter lite nu, gå igenom den felande raden och tänk på vad varje del får för värde.

Du sa att felet kommer i följande rad:
if (s.charAt(0)== wg.get(wg.size()-1).charAt(s.length()-1))

Säg att wg innehåller orden ['hej', 'test']. Du har just matat in ordet 'butik', som lagras i s.
Vilket värde får då s.charAt(0)? Vad får we.get(wg.size()-1)? Vad får s.length()-1? Vad får wg.get(wg.size()-1).charAt(s.length()-1)?

Att du lagrar ordet i en temporär variabel är helt korrekt. Du kan också lagra wg.get(wg.size()-1) i en temporär variabel om du vill, så kanske felet kommer fram.