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

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

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

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

Q&A

1回答

1224閲覧

Listの中にファイルから読み込んだ数字を格納したい

845Qooyouglt

総合スコア7

Java

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

0グッド

0クリップ

投稿2018/05/31 06:57

Listの中にファイルから読み込んだ数字を格納したい

タイトルに書いてある通りList野中にファイルから読み込んだ数字を格納して
バケットソートで並び替えて、昇順、降順で表示するプログラムを作ったのですが
うまくListの中に入らなくて困っています。
助言、修正の仕方をお聞きしたいです。よろしくお願いします。

該当のソースコード

java

1package Bucket; 2 3import java.util.ArrayList; 4import java.util.Arrays; 5import java.util.Collections; 6import java.util.List; 7import Bucket.FRead; 8 9public class test2 { 10 11 public static Integer[] sort(Integer[] arr, boolean desc) { 12 int n = arr.length; 13 int i; 14 Integer maxValue = null; 15 int exp = 1; // 桁数 16 Integer[] tmp = new Integer[n]; 17 Integer[] hists = new Integer[10]; 18 19 while (true) { 20 Arrays.fill(hists, 0); 21 22 if (maxValue == null) { // 終了条件 23 maxValue = arr[0]; 24 for (int ar : arr) { 25 if (ar > maxValue) 26 maxValue = ar; 27 hists[(ar / exp) % 10]++; // 桁のデータ 28 // 0..9 29 } 30 } else { 31 for (int ar : arr) 32 hists[(ar / exp) % 10]++; // 桁のデータ 33 // 0..9 34 } 35 36 if (desc) {//降順 37 for (i = 9; i > 0; i--) 38 hists[i - 1] += hists[i]; 39 40 } else { // 昇順 41 for (i = 1; i < 10; i++) 42 hists[i] += hists[i - 1]; 43 44 } 45 for (i = n - 1; i >= 0; i--) 46 tmp[--hists[(arr[i] / exp) % 10]] = arr[i]; 47 48 exp *= 10; // 次の桁へ 49 if (maxValue / exp <= 0) 50 return tmp; 51 System.arraycopy(tmp, 0, arr, 0, n); 52 } 53 } 54 55 public static void main(String[] args) { 56 String fn = "c://InputData.txt"; 57 String ret[] = FRead.getLines(fn); 58 String str = ret[0]; 59 System.out.println(ret[0]); 60 61 int[] data = new int[str.length()]; 62 for (int i = 0; i < data.length; ++i) 63 data[i] = 0; 64 int i, j; 65 for (i = 0, j = 0; i < str.length(); i++) { 66 if (str.charAt(i) >= '0' && str.charAt(i) <= '9') { 67 data[j] = 10 * data[j] + str.charAt(i) - '0'; 68 // charAt i番目の数字を取り出す 69 // 10*dataで(例34)なら3取り出して30にして4たす -'0'は0番目から始める 70 // 最後に空白をいれる 71 } else { 72 ++j; 73 } 74 75 } 76 List<Integer> lists = new ArrayList<>(); 77 lists.add(new Integer(data[j])); 78 // ここで数字をリストへ。ここがうまくいってない? 79 80 Collections.shuffle(lists); 81 System.out.println("Unsorted:" + lists); 82 Integer[] results = test2.sort(lists.toArray(new Integer[0]), false); 83 System.out.println("Sorted(asc): " + Arrays.asList(results)); 84 85 System.out.println("\nUnsorted:" + lists); 86 results = test2.sort(lists.toArray(new Integer[0]), true); 87 System.out.println("Sorted(desc): " + Arrays.asList(results)); 88 } 89}

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

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

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

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

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

m.ts10806

2018/05/31 07:07

「うまく動作しない」「うまくいかない」では何が起きているか伝わりません。「○○のようになるはずなのに××となってしまう」のような形で「ゴール」と「現象(エラーメッセージなども含む)」を明確にしてください。https://teratail.com/help/question-tips#questionTips3-4-1
swordone

2018/06/01 04:31

Integer.parseIntやInteger.valueOfは使わない約束ですか?
guest

回答1

0

for (i = 0, j = 0; i < str.length(); i++) {

.....

}
List<Integer> lists = new ArrayList<>();
lists.add(new Integer(data[j]));

これでは、data[] の最後の値しか入らないと思います。
for () の前で、lists の宣言(&初期化)をして、 ++j; の前で、 lists.add() で良いのではないでしょうか? もっともこうすると、data[] は配列の必要が無く、単なる 変数で済むと思います。 (使いまわす)

投稿2018/05/31 13:12

pepperleaf

総合スコア6383

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問