Konverter romertal fra inputfil til arabiske tal

Jeg har noget paa lageret til at konvertere fra romer tal:

public class RomanToArabic {
    private static int decodeChar(char c) {
        switch(c) {
            case 'I':
                return 1;
            case 'V':
                return 5;
            case 'X':
                return 10;
            case 'L':
                return 50;
            case 'C':
                return 100;
            case 'D':
                return 500;
            case 'M':
                return 1000;
            default:
                throw new IllegalArgumentException(c + " is not a roman digit");
        }
    }
    public static int decodeString(String s) {
        int acc = 0;
        int ix = 0;
        while(ix < s.length()) {
            if(ix < s.length() - 1 && decodeChar(s.charAt(ix)) < decodeChar(s.charAt(ix + 1))) {
                acc -= decodeChar(s.charAt(ix));
            } else {
                acc += decodeChar(s.charAt(ix));
            }
            ix++;
        }
        return acc;
    }
    public static void main(String[] args) {
        System.out.println(decodeString("III"));
        System.out.println(decodeString("IV"));
        System.out.println(decodeString("V"));
        System.out.println(decodeString("VI"));
        System.out.println(decodeString("MCMXCIX"));
        System.out.println(decodeString("MMVII"));
    }
}

Saa skal du:
- laese input filen en linie ad gangen
- konvertere fra romertal til normal tal med decodeString ovenfor
- gemme i en ArrayList<Integer>
- sortere denne
- udskrive den til en ny fil

Der ser da fint ud det der.
Må indrømme jeg nogenlunde forstår koden når jeg læser den, kan godt se logikken i det osv., men rammer lidt muren med dit anden indlæg.

Hvordan skal jeg fortælle programmet (din kode) at det fx er "romertal.txt" filen den skal læse fra, og hvordan kun en linie af gangen? (og kan denne fil egentlig bare gemmes som .txt eller skal dette også være java?)
Kan ikke gennemskue med hvad det er jeg skal konvertere? Altså er det "ix" jeg skal kalde?

Sortering? Det skal jeg nok finde ud af, det kan en googlesøgning vel godt svare mig på.

Når jeg udskriver til en ny fil, så er det noget i retning af

out = new FileOutputStream("minfil.txt");

, right? Og kan denne også godt nøjes med at være .txt ?

BufferedReader br = new BufferedReader(new FileReader(filename));
String line;
while((line = br.readLine()) != null) {
  ...
}

er en god maade at laese paa.

PrintWriter pw = new PrintWriter(new FileWriter(newfilename));
for(int i = 0; i < lst.size(); i++) {
    pw.println(lst.get(i));
}
pw.close();

vil skrive ud.

der mangler lige en

br.close();

til sidst i laesningen.

Så RomanToArabic koden så stå i inden i BufferedReader koden?
på "..." 's plads?

Og PrintWriter skal bare være nederest i programmet? Eller?

Sorry, men er som sagt i SÅ meget inde i Java.

Ja. Med en add til en ArrayList<Integer>.

Ja.

Jeg skal ikke importe Scanner her, eller sådan noget sjov?
Så i det hele hvordan skal det se ud?
Og hvor skal jeg så egentlig køre main fra?

Noget i retningen af?

BufferedReader br = new BufferedReader(new FileReader(filename));
String line;
while((line = br.readLine()) != null) {
  ...
}

Hov, kom til at trykke send.


BufferedReader br = new BufferedReader(new FileReader(filename));
String line;
while((line = br.readLine()) != null) {

public class RomanToArabic {
    private static int decodeChar(char c) {
        switch(c) {
            case 'I':
                return 1;
            case 'V':
                return 5;
            case 'X':
                return 10;
            case 'L':
                return 50;
            case 'C':
                return 100;
            case 'D':
                return 500;
            case 'M':
                return 1000;
            default:
                throw new IllegalArgumentException(c + " is not a roman digit");
        }
    }
    public static int decodeString(String s) {
        int acc = 0;
        int ix = 0;
        while(ix < s.length()) {
            if(ix < s.length() - 1 && decodeChar(s.charAt(ix)) < decodeChar(s.charAt(ix + 1))) {
                acc -= decodeChar(s.charAt(ix));
            } else {
                acc += decodeChar(s.charAt(ix));
            }
            ix++;
        }
        return acc;
    }
    public static void main(String[] args) {
        System.out.println(decodeString("III"));
        System.out.println(decodeString("IV"));
        System.out.println(decodeString("V"));
        System.out.println(decodeString("VI"));
        System.out.println(decodeString("MCMXCIX"));
        System.out.println(decodeString("MMVII"));
    }
}
}
br.close();

