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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Q&A

解決済

2回答

479閲覧

iteratorの実装・自然数の性の約数を小さいものから返す。

b_b_b

総合スコア32

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

0グッド

0クリップ

投稿2017/10/04 16:21

###前提・実現したいこと
{@link Iterator<Integer>} インターフェースを実装して、
{@link DivisorIteration#next()} を呼び出すごとに,
コンストラクタで与えた自然数の正の約数を小さいものから返していくクラスを作りたい。

next()が最後に返す数は、コンストラクタで与えた数になる。

next()はInteger型を返す。

上記の条件に沿って必要なコードをご教授お願い致します。

###DivisorIteration.Java

Java

1public class DivisorIteration implements Iterator<Integer> { 2 private int number; 3 private int currentNum; 4 5 public DivisorIteration(int number) { 6 if (number <= 0) { 7 throw new IllegalArgumentException("コンストラクタに与える整数は 0 以上でなければなりません。"); 8 } 9 this.number = number; 10 this.currentNum = 1; 11 while (hasNext()) { 12 next(); 13 currentNum++; 14 } 15 } 16 17 // @Override 18 public boolean hasNext() { 19 // TODO Auto-generated method stub 20 if (number >= currentNum) { 21 return true; 22 } else { 23 return false; 24 } 25 } 26 27 // @Override 28 public Integer next() { 29 // TODO Auto-generated method stub 30 while (number % currentNum != 0) { 31 currentNum++; 32 } 33 return currentNum; 34 } 35}

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

currentNumが1の時点で、nextを何回読んでもwhileに入れないため、インクリメントが起こりません。
nextのreturn時にインクリメントするのが単純な解決法でしょうか。

投稿2017/10/04 16:35

swordone

総合スコア20649

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

DivisorIteration.java

java

1import java.util.Iterator; 2 3public class DivisorIteration implements Iterator<Integer> { 4 private int number; 5 private int currentNum = 0; 6 7 public DivisorIteration(int number) { 8 if (number <= 0) { 9 throw new IllegalArgumentException("コンストラクタに与える整数は 0 以上でなければなりません。"); 10 } 11 this.number = number; 12 } 13 14 // @Override 15 public boolean hasNext() { 16 return (currentNum < number); 17 } 18 19 // @Override 20 public Integer next() { 21 do { 22 currentNum++; 23 } while (number % currentNum != 0); 24 return currentNum; 25 } 26} 27

DivSample.java

java

1import java.util.Iterator; 2 3public class DivSample { 4 5 public static void main(String[] args) { 6 for( int num = 1; num <= 10; num++) { 7 Iterator<Integer> iterator = new DivisorIteration(num); 8 System.out.format("%4d:", num); 9 while (iterator.hasNext()) { 10 Integer div = (Integer) iterator.next(); 11 System.out.print(" " + div); 12 } 13 System.out.println(); 14 } 15 } 16} 17

実行例:
イメージ説明

投稿2017/10/05 15:04

katoy

総合スコア22324

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

swordone

2017/10/05 15:13

一応next()ですでに最大要素を出してしまった場合はNoSuchElementExceptionをスローしたほうがいいのかなと
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問