Parsování a check kódu – Java – Fórum – Programujte.com

Je tento kód syntakticky správně?

        String [] s = input.split(" ");
        double firstNumber = Double.parseDouble(s[1]);
        double secondNumber = Double.parseDouble(s[2]);
        String sign = s[0];
        int fact = Integer.parseInt(s[1]);

Jednou tu hodnotu parsuji na double a po druhé na int a u fact, kde používám int hodnotu hlásí error, přítom fact mám správně a nevím, kde jinde by mohla být chyba.

#1 Siggi
Syntakticky to je správně, proto je možné ten program přeložit. Při spuštění do něj zřejmě zadáváš desetinné číslo. Error, který jsi neuvedl, zřejmě vypadá asi takto: 

Exception in thread "main" java.lang.NumberFormatException: For input string: "20.7"
	at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65)
	at java.lang.Integer.parseInt(Integer.java:492)
	at java.lang.Integer.parseInt(Integer.java:527)
	at Jmeno$Test.main(Jmeno.java:9)

V podstatě se to dá vyložit tak, že při parsování hodnoty integer mu vadí ta tečka ve vstupním stringu.

Tests run: 6, Failures: 0, Errors: 2, Skipped: 0, Time elapsed: 0 sec <<< FAILURE!
factorial(BasicCalculatorProcessTest)  Time elapsed: 0 sec  <<< ERROR!
java.lang.ArrayIndexOutOfBoundsException: 2

a 

unknownOperation(BasicCalculatorProcessTest)  Time elapsed: 0 sec  <<< ERROR!
java.lang.ArrayIndexOutOfBoundsException: 1

#3 Siggi
Tak mi ještě prozraď, co máš ve stringu "input". Podle těch chybových hlášek v něm nemáš ani jednu mezeru a přitom potřebuješ alespoň dvě.

#4 Kit 

    public void factorial() {
        String cmd = cmd(Calculator.FAC_CMD, "4");
        assertNumericResult(cmd, 24, getCalc().eval(cmd));

    @Test
    public void sum() {
        String cmd = cmd(Calculator.SUM_CMD, "4", "2");
        assertNumericResult(cmd, 6, getCalc().eval(cmd));
    }

Tzn. input: (FAC_CMD, "4")

atd...

Mohu ti poslat celý balíček, pokud bys chtěl se podívat blíže. Teď je to o dosti větší.

#5 Siggi
Metoda 

input.split(" ");

rozdělí input podle mezer takto: 

s[0] = "(FAC_CMD,";
s[1] = "\"4\")";

Obávám se, že jsi očekával něco jiného.

Edit: Pokud chceš něco vystavit veřejně, můžeš použít služeb http://pastebin.com (pro jeden zdroják) nebo http://gist.github.com a tady použít jen odkaz. Pokud privátně tak také, ale je vhodné se nejprve registrovat, abys s nimi mohl později manipulovat.

#6 Kit
Na všech ostatních jako + - * / atd.. funguje, jen u factoriálu a error msg hlásí tento divný error.

Na jednu hodinu: http://pastebin.com/NM6z08fy

#8 Siggi
Máš tam nějaké závislosti, takže to nemohu přeložit. Nevadí, zkusím věštit z toho, co jsi dodal:

Máš tam konstrukci 

String sign = s[0];
switch(sign) {
    case SUM_CMD: ...

Ve switchi nelze použít String, ale hodně podobně by vypadal zápis s enum: 

SignEnum sign = SignEnum.valueOf(s[0]);
switch(sign) {
    case SUM_CMD: ...

Třídu SignEnum je samozřejmě nutné definovat: 

enum SignEnum = { SUM_CMD, SUB_CMD, MUL_CMD, DIV_CMD, FAC_CMD }

V Javě je možné do toho enumu nadefinovat i chování. Taková kalkulačka pak vypadá docela skvěle a není nutný žádný switch.

#9 Kit
Tím mi akorát přibylo spoustu dalších errorů. :D Předtím to fungovalo i s tím porovnáváním řetězců.

#9 Kit
Méně častou možností, jak řídit tok, je příkaz switch (přepínač). Switch, jak již název napovídá, umožňuje na základě hodnoty proměnné přepínat mezi několika větvemi programu. Proměnná, která složí jako přepínač může být následujících typů: char, byte,short, int. Tento seznam je platný pro Javu 1.6, ve verzi 1.7 by se měl rozšířit o řetězec.

Takže to enum mi dělá akorát bordel.

Problém je, že když přistoupím k factoriálu a nebo chybě, tak se tam snažím vecpat neznámé prvky z inputu a proto ta chyba, teď se ji snažím odstranit. Ale ten kód vypadá strašně hnusně.