Java springboot cannot resolve method が解決できない

Question

参考にした記事

こちらの記事を参考にTodoリストを作成していたところ、

package com.example.project_todolist.;

import com.example.project_todolist.TodoItemForm;
import com.example.project_todolist.TodoItemRepository;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.ModelAttribute;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RequestMethod;
import org.springframework.web.bind.annotation.RequestParam;

import java.util.Optional;

@Controller
public class HomeController {

    @Autowired
    TodoItemRepository repository;

    @RequestMapping
    public String index(@ModelAttribute TodoItemForm todoItemForm, @RequestParam("isDone")Optional<Boolean> isDone) {
        todoItemForm.setDone(isDone.isPresent() ? isDone.get() : false);
        todoItemForm.setTodoItems(this.repository.findByDoneOrderByTitleAsc(todoItemForm.isDone()));
        return "index";
    }

    @RequestMapping(value = "/done", method = RequestMethod.POST)
    public String done(@RequestParam("id") long id) {
        Optional<Todo> item = this.repository.findById(id) ;
        item.setDone(true);  //cannot resolve method
        this.repository.save(item);
        return "redirect:/?isDone=false";
    }

    @RequestMapping(value = "/restore", method = RequestMethod.POST)
    public String restore(@RequestParam("id") long id) {
        Optional<Todo> item = this.repository.findById(id);
        item.setDone(false);   //cannot resolve method
        this.repository.save(item);
        return "redirect:/?isDone=true";
    }

    @RequestMapping(value = "/new", method = RequestMethod.POST)
    public String newItem(Todo item) {
        item.setDone(false);
        this.repository.save(item);
        return "redirect:/";
    }

}

item.setDoneの部分でcannot resolve method　setDone(boolen)と出てしまいます。

この問題を解決できないのですが、どのようにすればエラーを解決できますでしょうか？
ご教授よろしくお願いいたします。

Comments

[退会済みユーザー]

とりあえず このコードをコピーするとコンパイルエラーになるので直してね
エラーの箇所を示すのはコメントにしましょう

[pokerStars]

返信遅くなり申し訳ございません。
ご回答ありがとうございます。
エラー部分をコメントに変更いたしました。
一度optionalの見直しをし、findById(id)をgetOne(id)に変更したところエラーを消すことができました。
ご協力ありがとうございました。

Answer 1

Optional にそんなメソッドないです

エラーメッセージも日本語にすれば「メソッドを解決できません」となっています。

エラーメッセージを読みましょう。

Answer 2

itemはItem型ではなく、Optional<Item>型として宣言されています。
asahina1979さんの言う通り、OptionalにsetDoneというメソッドは存在しません。
存在しないメソッドをメソッドのように扱おうとしているため、「メソッドを解決できない」というエラーになります。
やるとすればこうなるかと…

item.ifPresent(i -> i.setDone(false));

ただこれ、このままだと

this.repository.save(item);

が、Optional<Item>をsaveしようとしているのだけど、これは正しいのでしょうか。

Answer 3

Optional<Todo> item

は Optional になっているので、ラップされているTodoの内容を取得しましょう。

item.ifPresent() {
  item.get().
}

で初めてTodoのインスタンスになります。

Comments

[pokerStars]

ご回答ありがとうございます。
一度optionalの見直しをし、findById(id)をgetOne(id)に変更したところエラーを消すことができました。
自身のoptionalについての理解が浅いと感じましたので学習の方を進めて行こうと思います。