標準入力された数字の絶対値の合計を表示するプログラム（java）について

やりたいこと

最初に数字１つが標準入力され、その数字の数分だけ新たに数字が標準入力されます。
それらの数字の列の前後の差の絶対値の合計を表示したいです。

例）
4
1
5
6
7

合計＝|1-5|+|5-6|+|6-7|=6
■■な機能を実装中に以下のエラーメッセージが発生しました。

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

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 3 out of bounds for length 3
at Main.main(Main.java:16)

該当のソースコード

import java.util.*;

public class Main {
public static void main(String[] args) {
// 自分の得意な言語で
// Let's チャレンジ！！
Scanner sc = new Scanner(System.in);
int n = sc.nextInt();

    for(int i=0;i<=n;i++){
        int log[]= new int[n];
        log[i] = sc.nextInt();
        
        int x =Math.abs(log[i]-log[i+1]);
        
        int sum=0;
        
        sum=sum+x;
        
        System.out.println(sum);
                
        }
    }
}

試したこと

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

1T2R3M4

2021/02/14 06:39

paizaの問題でしょうか。問題ページのリンク貼れますか。

user-_.80

2021/02/14 13:06

Cランク問題のこちらになります。よろしくお願いいたします。 https://paiza.jp/challenges/231/show

行動規範の内容に同意します

回答2件

ベストアンサー

端的にいえば、iが0～nまで回すなら

int x =Math.abs(log[i]-log[i+1]);

の部分は

int x =Math.abs(log[i-1]-log[i]);

にすればいい。
iが0の場合は、当然 -1のインデックスにアクセスできないので、if文で0以外って条件つければ良い。

あと、

for(int i=0;i<=n;i++){

ｎ回のループをするときに＝を含めちゃダメです。
ｎが４の時に、
０、１、２、３、４だと、５回ループするので例外が発生します
※ｎ＋１で、初期化すれば、たしかに例外は回避しますが、配列の最初のindexが０か１かって
基本的なところなので、０で慣れておいた方がいいです。

ループの中でnew int[n]すると、

int log[]= new int[n];

せっかく保存した

log[i] = sc.nextInt();

の部分が、ループをまわすたびにnew で初期化されるので、期待した結果にならないはずです。
たとえば、log[0] = 1と設定したあとに、log[1] = 5 としても、
log[0]　は、再度、初期化されてるので、１でなく０になります。

あと、

int sum=0;

も、ループの中で０にすると最後の部分しか加算されないです。

投稿2021/02/14 11:40

編集2021/02/14 11:44

momon-ga

総合スコア4826

エラーメッセージのとおりで、int log[]= new int[3];として宣言された配列は、2までしか添え字を使えません。

投稿2021/02/14 06:21

maisumakun

総合スコア145975

user-_.80

2021/02/14 06:44

maisumakun さん迅速なご回答ありがとうございました。ご指摘頂いた点を[n]→[n+1]としてもう一度出力したところ今度は下記のエラーメッセージが出ました。何度も質問してしまい申し訳ありませんが分かりましたらご回答お願いいたします。 Exception in thread "main" java.util.NoSuchElementException at java.base/java.util.Scanner.throwFor(Scanner.java:937) at java.base/java.util.Scanner.next(Scanner.java:1594) at java.base/java.util.Scanner.nextInt(Scanner.java:2258) at java.base/java.util.Scanner.nextInt(Scanner.java:2212) at Main.main(Main.java:14) Exception in thread "main" java.util.NoSuchElementException at java.base/java.util.Scanner.throwFor(Scanner.java:937) at java.base/java.util.Scanner.next(Scanner.java:1594) at java.base/java.util.Scanner.nextInt(Scanner.java:2258) at java.base/java.util.Scanner.nextInt(Scanner.java:2212) at Main.main(Main.java:14)