stack = java.lang.StackOverflowError：スタックサイズ8MB

Question

このStackOverFlowエラーが発生しましたが、完全に理解していますが、大きなデータを扱っていないという問題があります。

私は、Framelayoutの断片、3つのオプションを持っています。

これらのオプションの1つをクリックすると、フラグメントが再作成され、乱数が入力されます.MAXは15です。その大きさではないため、このエラーは、このオーバーフローを引き起こすオプション。

これは「強化」するためのアイデアを生成するためのコードですか？私はこのコードがmomeryを使用するための悪い習慣であるかどうかわかりません。

private static List<Integer> SavedNumbers; 

public static void SetupSavedNumbersLIst(){ 
    SavedNumbers = new ArrayList<>(); 
} 


static List<Integer> range; 
private static void AddDiff(int mMAX){ 
    range = new ArrayList<>(); 
    for(int i = 0 ; i < mMAX ; i++){ 
     range.add(i); 
    } 

    range.removeAll(SavedNumbers); 
} 

private static int ReturnIfDuplic(int mMAX){ 
    AddDiff(mMAX); 
    return new Random().nextInt(range.size()); 
} 


public static int ReturnUniqueSavedNumber(int mMAX){ 
    int Random = ReturnRandom(mMAX); 
    if(SavedNumbers != null && SavedNumbers.size() > 0) { 
     if(DoesSavedNumberExist(Random)){ 
      return ReturnIfDuplic(mMAX); 
     } else { 
      SavedNumbers.add(Random); 
      return Random; 
     } 
    } else if (SavedNumbers != null && SavedNumbers.size() == 0){ 
     SavedNumbers.add(Random); 
     return Random; 
    } else if(SavedNumbers == null){ 
     SetupSavedNumbersLIst(); 
     return ReturnUniqueSavedNumber(mMAX); 
    } else { 
     return 1; 
    } 
} 

private static boolean DoesSavedNumberExist(int Number){ 
    for(int s: SavedNumbers){ 
     if(Number == s) 
      return true; 
    } 
    return false; 
} 

private static int ReturnRandom(int mMAX){ 
    return new Random().nextInt(mMAX); 
} 

Answer 1

あなたがランダムにすでに保存されている同じ番号が起こる場合は、この行は再帰さ：すでに1-15を保存している、とあなたの最高は15で、これはスタックを保証する場合

if(DoesSavedNumberExist(Random)){ 
    return ReturnUniqueSavedNumber(mMAX); 
} 

を-overflowはif文を決して満たしません。これは一般的に「乱数はいいえ、もう一度やり直してください」という悪いアプローチです。なぜなら、より多くの数を追加するにつれて、より多くの再帰を追加するからです。より良い方法は、乱数を、選択可能な数値にインデックスとして使用させることです。

Answer 2

同じ問題が発生しました。スタックサイズ8MB。私がアイテムを追加した時、apiは同じエラーを受けました。私はそれが編集テキストから価値を得る時に起こっていると思った。私はこれを使用して

inIT() 
sys = input_Systolic.getText().toString(); 
       dys = input_Diastolic.getText().toString(); 
       _date = dateView.getText().toString(); 
       tim = time.getText().toString(); 

レトロフィットコール

HashMap<String, Object> pMap = new HashMap<>(); 
     pMap.put("user_id",loginStatusModal.getUser_id()); 
     pMap.put("relative_id",rel_id); 
     pMap.put("systolic",sys); 
     pMap.put("diastolic",dys); 
     pMap.put("report_date",_date); 
     pMap.put("report_time",tim); 

を私の方法を変更し、サービスコールの前に私のinIT();の値を取得するように、私は私のスタックサイズ8メガバイトのエラーを解決します。