mongoose castError とは

Question

JavaScript/mongoDB初心者です。

クエリ情報を更新しようとすると下記エラーとなります。
{"msg":"Cast to Boolean failed for value "on" (type string) at path "shipIsBroken""}

shipIsBrokenはBooleanを指定しているつもりですが、
エラーとなる原因が分かりませんでした。
このエラーは、どういった時に出るものでしょうか、
または、確認すべき場所をご指摘いただけますと幸いです。

const { Schema, model } = require('mongoose');
const logSchema = new Schema({
  title: { type: String, required: true, unique: true },
  entry: { type: String, required: true ,unique: true },
  shipIsBroken: { type: Boolean, required: true },
});


const Log = model('Log', logSchema);
module.exports = Log;

equire('dotenv').config()
const express = require('express');
const app = express();
const PORT = process.env.PORT || 3000;
const Log = require('./models/logs')
/*******
Database Setup
******/
const mongoose = require('mongoose');
mongoose.connect(process.env.MONGO_URI, {
  useNewUrlParser: true,
  useUnifiedTopology: true,
  useCreateIndex: true,
  useFindAndModify: false
})
mongoose.connection.once('open', () => {
  console.log('connected to mongo')
})

app.set('view engine', 'jsx');
app.engine('jsx', require('express-react-views').createEngine())
app.use((req, res, next) => {
  console.log('**********************')
  console.log('***********Middleware checking in***********')
  console.log('I run before all routes')
  console.log('**********************')
  next()
})

app.use(express.urlencoded({ extended: true })) // Without this half my code wont work because i need req.body
const methodOverride = require('method-override');
app.use(methodOverride('_method'));

/*
Index
*/
app.get('/logs/', (req, res) => {
  Log.find({}, (err, foundLogs)=>{
    if(err){
      res.status(404).send({
          msg: err.message
      })
    } else {
      res.render('Index', {
        logs: foundLogs
      })
    }
  })
})

/*
New
*/
app.get('/logs/new', (req, res) => {
  res.render('New')
})

/*
Delete
*/
app.delete('/ logs/:id', (req, res) => {
  Log.findByIdAndDelete(req.params.id, (err, foundLog)=>{
    if(err){
      res.status(404).send({
          msg: err.message
      })
    } else {
      res.redirect('/logs')
    }
  })
})

/*
Put/Update
*/
app.put('/logs/:id', (req, res) => {
  if(req.body.shipIsBroken === 'on'){
    req.body.shipIsBroken = true;
  } else {
    req.body.shipIsBroken = false;
  }
  Log.findByIdAndUpdate(req.params.id, req.body, { new: true }, (err, updatedLog)=>{
    if(err){
      res.status(404).send({
          msg: err.message
      })
    } else {
      res.redirect('Index', {
        log: updatedLog
      })
    }
  })
})


/*
Create
*/
app.post('/logs', (req, res) =>{
  if(req.body.shipIsBroken === 'on'){
    req.body.shipIsBroken = true;
  } else {
    req.body.shipIsBroken = false;
  }
  Log.create(req.body, (err, createdLog ) => {
    if(err){
    res.status(404).send({
      msg: err.message
    })
  } else {
    console.log(createdLog);
    res.redirect('/logs')
  }
})
})

  /*
Edit
*/
app.get('/logs/:id/edit', (req, res) => {
  Log.findById(req.params.id, (err, foundLog)=>{
    if(err){
      res.status(404).send({
          msg: err.message
      })
    } else {
      res.render('Edit', {
        log: foundLog
      })
    }
  })
})

/*
Show
*/
app.get('/logs/:id', (req, res) => {
  Log.findById(req.params.id, (err, foundLog)=>{
    if(err){
      res.status(404).send({
          msg: err.message
      })
    } else {
      res.render('Show', {
        log: foundLog
      })
    }
  })
})

  app.listen(PORT, () => {
  console.log('in the building', PORT)
})

const React = require('react');
class Edit extends React.Component {
  render(){
    return (

        <form method="POST" action={`/logs/${this.props.log._id}?_method=PUT`}>
        title: <input type="text" name="title" defaultValue={this.props.log.title}/><br/>
        entry: <input type="text" name="entry"  defaultValue={this.props.log.entry}/><br/>
        shipIsBroken:
            { this.props.log.shipIsBroken
              ? <input type="checkbox" name="shipIsBroken" defaultChecked />
              : <input type="checkbox" name="shipIsBroken"/>
            }
        <br/>
        <input type="submit" value="Submit Changes"/>
        </form>

    )
  }
}
module.exports = Edit;

