質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.49%

  • JavaScript

    16461questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • jQuery

    6720questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

JavaScriptでのナビボタンの処理でエラーがました。

受付中

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 353

nenechi

score 2

 前提・実現したいこと

「jQuery テクニカルノート 矢次悟郎」という参考書についてです。
複数画像をクロスフェードで切り替えるスライドショーの機能を実装中に以下のエラーメッセージが発生しました。エラーが出ているのはjavascriptです。

 発生している問題・エラーメッセージ

エラーメッセージ
function declaration should not be plyed in blocks.use a function expression or move the statement to the top of the outer function.

 該当のソースコード

function switchNext(){
                    var setActive = pagination.find('.pnActive'),
                        pnIndex   = pnPoint.index(setActive),
                        pnNum     = pnIndex + 1;

                    if(pnCount === pnNum){
                        pnFirst.click();
                    }    else {
                        setActive.next('a').click();
                    }
                }

function switchPrev(){
                    var setActive = pagination.find('.pnActive'),
                        pnIndex   = pnPoint.index(this),
                        pnNum     = pnIndex + 1;
                    if(1 === pnNum){
                        pnLast.click();
                    }    else {
                        setActive.prev('a').click();
                    }
                }

                btnNext.click(function(){switchNext();});
                btnPrev.click(function(){switchPrev();});
            }
<!doctype html>
<html>
<head>
<meta charset="utf-8">
<title>架空旅館サイト</title>
<link rel="stylesheet" href="style.css">
<link rel="stylesheet" href="https://fonts.googleapis.com/css?family=Bitter:400;700">
<script type="text/javascript" src="js/jquery-1.11.3.min.js"></script>
<script src="js/script.js"></script>

</head>

<body>

<header>
    <div class="logo">
        <a href="">
            <h1>架空旅館サイト</h1>
        </a>
    </div>

    <nav>
        <ul>
            <li></li>
            <li><a href="">客室</a></li>
            <li><a href="">お食事</a></li>
            <li><a href="">船外体験</a></li>
            <li><a href="">ブログ</a></li>
            <li><a href="">アクセス</a></li>
        </ul>
    </nav>
</header>

<div class="slideshow">
    <ul>
        <li><img src="img/img01.png" alt=""></li>
        <li><img src="img/img02.png" alt=""></li>
        <li><img src="img/img03.png" alt=""></li>
    </ul>
</div>


</body>
</html>
.slideshow ul {
    width: 100%;
}

.slideshow ul li {
    top: 150px;
    left: 0;
    width: 100%;
    display: none;
    position: absolute;
}

.slideshow ul li img {
    width: 100%;
}

@media only screen and (max-width: 1020px) {
    .slideshow {
        width: 100%;
    }
}

.pagiNation {
    bottom: 20px;
    left: 0;
    width: 100%;
    height: 15px;
    text-align: center;
    position: absolute;
    z-index: 110;
}

.pagiNation a {
    margin: 0;
    width: 15px;
    height: 15px;
    display: inline-block;
    overflow: hidden;
    background: #aaa;
    border: #fff 2px solid;
    border-radius: 50%;
    transition: all 0.2s ease-in-out;
}

.pagiNation a:hover {
    opacity: 0.6;
}

.pagiNation a.pnActive {
    background: #000;
}

.pagiNation a.pnActive:hover {
    opacity: 1;
}

.slideshow .btnPrev,
.slideshow .btnNext {
    margin-top: -25px;
    top: 50%;
    width: 50px;
    height: 50px;
    position: absolute;
    z-index: 105;
    transition: all 0.2s ease-in-out;
}

.slideshow .btnPrev {
    left: 15px;
    background: transparent url(../img/btnPrev.png) 
    no-repeat center center;
}

.slideshow .btnNext {
    right: 15px;
    background: transparent url(../img/btnNext.png) 
    no-repeat center center;
}

.slideshow .btnPrev:hover,
.slideshow .btnNext:hover {
    opacity: 0.6;
}

 試したこと

ネットでエラーメッセージを調べましたが、日本語の参考になるサイトがほとんどなく、私がJSについて初心者なので意味がいまいちわかりません。
参考書のサンプルコードも上記のコードと同じで、エラーがでています。jQueryも参考書とバージョンを合わせています。何かしらのバージョンの問題なのかもしれませんが、お助け下さい。

他にもエラーがあり、調べてみてif(1 == pnNum)をif(1 === pnNum)に直しました。
これで正しいか正直自身はありませんがエラーはなくなりました。

 補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。
jquery-1.11.3.min.js
dreamweaver CC 2017を使用

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • nenechi

    2018/05/22 23:57

    ご指摘ありがとうございます。html、cssも記載致しました。

    キャンセル

  • mts10806

    2018/05/23 06:54

    「日本語の参考になるサイトがほとんどなく」とのことですが海外サイトはそれっぽいものがあったということでしょうか?

    キャンセル

  • nenechi

    2018/05/24 05:05

    検索に的確にヒットしているかわかりかねますので、それっぽい回答かもわかりませんでした。

    キャンセル

回答 1

+1

例示いただいたスクリプトはケツの「}」が文法エラーになります
かりにそれを取り除くとbtnNext、btnPrevが定義されなくてエラーになります。
かりに定義されていてもclickメソッドを実行してコールバックを返しているのはおかしいです
前提としてjQueryを利用しているようにもみえません

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/24 05:02

    このpartはプラグインを使わないで実装してみる問題でしたのでjQueryは使用していないみたいです、すみません。ご指摘頂いた事を注意して一から読み直し、検証してみます。
    ご回答ありがとうございました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • JavaScript

    16461questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • jQuery

    6720questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。