Redirect in einer Spring Boot WebApplication

[FantastischMan]

hallo ich habe ein großes problem und zwar geht es um eine kleine web anwendung die ich implementieren möchte und das Formular, das ich absende. Also ich habe ein Formular, welches ich befülle und es abschicke, der controller dieser Klasse nimmt diese entgegen per @postmapping und setzt die Werte. und jetzt kommt das problem wenn ich die Werte redirecte auf die GetMapping methode umso die Werte aneigen zu lassen, wird nix angezeigt.

@Controller
public class AlkBerechnerController {
    @Autowired
    AlkAtomat alkAtomat;

    @GetMapping("/")
    public String text(Model m){
        if(alkAtomat == null){
            alkAtomat = new AlkAtomat();
        }else{
            m.addAttribute("blutwert", ((int)((alkAtomat.getBlutwert())*100))/100.0 );
        }
        m.addAttribute("alkohol", alkAtomat);
        return "index";
    }



    @PostMapping("/alk")
    public String alkoholBerechner( @ModelAttribute AlkAtomat akohol){
        this.alkAtomat = akohol;


        return "redirect:/";
    }
}

 

[FantastischMan]

<h1 style="text-align: center;" th:text="${blutwert} ? 'Dein Blutwert ist '+ ${blutwert} + '‰': 'Hallo Korrektor'">Hallo </h1>
<form method="POST" action="/alk" th:object="${alkohol}">
    <input type="radio" name="geschlecht" id="m" th:value="0" value="männlich" th:field="*{geschlecht}"> <label for="m">Männlich</label>
    <input type="radio" name="geschlecht" id="w" th:value="1" value="weiblich" th:field="*{geschlecht}"> <label for="w">Weiblich</label>
    <label for="gewicht">Gewicht</label>
    <input class="form-control" type="number" id="gewicht" th:field="*{gewicht}" name="gewicht" placeholder="in Kg">
    <input class="form-control" type="number" th:field="*{bier}" id="bier" name="bier" placeholder="in ml">
    <input class="form-control" type="number" th:field="*{wein}" id="wein" name="wein" placeholder="in ml">
    <input class="form-control" type="number" th:field="*{korn}" id="korn" name="korn" placeholder="in ml">
    <input class="form-control" type="number" th:field="*{vodka}" id="vodka" name="vodka" placeholder="in ml">
    <input class="form-control" type="number" th:field="*{whiskey}" id="whiskey" name="whiskey" placeholder="in ml">
    <input style="width: 100%" class="btn btn-outline-success" type="submit" value="Berechne!">
</form>

die index datei befindet sich unter resources template,
wenn ich das formular ausfülle wird und abschicke wird der blutwert nicht angezeigt

 

[kneitzel]

Also ich verstehe Deine Logik nicht.

a) Dein redirect bringt den Browser dazu, den gegebenen Link aufzurufen - ohne irgendwelche Daten mitzugeben.

b) State im Controller? Was soll das bitte bringen? Der Controller ist ein Spring Bean und ohne spezielle Angabe ist dies ein Singleton. Also da irgendwas zu speichern ist relativ Unsinnig - das würden dann ja alle Zugriffe sich teilen!

c) Was ist denn mit Deinem AlkAtomat? Ist es ein Bean? Dann ist das @Autowired verständlich, aber dann solltest Du tunlich nicht selbst einfach Instanzen anlegen. Und dann kommt das auch nicht vom Browser. Daher ist es doch eigentlich eine Entity, oder?

Warum bleibst Du nicht beim postMapping und machst Da die Aufbereitung? Da kannst Du den blutwert berechnen und das model füllen und am ende return "index"; sagen. Was spricht dagegen? Dann hast Du die Daten in der Form gespeichert und alles ist ok.

 

[FantastischMan]

Also ich verstehe Deine Logik nicht.

a) Dein redirect bringt den Browser dazu, den gegebenen Link aufzurufen - ohne irgendwelche Daten mitzugeben.

b) State im Controller? Was soll das bitte bringen? Der Controller ist ein Spring Bean und ohne spezielle Angabe ist dies ein Singleton. Also da irgendwas zu speichern ist relativ Unsinnig - das würden dann ja alle Zugriffe sich teilen!

c) Was ist denn mit Deinem AlkAtomat? Ist es ein Bean? Dann ist das @Autowired verständlich, aber dann solltest Du tunlich nicht selbst einfach Instanzen anlegen. Und dann kommt das auch nicht vom Browser. Daher ist es doch eigentlich eine Entity, oder?

Warum bleibst Du nicht beim postMapping und machst Da die Aufbereitung? Da kannst Du den blutwert berechnen und das model füllen und am ende return "index"; sagen. Was spricht dagegen? Dann hast Du die Daten in der Form gespeichert und alles ist ok.

Ich versteh nicht ganz das Problem mit redirect

 

[kneitzel]

Erkläre doch einfach einmal, was Du überhaupt genau erreichen willst.

Den Punkt, dass man im Controller kein State haben sollte, ist aber erst einmal verständlich?

Aber kommen wir nur auf Deine Kernfrage zurück: Warum wird der Blutwert nicht angezeigt? Vermutlich, weil er 0 ist? Ein 0 Wert würde das nicht triggern. Die korrekte Prüfung, ob ein Wert gesetzt wurde, ist:
th:text="${blutwert != null} ? 'Dein Blutwert ist '+ ${blutwert} + '‰': 'Hallo Korrektor'"