også skal jeg vel have
public static void main(String[] args)
her?

Hvad mener du med add til en ArrayList? Det skal bare stå til sidst i RomanToArabic klassen eller hvad?

int KonvTal;
KonvTal = new int[10]

noget i den stil?

Kun en klasse nemlig din.

3 metider: decodeChar, decodeString, main

du skal kun kalde decodeString i while loekken ikke have koden der

Okay, må indrømme jeg er faldet lidt af hesten.

Gider du ikke prøve at opsummere lidt, for kan ikke rigtig følge med. Heh, sorry.

en klasse - din

3 metoder

decodeChar - fra min klasse
decodeString - fra min klasse
main - lav ArrayList<Integer>
      laes ind fra fil med while loekke (se kode skitsering)
      sorter
      udskriv til fil med for loekke (se kode skitsering)

Jamen hvordan får jeg så de konverterede tal i min array?
Altså, hvordan skal det se ud til i det hele?

i min main der skal jeg have
ArrayList<Integer>
og hvordan fortæller jeg den at jeg skal gemme de tal der bliver konverteret? Eller er det først efter while løkken?

lst.add(decodeString(line));

skulle tilfoeje et tal til listen.

Så ?

public class RomanToArabic {
    private static int decodeChar(char c) {
        switch(c) {
            case 'I':
                return 1;
            case 'V':
                return 5;
            case 'X':
                return 10;
            case 'L':
                return 50;
            case 'C':
                return 100;
            case 'D':
                return 500;
            case 'M':
                return 1000;
            default:
                throw new IllegalArgumentException(c + " is not a roman digit");
        }
    }
    public static int decodeString(String s) {
        int acc = 0;
        int ix = 0;
        while(ix < s.length()) {
            if(ix < s.length() - 1 && decodeChar(s.charAt(ix)) < decodeChar(s.charAt(ix + 1))) {
                acc -= decodeChar(s.charAt(ix));
            } else {
                acc += decodeChar(s.charAt(ix));
            }
            ix++;
        }
        return acc;
    }
    public static void main(String[] args) {
        System.out.println(decodeString("III"));
        System.out.println(decodeString("IV"));
        System.out.println(decodeString("V"));
        System.out.println(decodeString("VI"));
        System.out.println(decodeString("MCMXCIX"));
        System.out.println(decodeString("MMVII"));
    }
}

public static void main(String[] args){

BufferedReader br = new BufferedReader(new FileReader(RomerTal.txt));
String line;
while((line = br.readLine()) != null) {

    public static int decodeString(String s)

}
br.close()
}


Er det rigtigt indtil videre?
Og hvad skal jeg så?
Forstår stadig ikke hvordan jeg skal implementere det med ArrayList, og hvordan jeg så skal få denne sorteret.

ArrayList<Integer> lst = new ArrayList<Integer>();

laver en ArrayList.

lst.add(decodeString(line));

tilfoejer til den

Så jeg skal erstatte alle dine System.out.println(decodeString())
med BufferRead, også inde i den lst.add(decodeString(1)) ?

Hvor jeg før det selvf. har lavet lst = new ArrayList<Interger>();

du skal slet ikke bruge noget fra min main - den er kun til test

Det giver selvfølgelig en masse fejl. Øv.

En masse "cannot find symbol" ved lst.add og BufferRead og hvad der ellers har med det at gøre.

