Editを押すと404notfoundになる。

Question

実現したいこと

Edit機能がちゃんとできるようになりたい

発生している問題・分からないこと

Editを押すと404notfoundになる。

エラーメッセージ

404notfound

該当のソースコード

/backend/app/controllers/todos_controller.rb

class TodosController < ApplicationController
    
    def index
        # 日付が新しい順に10件まで取得する
        @todos = Todo.all.order(created_at: :desc).limit(10)
    
        render json: @todos
    
    end
    
    def show
    
        @todo = Todo.find(params[:id])

    render json: @todo
    end

    def create
          
        @todo = Todo.new(todo_params)
      
        if @todo.save
            render json: @todo, status: :created, location: @todo
          else
            render json: @todo.errors, status: :unprocessable_entity
          end
        end
      
        def update
          @todo = Todo.find(params[:id])
      
          if @todo.update(todo_params)
            render json: @todo
          else
            render json: @todo.errors, status: :unprocessable_entity
          end
        end

        def todo_params
          params.require(:todo).permit(:title, :content)
    end
end

/frontend/app/components/EditTodoForm.tsx

"use client";

import React, { useState } from 'react';
import { useRouter } from 'next/navigation';
import axios from 'axios';
import useSWR from 'swr';

type EditTodoFormProps = {
  id: number;
};

const fetcher = (url: string) => axios.get(url).then(res => res.data);

const EditTodoForm = ({ id }: EditTodoFormProps) => {
  const router = useRouter();
  const [title, setTitle] = useState('');
  const [content, setContent] = useState('');
  const [error, setError] = useState('');

  const { data, error: fetchError } = useSWR(id ? `http://localhost:3000/todos/${id}` : null, fetcher);

  // Set initial form values when data is loaded
  React.useEffect(() => {
    if (data) {
      setTitle(data.title);
      setContent(data.content);
    }
  }, [data]);

  const handleSubmit = async (e: React.FormEvent) => {
    e.preventDefault();

    if (!title || !content) {
      setError('Title and content are required');
      return;
    }

    try {
      await axios.put(`http://localhost:3000/todos/${id}`, { todo: { title, content } });
      router.push(`/todos/${id}`);
    } catch (error) {
      setError('Failed to update todo');
      console.error(error);
    }
  };

  if (fetchError) return <div>Failed to load todo</div>;
  if (!data) return <div>Loading...</div>;

  return (
    <div className="space-y-6 py-16">
      <form onSubmit={handleSubmit} className="space-y-6">
        <label className="block text-xl font-bold text-gray-700">Edit Todo</label>
        <input
          type="text"
          value={title}
          onChange={(e) => setTitle(e.target.value)}
          placeholder="タイトル"
          className="block w-full py-2 pl-3 pr-4 text-base border border-gray-300 rounded-md shadow-sm focus:outline-none focus:ring-gray-500 focus:border-gray-500 sm:text-sm"
        />
        <textarea
          value={content}
          onChange={(e) => setContent(e.target.value)}
          placeholder="本文"
          className="mt-1 block w-full border border-gray-300 rounded-md shadow-sm py-2 px-3 focus:outline-none focus:ring-gray-500 focus:border-gray-500 sm:text-sm"
        />
        {error && <p className="text-red-500">{error}</p>}
        <button
          type="submit"
          className="mt-3 ml-auto flex justify-center py-2 px-8 border border-transparent rounded-md shadow-sm text-sm font-medium text-white bg-blue-600 hover:bg-blue-700 focus:outline-none focus:ring-2 focus:ring-offset-2 focus:ring-blue-500"
        >
          Update
        </button>
      </form>
    </div>
  );
};

export default EditTodoForm;

/frontend/app/app/todos/[id]/edit.tsx

"use client";

import { useRouter } from 'next/navigation';
import Link from 'next/link';
import EditTodoForm from '@/components/EditTodoForm';
import useSWR from 'swr';

  const EditTodoPage = ({ params }: { params: { id: string } }) => {
  const router = useRouter();
  const { id } = params;
  // SWR を使ってクエリパラメータがあるか確認します
  const { data: todoId, error } = useSWR(
    id ? `/todos/${id}` : null, 
    () => parseInt(id as string)
  );

  if (error) return <div>Failed to load the page</div>;
  if (!todoId) return <div>Loading...</div>;

  return (
    <div className="flex justify-center items-center">
      <div className="flex flex-col w-3/4 max-w-lg">
        <EditTodoForm id={todoId} />
        <Link
          href="/"
          className="ml-auto font-medium text-blue-600 hover:bg-blue-300 focus:outline-none"
        >
          Back
        </Link>
      </div>
    </div>
  );
};

export default EditTodoPage;

/frontend/app/app/todos/[id]/page.tsx

"use client";

import { useEffect, useState } from 'react';
import { useRouter } from 'next/navigation';
import useSWR from 'swr';
import Link from 'next/link';
import Todo from '@/components/Todo';
import { TodoType } from '@/types/Todo';
import axios from 'axios';

// Fetcher function for SWR
const fetcher = (url: string) => axios.get(url).then(res => res.data);

// Todo詳細ページを表示するコンポーネント
const TodoDetail = ({ params }: { params: { id: string } }) => {
  // ルーティング情報を取得する
  const router = useRouter();
  const { id } = params;

  // SWRを使ってデータを取得する
  const { data: todo, error } = useSWR<TodoType>(id ? `http://localhost:3000/todos/${id}` : null, fetcher);

  // エラーが発生した場合の処理
  if (error) return <div>Failed to load</div>;

  // Todoを取得中の場合は「Loading...」を表示する
  if (!todo) return <div>Loading...</div>;

  return (
    <div className="flex justify-center items-center">
      <div className="flex flex-col space-y-6 w-3/4 max-w-lg pt-10">
        <label className="block text-xl font-bold text-gray-700">Todo</label>
        <div>
          <h2 className="text-2xl font-semibold">{todo.title}</h2>
          <p className="mt-2 text-gray-600">{todo.content}</p>
        </div>
        <div className="flex justify-end">
          <Link
            href={`/todos/${id}/edit`}
            className="mt-auto font-medium text-blue-600 hover:bg-blue-300 focus:outline-none mr-12"
          >
            Edit
          </Link>
          <Link
            href="/"
            className="mt-auto font-medium text-blue-600 hover:bg-blue-300 focus:outline-none"
          >
            Back
          </Link>
        </div>
      </div>
    </div>
  );
};

export default TodoDetail;

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

自分で試したこと

import { useRouter } from 'next/router'→import { useRouter } from 'next/navigation';
修正

  const EditTodoPage = ({ params }: { params: { id: string } }) => {
  const router = useRouter();
  const { id } = params;

に書き換えた。

補足

tree構造

|-- README.md
|-- app
|   |-- favicon.ico
|   |-- globals.css
|   |-- layout.tsx
|   |-- page.tsx
|   `-- todos
|-- components
|   |-- CreateTodoForm.tsx
|   |-- EditTodoForm.tsx
|   |-- Todo.tsx
|   `-- Todos.tsx
|-- next-env.d.ts
|-- next.config.mjs
|-- node_modules

Comments

[dodox86]

マルチポストですね。
[Editがうまくいかない - Qiita/質問] https://qiita.com/gabakugik/questions/bdc5a7f6ca4222f04287

[他のサービスにも質問投稿をしたい（してしまった）- teratail] https://teratail.com/help#posted-otherservice

Answer 1

マルチポストなので閉めます。
すいませんでした。