Comments

[suama]

なんとなく、キャストが必要というのは掴めるかと思いますが、エラーメッセージの前後の、何行目にエラーが出ているとか、どのアクション（POST? PUT？) でのエラーなのか、可能なら追記いただけますか？

POSTの処理の時かな？とおもうのですが、PUT (Update) の箇所に、readyToEat というスキーマに定義されていないパラメーターも出ているので、ちょっと気になっています。

[hiro_ike]

ご指摘ありがとうございます。
以前のコードのコピペが残っておりました。。。

上記と、ルーティング設定で
else {
res.redirect('Index', {
log: updatedLog
としたかったのですが、

else {
res.render('Show', {
log: updatedLog
とすると、正常に更新できるのですが、
前者だとエラーになる理由ってお分かりになりますでしょうか。

[hiro_ike]

あと、ベストアンサーとさせて頂きたいので、
解決方法にコメントいただけますと助かります。色々ありがとうございます。

[suama]

リダイレクトはURLを転送してIndex 画面に「遷移」するので、log: updateLog を渡してはダメです。あと、findById... で取得できるのは１件で、Index は 複数を期待していますので、この点もエラーの原因ですね。

もう一度お願いなのですが、それぞれエラーになった際のエラーメッセージを、複数行貼っていただけませんでしょうか？

エラーの発生した箇所について、推測でしかお返事できないので...。
リダイレクトのところは回答側にお返事したいと思います。

[hiro_ike]

出ていたエラーは、
(1){"msg":"Cast to Boolean failed for value \"on\" (type string) at path \"shipIsBroken\""}
(2)このサイトにアクセスできません。
上記２件で、ご指摘の箇所を修正して、(1)は解消して、ルーティングを下記に書き換えて(2)も解消となりました。
res.render('Show', {

＞リダイレクトはURLを転送してIndex 画面に「遷移」するので、log: updateLog を渡してはダメです。
なるほど。理解しました。お手数おかけ致しました。

お手隙の際に、回答に入力してもらえたら、ベストアンサーにしますのでどうぞよろしくお願い致します。
ありがとうございました。

[suama]

こんにちは、ひとまず解消とのことで良かったです！
ご自身で解決されたということで、回答欄にはわかった点をご自身で書く、というのも良いと思います。
簡単ですが、回答欄に書いておきますね。

Answer 1

こんにちは。質問のほうでも少し書かせていただきましたが、エラーの発生に関して気になった点をいくつか書きますね。

{"msg":"Cast to Boolean failed for value "on" (type string) at path "shipIsBroken\”"}

こちらは、mongooseを用いてMongoDBに値を登録するときに、shipIsBroken のデータ型に Boolean を期待しているのだけれど String で渡って来ててダメです、といったエラーになりますね。

最初の段階でコードを見ると、POST のところでは

if (req.body.shipIsBroken === 'on') {
  req.body.shipIsBroken = true;
}

と変換されていたので、POST の処理に関しては正しく扱えている気がしました。
そこで、エラーが出たのは POST のところではなく PUT のところかなと思いました。

そのため、質問のところで、以下のように聞いた次第です。

POSTの処理の時かな？とおもうのですが、PUT (Update) の箇所に、readyToEat というスキーマに定義されていないパラメーターも出ているので、ちょっと気になっています。

PUT のところでは、当初 readyToEat に対しての判定が書かれていて、req.body.shipIsBroken は変換をしていないようだったので、ここでエラーかなと思いました。
こちらで合っていますでしょうか？

何度か「エラーになった際のエラーメッセージを、複数行貼っていただけませんでしょうか？」とお願いをしたのは、エラーになったリクエストが POST でなのか、PUT でなのかをはっきりしておきたかったからです。

---

また、コードを修正して、画面を遷移させる場合ですが、リダイレクトはURLを「転送」してIndex 画面に「遷移」するので、log: updateLog を渡してはダメです。

あと、findById... で取得できるのは１件で、Index は 複数を期待していますので、この点もエラーの原因になるかと思います。

render の場合は、URL を変更せずに、処理のあとに対応するテンプレート (Show) を使って画面に結果を表示する、というものになります。

Express側のログと、ブラウザの開発者ツールのネットワークでのリクエストの流れを丁寧に見ていくと、違いがわかってくると思います。

違いを少しずつ確認してみてくださいね。