public class RomanToArabic {
    private static int decodeChar(char c) {
        switch(c) {
            case 'I':
                return 1;
            case 'V':
                return 5;
            case 'X':
                return 10;
            case 'L':
                return 50;
            case 'C':
                return 100;
            case 'D':
                return 500;
            case 'M':
                return 1000;
            default:
                throw new IllegalArgumentException(c + " is not a roman digit");
        }
    }
    public static int decodeString(String s) {
        int acc = 0;
        int ix = 0;
        while(ix < s.length()) {
            if(ix < s.length() - 1 && decodeChar(s.charAt(ix)) <

decodeChar(s.charAt(ix + 1))) {
                acc -= decodeChar(s.charAt(ix));
            } else {
                acc += decodeChar(s.charAt(ix));
            }
            ix++;
        }
        return acc;
    }
    public static void main(String[] args) {

    lst = new ArrayList<Integer>();

        BufferedReader br = new BufferedReader(new FileReader(RomNum.txt));
        String line;
        while((line = br.readLine()) != null) {
        lst.add(decodeString(1));
        lst.add(decodeString(2));
        lst.add(decodeString(3));
        lst.add(decodeString(4));
        lst.add(decodeString(5));
        lst.add(decodeString(6));
        lst.add(decodeString(7));
        lst.add(decodeString(7));
        lst.add(decodeString(8));
        lst.add(decodeString(10));

        }

    }
}


Det er det jeg har prøvet med nu, hvor RomNum.txt ser således ud:
V
X
VII
VI
III
V
IV
IX
II
VII

også får jeg alle de overstående fejl. 7 af dem.

Du skal nok lige importe java.io.* og java.util.* - og saa mangler du ar erklaere lst - og
saa skal du decode line ikke nogle tal.

Hm, det er nok java.util.Scanner jeg skal importere, hvad skulle java.io.* være?

Erklære lst? Hvad betyder/hvordan det?

så jeg skal lst.add(decodeStrin(line)); og kun EN gang så, eller?

Har java.util.Scanner i starten af koden, og får nu en fejl når jeg prøver at compile:

"error: Class names, RomanToArabic, are only acceptet if annotation processing is explicity requested"

?

Do'h, ja selvfølgelig.

Nå, får stadig de fejl omkring BufferReader, også forsætter den ned til lst.add.

Hvordan er det jeg skal erklære lst, hvad betyder det?

Er det ikke det jeg har gjort med lst = new ArrayList ?

21/09-2008 22:38:26

Man kan også løse problemet med recursion. men her er et lille program der efter hvad jeg forstår skulle kunne løse problemet:

import java.io.*;
import java.util.*;

public class RomanToNumbers {

    public static void main(String[] args) throws IOException {
        String inputFile = "c:/in.txt";
        String outputFile = "c:/out.txt";
        FileReader read = new FileReader(inputFile);
        BufferedReader buf = new BufferedReader(read);
        List<Integer> lst = new ArrayList<Integer>();
        String input = buf.readLine();
        do {
            lst.add(decodeString(input));
            input = buf.readLine();
        } while (input != null);
        buf.close();
        read.close();
        FileWriter fw = new FileWriter(outputFile);
        BufferedWriter writer = new BufferedWriter(fw);
        for (Integer number : lst) {
            writer.write(number);
        }
        writer.close();
        fw.close();
    }

    public static int decodeString(String input) {
        if (input == null) {
            throw new NullPointerException("Input cannot be null!");
        }
        if (input.length() == 0) {
            return 0;
        }
        if (input.length() == 1) {
            return decodeChar(input.charAt(0));
        }
        int firstChar = decodeChar(input.charAt(0));
        int secondChar = decodeChar(input.charAt(1));
        int result = 0;
        if (secondChar > firstChar) {
            result = -firstChar;
        } else {
            result = firstChar;
        }
        return result + decodeString(input.substring(1));
    }

    private static int decodeChar(char c) {
        switch (c) {
            case 'I':
                return 1;
            case 'V':
                return 5;
            case 'X':
                return 10;
            case 'L':
                return 50;
            case 'C':
                return 100;
            case 'D':
                return 500;
            case 'M':
                return 1000;
            default:
                throw new IllegalArgumentException(c + " is not a roman digit");
        }
    }
}

Glemte at sortere, men kan gøres ved at indsætte:

Collections.sort(lst);

i linje 21 altså lige før:

FileWriter fw = new FileWriter(outputFile);

og for loopet skal ændres til:

for (Integer number : lst) {
    writer.append(number + "\n");
}

løst

Uden hjaelp??

Nej arne_v undskyld, det var med meget hjælp, primært din, der gjorde at jeg kom igennem det.
Det var faktisk bare for at lukke tråden jeg besvarede, troede du havde glemt alt o denne tråd.
Jeg oprette lige et ekstra spørgsmål, hvor jeg tildeler dig pointene.
http://www.eksperten.dk/spm/916563