Inlägg

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.

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?

@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.

wg.get(wg.size()).charAt(s.length())==s.charAt(0))

Här hämtar du sista elementet från den länkade listan men du använder den inmatade strängens längd och försöker indexera i strängen i listan.
Något sånt här borde fungera.

String l = wg.getLast();
if (l.charAt(l.length()) !=s.charAt(0)) {
...
}

Du skapar din FileWriter inuti while-loopen, så du kommer alltså öppna filen för varje karaktär du skriver. Och eftersom standardläget för FileWriter är att skriva över allt i filen så bör Hello2.txt till slut innehålla endast det sista tecknet från filen som du läser ifrån, vilket troligtvis är ett nyradstecken (och alltså osynligt). Flytta bw till ovanför while-loopen så att ut-filen bara öppnas en gång så kommer det säkert gå bättre.

Du måste självklart ha bw.write(c); innanför while-loopen. Även om du följer en bok skulle jag rekommendera att faktiskt funderar på vad koden gör också, av dina frågor att döma verkar du skriva kod utan att veta varför.

Med Java 8 går detta att beskriva på ett enklare sätt:

public void convertToLowerCase(Path in, Path out) {
        // Finess #1 från Java8: try-with-resources, stänger automatiskt resursen om ett fel uppstår
        // Finess #2: Streams. Data går direkt från infilen till utfilen utan att allting mellanlagras i datorns minne.
        try (Stream<String> lines = Files.lines(in)) {
            // Finess #3: Method references. lines.map(String::toLowerCase) kan även skrivas lines.map(line -> line.toLowerCase())
            Iterable<String> lowercased = lines.map(String::toLowerCase)::iterator;
            Files.write(out, lowercased);
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

Det kan det visst bli, dock kan det bli enklare ur design- och kodarsynvinkel om ett standard bostadshus är isolerad by default, för vi antar väll att bostadshus (där folk ska bo) är isolerade? Men, om det skulle uppstå någon avvikelse av någon anledning, nybyggt, renovering, så går det att ändra parametern via en extra konstruktor. I vissa fall kan det vara jobbigt att behöva sätta alla parametrar varje gång man skapar ett object, ibland kan det finnas betydligt fler, som man dessutom nästan aldrig behöver bry sig om.

@ProgrammeringElev: det låter rätt

Den kan inte ha hur många parametrar som helst helt utan vidare, det måste matcha några av de andra konstruktorerna i Bostadshus och råkar i detta fall matcha Bostadshus(double l, double b, int v, boolean isol).

Dvs, this(l, b, v, true) resolvar till Bostadshus(double l, double b, int v, boolean isol)!

Det är två olika konstruktorer. Du kan instansiera klassen på flera sätt:

new Bostadshus(1.0,1.0,1)

eller

new Bostadshus(1.0,1.0,1,true)

Väljer du den senare metoden har du möjlighet att specificera om det ska vara tilläggsisolerat eller inte, använder du den första konstruktorn kommer den alltid sättas till true.

Edit:
Fördelen är alltså att det blir färre argument som behöver skickas med i de fall där du vet att det ska vara tilläggsisolerat. I många andra språk kan man sätta defaultvärden på parametrar, men i java måste man overloada på det här sättet.

@ProgrammeringElev: I den ena skickar du med en bool som anger ifall huset är isolerat eller ej, i den andra gör man inte det utan förutsätter att det är det (isolerat alltså). Den andra anropar ju den första och skickar med de parametrar den har fått samt då true för isoleringen. Det handlar om att förenkla anropet, du skulle lika gärna kunna använda den första för allt men om merparten av husen som skapas är isolerade så kan det vara skönt att kunna skippa att skicka true på de som är det.

Om vi först tittar på konstruktorn nedan:

Bostadshus(double l, double b, int v)
{
   this(l, b, v, true);
}

Denna skickar vidare l, b, v samt isol=true till ytterligare en av Bostadshus konstruktorer, nämligen

Bostadshus(double l, double b, int v, boolean isol)
{
   super(l,b,v);
   tilläggsisolerat=isol;
}

Här sätts l, b, v till Hus via Huskonstruktorn och isol sätts till Bostadshus via vanlig tilldelning.

Däremot, om du kallar nedanstående direkt så måste du själv sätta parametern isol till true för att få samma effekt, så visst finns det skillnad även om den är liten.

Bostadshus(double l, double b, int v, boolean isol)
{
   super(l,b,v);
   tilläggsisolerat=isol;
}

Edit: too late

Du skapar din FileWriter inuti while-loopen, så du kommer alltså öppna filen för varje karaktär du skriver. Och eftersom standardläget för FileWriter är att skriva över allt i filen så bör Hello2.txt till slut innehålla endast det sista tecknet från filen som du läser ifrån, vilket troligtvis är ett nyradstecken (och alltså osynligt). Flytta bw till ovanför while-loopen så att ut-filen bara öppnas en gång så kommer det säkert gå bättre.

Det verkar funka för mig i alla fall, tog din kod, satte in klassen i katalogen, ändrade code taggen och satte in "" på alla attribut, se bilden.

http://imgur.com/a/bmwDD

Jag står alltså i "test" och kör appletviewer index.html som har applet taggen, och PolyDemo klassen ligger i katalogen Exempel_KAP9

Kontrollera allt emot den lösningen på bilden.

Du får göra lite strategiska utskrifter med System.out.println("...") precis före där felet uppstår. Lämpligtvis skriva ut parametrar och sådant för att se vad som kommer in.

Pröva med "" runt alla html attribut.

<param name="antal" value="7">
<param name="radie" value="100">

Nu var det väldigt länge sedan jag gjorde något med applets men tror den söker efter Exempel_KAP9/PolyDemo.class eftersom du har package Exempel_KAP9

code attributet ska nog vara code="Exempel_KAP9.PolyDemo" och klassen ska ligga i en katalog med namnet Exempel_KAP9