Spring Boot で IndexOutOfBoundsException

Question

Spring Boot でチャットアプリを作成しています。
/roomsでformのボタンをクリックするとPostメソッドからgetUserByIdメソッドでUsersテーブルからselectするのですが、
レコードが0件だったようでIndexOutOfBoundsExceptionになりました。
まだユーザー認証を作ってないのでダミーのユーザーID1を決め打ちしてます。
ログを貼ろうとしたら字数制限にかかってしまいましたので、必要な場合はお申しつけください。
どのあたりから調べればよいかご指導いただけないでしょうか。

よろしくお願いいたします。

DDL

create table app.users (
  id number generated always as identity primary key,
  name nvarchar2(32) not null,
  password nvarchar2(32) not null);

create table app.rooms (
  id number generated always as identity primary key,
  owner_id number references app.users(id));

create table app.memberships (
  membership_id number generated always as identity primary key,
  room_id number references app.users(id),
  user_id number references app.rooms(id));

create table app.messages (
  id number generated always as identity primary key,
  sender_id number references app.users(id),
  room_id number references app.rooms(id),
  text nvarchar2(256) not null,
  created_at timestamp default systimestamp not null);

usersテーブルの状態
select * from app.users

        ID NAME                             PASSWORD
---------- -------------------------------- --------------------------------
         1 test01                           password

application.yml

spring:
  datasource:
  url: jdbc:oracle:thin:@localhost:1521:chatdb
  username: app
  password: password
  driverClassName: oracle.jdbc.driver.OracleDriver
server:
  port: 80

RoomsController.java

package com.springTalk.web;

import java.util.List;

import javax.servlet.http.HttpServletRequest;
import javax.validation.Valid;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Controller;
import org.springframework.ui.Model;
import org.springframework.validation.BindingResult;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.PostMapping;

import com.springTalk.domain.Message;
import com.springTalk.domain.Room;
import com.springTalk.domain.User;
import com.springTalk.service.MessageService;
import com.springTalk.service.RoomService;
import com.springTalk.service.UserService;

@Controller
public class RoomsController {
  @Autowired
  private UserService userService;
  @Autowired
  private RoomService roomService;
  @Autowired
  private MessageService messageService;

  @GetMapping("/rooms")
  public String roomsGet(Model model) {
    List<Room> rooms = roomService.getRoomsOrderByIdDesc();
    model.addAttribute("room", rooms);
    model.addAttribute("roomsForm", new RoomsForm());
    return "rooms";
  }

  @PostMapping("/rooms")
  public String roomsPost(Model model, @Valid RoomsForm roomsForm, BindingResult bindingResult,
      HttpServletRequest request) {
    if (bindingResult.hasErrors()) {
      List<Room> rooms = roomService.getRoomsOrderByIdDesc();
      model.addAttribute("room", rooms);
      model.addAttribute("roomsForm", new RoomsForm());
      return "rooms";
    }

    // get dummy user
    // ToDo: get user id from request
    final Long OWNER_ID = 1L;
    User user = userService.getUserById(OWNER_ID).get(0);
    Room room = roomService.saveAndFlush(new Room(user));
    messageService.save(new Message(user, room, roomsForm.getText()));
    return "redirect:/rooms";
  }
}

UserService.java

package com.springTalk.service;

import java.util.List;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Service;
import org.springframework.transaction.annotation.Transactional;

import com.springTalk.domain.User;
import com.springTalk.domain.UserRepository;

@Service
public class UserService {

  @Autowired
  private UserRepository repository;

  public List<User> getUserById(Long id) {
    return repository.findById(id);
  }

  public List<User> getUserByNameOrderById(String name) {
    return repository.findByNameOrderById(name);
  }

  @Transactional
  public void save(User user) {
    repository.save(user);
  }
}

UserRepository.java

package com.springTalk.domain;

import java.util.List;

import org.springframework.data.jpa.repository.JpaRepository;
import org.springframework.stereotype.Repository;

@Repository
public interface UserRepository extends JpaRepository<User, Long> {
  List<User> findById(Long id);

  List<User> findByNameOrderById(String name);
}

User.java

package com.springTalk.domain;

import java.util.List;

import javax.persistence.CascadeType;
import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.FetchType;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.OneToMany;
import javax.persistence.Table;

@Entity
@Table(name = "app.users")
public class User {
  @Id
  @GeneratedValue(strategy = GenerationType.AUTO)
  private Long id;

  @Column(nullable = false)
  private String name;

  @Column(nullable = false)
  private String password;

  @OneToMany(mappedBy = "ownerId", fetch = FetchType.LAZY, cascade = CascadeType.ALL)
  private List<Room> rooms;

  @OneToMany(mappedBy = "userId", fetch = FetchType.LAZY, cascade = CascadeType.ALL)
  private List<Membership> memberships;

  @OneToMany(mappedBy = "senderId", fetch = FetchType.LAZY, cascade = CascadeType.ALL)
  private List<Message> messages;

  // JPA requirement
  protected User() {
  }

  public User(String name, String password) {
    this.name = name;
    this.password = password;
  }

  public Long getId() {
    return id;
  }

  public String getName() {
    return name;
  }

  public void setName(String name) {
    this.name = name;
  }

  public String getPassword() {
    return password;
  }

  public void setPassword(String password) {
    this.password = password;
  }
}

環境
Spring Boot 1.5.9 RELEASE
Oracle VM VirtualBox 5.1.22 r115126 (Qt5.6.2)
CentOS 7.3.1611
openjdk version "1.8.0_141"
Oracle Database 12c Release 2

Answer 1

RoomsController の roomsPost メソッドにて、

User user = userService.getUserById(OWNER_ID).get(0);

と書かれていますが、ここで1件も取得できないのであれば get(0)で指定しても取得できず IndexOutOfBoundsException がスローされてしまうでしょうか。

userService.getUserById した後に要素があるのかどうかをチェックして値を格納すべきでしょう。

Comments

[退会済みユーザー]

ありがとうございます。
ただ、この場合はID=1のレコードが1件返される想定なのですが、これに関してはなぜなのでしょうか？

[A-pZ]

実際テーブルにIDが1のレコードが含まれていないか、JPAの設定に何らかの抜けがあるのだと思います。
ほかにも　Userクラスのidに対し、ミューテータ(setメソッド)がありません。

[退会済みユーザー]

ありがとうございます。
解決しました。
ちなみにidは自動生成なのでsetterはつけていません。

Answer 2

OracleではなくH2に接続していたようです。
application.ymlのインデントが誤っていました。
また別事象でトラブルに遭っていますが、別途質問したいと思います。
ご覧いただきありがとうございました。

application.yml

spring:
  datasource:
    url: jdbc:oracle:thin:@localhost:1521:chatdb
    username: app
    password: password
    driverClassName: oracle.jdbc.driver.OracleDriver
server:
  port: 80