Aber das ändert nichts daran, dass es Unsinn ist, dem Controller ein State zu geben und da dann etwas zu speichern. Mach einfach mal mehrere Browser auf und ruf die Seite auf....

 

[FantastischMan]

Erkläre doch einfach einmal, was Du überhaupt genau erreichen willst.

Den Punkt, dass man im Controller kein State haben sollte, ist aber erst einmal verständlich?

Aber kommen wir nur auf Deine Kernfrage zurück: Warum wird der Blutwert nicht angezeigt? Vermutlich, weil er 0 ist? Ein 0 Wert würde das nicht triggern. Die korrekte Prüfung, ob ein Wert gesetzt wurde, ist:
th:text="${blutwert != null} ? 'Dein Blutwert ist '+ ${blutwert} + '‰': 'Hallo Korrektor'"

Aber das ändert nichts daran, dass es Unsinn ist, dem Controller ein State zu geben und da dann etwas zu speichern. Mach einfach mal mehrere Browser auf und ruf die Seite auf....

Erkläre doch einfach einmal, was Du überhaupt genau erreichen willst.

Den Punkt, dass man im Controller kein State haben sollte, ist aber erst einmal verständlich?

Aber kommen wir nur auf Deine Kernfrage zurück: Warum wird der Blutwert nicht angezeigt? Vermutlich, weil er 0 ist? Ein 0 Wert würde das nicht triggern. Die korrekte Prüfung, ob ein Wert gesetzt wurde, ist:
th:text="${blutwert != null} ? 'Dein Blutwert ist '+ ${blutwert} + '‰': 'Hallo Korrektor'"

Aber das ändert nichts daran, dass es Unsinn ist, dem Controller ein State zu geben und da dann etwas zu speichern. Mach einfach mal mehrere Browser auf und ruf die Seite auf....

Das mit state habe ich verstanden denn Controller sollen keine logiks beinhalten oder Zustände aber das ist ja nicht der Grund wieso es nicht funktioniert. Ich probiere das gleich aus

 

[FantastischMan]

org.thymeleaf.exceptions.TemplateProcessingException: Error during execution of processor 'org.thymeleaf.spring5.processor.SpringInputRadioFieldTagProcessor' (template: "index" - line 13, col 80)



Meine Fehlermeldung sieht so aus nun und mein controller code lautet so

@Controller
public class AlkBerechnerController {
    @Autowired
    AlkAtomat alkAtomat;

    @GetMapping("/")
    public String text(Model m){
        m.addAttribute("alkohol", alkAtomat);
        return "index";
    }



    @PostMapping("/alk")
    public String alkoholBerechner( @ModelAttribute AlkAtomat akohol, Model model){
        //model.addAttribute("alk",akohol);
        model.addAttribute("blutwert",((int)((akohol.getBlutwert())*100))/100.0 );


        return "index";
    }
}

 

[kneitzel]

Dann zeig mal die Klassen und Dateien ganz. Ich würde jetzt z.B. erst einmal schauen, was denn in deinem index Template an Zeile 13 steht (80tes Zeichen).

Ich habe hier lokal folgenden Test gemacht:
index.html:

<html xmlns="http://www.w3.org/1999/xhtml"
      xmlns:th="http://www.thymeleaf.org">
<head><title>Test</title></head>
<body>
<h1 style="text-align: center;" th:text="'Hallo '+ ${nameEntity.name} + '!'"/>
<h1 style="text-align: center;" th:text="${nummer != null} ? 'Nummer: '+ ${nummer} + '!' : 'Keine Nummer!'"/>
<form method="POST" action="/set" th:object="${nameEntity}">
    <input type="text" th:field="*{name}" id="name" name="name" />
    <input type="submit" value="Set Name!" />
</form>
</body></html>

webcontroller.java

package com.javaforum.springdemo;

import org.springframework.stereotype.Controller;
import org.springframework.ui.Model;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.ModelAttribute;
import org.springframework.web.bind.annotation.PostMapping;

@Controller
public class WebController {

    /**
     * Just a state inside a controller - bad idea!
     */
    NameEntity nameEntity = new NameEntity("Welt");

    @GetMapping("/")
    public String index(Model m) {
        if (nameEntity.getName().equals("nr0")) {
            m.addAttribute("nummer", 0);
        } else if(nameEntity.getName().equals("nr1")) {
            m.addAttribute("nummer", 1);
        }

        m.addAttribute("nameEntity", nameEntity);
        return "index";
    }

    @PostMapping("/set")
    public String alkoholBerechner(@ModelAttribute NameEntity nameEntity){
        this.nameEntity = nameEntity;
        return "redirect:/";
    }
}

NameEntity.java

package com.javaforum.springdemo;

public class NameEntity {
    private String name;
    public String getName() { return name; }
    public void setName(String name) { this.name = name; }

    public NameEntity(){}
    public NameEntity(String name){ this.name = name; }
}

Das zeigt die Funktionalität recht schön und ich habe Dein Szenario etwas abgebildet:
- Ich habe eine Entity, deren Werte eingegeben werden können (nur ein Feld name, aber egal).
- Ich setze dann ggf. noch einen weiteren Wert (int). Und zwar nummer . bei name "nr0" auf 0 und bei "nr1" auf 1.
Da kannst Du dann auch ausprobieren, wie es sich bei deinem Check verhält. Wenn ich das != null bei mir raus nehme, dann wird 0 als kein Wert ausgewertet.