Java ゼロパディング処理について

Question

###前提・実現したいこと
XMLファイルから取得したデータ(文字列の数)が指定の桁数に満たない場合
前に0をつけ指定の桁数になるような処理を記述したいです。
###発生している問題・エラーメッセージ

エラー等は出ていませんが、指定桁数を3として取得した文字列を"0"とすると
結果は000としたいのですが今のところ0としか表示されていません。
for文を使用したゼロパディングの例がなかなか見つからず躓いています。
宜しくお願いいたします。

###該当のソースコード

    //前0処理用メソッド
    //引数で取得したString型の文字列と指定桁数を受け取る
	static String padding(String ck, int n){
		String wk;
		int digit;
		
		wk = ck;
		digit = n;
        //データの文字数を取得
		int len = wk.length();
		
　　　  //指定桁数から取得した文字数を引いてその分文字列の前に0を結合
		for(int i=0; i >= digit - len; i++){
			wk = "0" + wk;
			
		}
		return wk;
	}

Answer 1

原因

for文の条件部が成立しないように見えます。

for(int i=0; i >= digit - len; i++){
wk = "0" + wk;
}

digit - lenは正数ですよね？条件部はループを続ける条件です。

ついでに

次の部分はかなり冗長です。

String wk;
int digit;
wk = ck;
digit = n;

Javaでは引数が値渡しされるうえに、Stringのインスタンスはイミュータブルです。
メソッド内でwkとnをモロに弄んでも危険性はまずないです。

書くなら

以下のように書いた方が簡潔な気がします。

static String padding(String src, int n) {
    return String.format("%0"+n+"d", Integer.valueOf(src));
}

Comments

[umyu]

コードの指摘事項が3点あります。
1,Integer.valueOfの場合、int型の範囲を超えるとjava.lang.NumberFormatExceptionの例外が発生します。桁数的には10桁の数値(2147483648)で発生するので、Long.valueOf(src)の方が良いのではないでしょうか。
2,マイナス数値の考慮
符号がマイナスの数値に対してパディング処理を行った時にマイナス記号の分文字の桁数が増える事の考慮が必要な時もあるかと。
3,メソッド名(padding)についてですが、padStart/padEndのようにどちらの方向からパディング処理を行うのかをメソッド名で表したほうが好みです。
以降はテストソースです、ご参考まで
import java.util.stream.Stream;

public class Q107686 {
public static void main(String[] args) {
Stream<String> stream = Stream.of(String.valueOf(Integer.MIN_VALUE),
String.valueOf(Integer.MAX_VALUE),
String.valueOf(Long.MIN_VALUE),
String.valueOf(Long.MAX_VALUE));
for (String str : stream.toArray(s -> new String[s])) {
String pads = padding(str, 30);
System.out.println(pads);
}
}

static String padding(String src, int n) {
return String.format("%0" + n + "d", Integer.valueOf(src));
}
}

[退会済みユーザー]

NumberFormatExceptionをきにするなら

DecimalFormat format = new DecimalFormat("00000");
format.setMinimumFractionDigits(0);
format.setMaximumFractionDigits(0);

BigDecimal number = new BigDecimal("600.01");
System.out.println(format.format(number));

BigDecimal ＋ DecimalFormat じゃろ

[LouiS0616]

umyuさん、詳細なご指摘勉強になります。
asahina1979さんもコメントありがとうございます。

1. 質問のケースが高々3桁なので考えていませんでした。巨大な数を扱う場合、asahina1979さんのご紹介されている方法が使われるようですね。 https://qiita.com/Co_Pilot/items/6d6321c2f8a0425d6722

2. こちらは少し検討したのですが、やはり質問のコードでは考慮されていないようなのでスルーしました。符号を桁数に含めるか？というのは状況に依りますしね。

3. 変えた方が丁寧かもしれませんね。ただし、ゼロの場合右側に埋めると意味が変わってしまいますので、padEndは考慮しなくて良く、padStartではなくpadZeroなどの方が良いかと。

[umyu]

>LouiS0616さん
コメント欄への質問の回答ありがとうございました。メソッド名はpadZeroの方が適切だと思いました。
>asahina1979さん
勉強になりますー。

[minty]

皆様ありがとうございました。
参考にさせていただき引き続き勉強していきたいと思います。

Answer 2

javaはよく存じ上げないのですが仕組みとしては
左から"000"（1文字以上が確定なら"00"）を足して右から3桁切り出せばよいかと。
forで回すというのはかなり違和感があります。

Comments

[退会済みユーザー]

Java なら各種フォーマットクラス
PHP,C,PERLや同派生言語なら sprintf があるのじゃよ・ｗ・

[LouiS0616]

例えばPythonだとこの方法でもいい ("0"*n + ck)[-n:] んですけれど、Javaだと少し書きづらいですね。

[退会済みユーザー]

# commons.lan3
StringUtils.substring(StringUtils.repeat("0", n) + x, 0-n)

たしかに、標準ではないからなぁ

[sousuke]

フォーマットクラスあるならそれでいいですね。substringは左からしか切れないんですね。
左右反転できるならreverseして右から0埋めして左からn切ってreverse…やっぱり上の方がいいですね。