0

だから私は個人的な理由で小さな小さなアプリを作っています。基本的に、Web サイトから xml を解析するだけで、色、タイトル、および作成者のリストが返され、すべてがリストビューに表示されます。それはある程度機能しますが、たとえば、ここの for ループで常に発生するこれらの indexoutofboundsexceptions が発生することがあります。for ループをもう少し詳しく説明します。結果は配列リストです。リストビューの各行は、タイトル、ユーザー名、色の 3 つの要素で構成されています。for ループは、基本的に行に値を割り当てる配列を反復処理するため、最初の要素はタイトルに割り当てられ、2 番目の要素はユーザー名などに割り当てられます。パターンが繰り返されるだけです。誰かが問題を見つけたり、これを行うためのより良い方法を知っている場合は、共有してください:)

   for (int i = 0; i < result.size(); i++) {
       sr = new SearchResults();
       sr.setTitle(result.get(i));
       i++;
       sr.setUser(result.get(i));
       i++;
       sr.setHex(result.get(i));
       results.add(sr);
   }

エラーは通常、次の行に沿って表示されます。

07-11 14:48:08.272: E/AndroidRuntime(15370): FATAL EXCEPTION: main
07-11 14:48:08.272: E/AndroidRuntime(15370): java.lang.IndexOutOfBoundsException: Invalid index 62, size is 62

コードは次のとおりです。

public class ColorsActivity extends ListActivity {
    /** Called when the activity is first created. */
    private int selection = 0;
    private String info;
    private ArrayList<String> myArr = new ArrayList<String>();
    private int resultOffset=0;
    private TextView settings;
    private SearchResults sr = new SearchResults();
    private ArrayList<SearchResults> results = new ArrayList<SearchResults>();

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);

        settings = (TextView) findViewById(R.id.settings_text);

        final ListView lv = getListView();
        lv.setDividerHeight(0);

        colorTask refTask = new colorTask(); 
        refTask.execute();

        settings.setOnClickListener(new View.OnClickListener() {
            public void onClick(View v) {
                colorTask refTask = new colorTask(); 
                refTask.execute();
            }
        });  
    }

    public class MyCustomBaseAdapter extends BaseAdapter {
        private ArrayList<SearchResults> searchArrayList;

        private LayoutInflater mInflater;

        public MyCustomBaseAdapter(Context context, ArrayList<SearchResults> results) {
            searchArrayList = results;
            mInflater = LayoutInflater.from(context);
        }

        public int getCount() {
            return searchArrayList.size();
        }

        public Object getItem(int position) {
            return searchArrayList.get(position);
        }

        public long getItemId(int position) {
            return position;
        }

        public View getView(int position, View convertView, ViewGroup parent) {
            ViewHolder holder;
            if (convertView == null) {
                convertView = mInflater.inflate(R.layout.list_item, null);
                holder = new ViewHolder();
                holder.txtHex = (TextView) convertView.findViewById(R.id.text1);
                holder.txtTitle = (TextView) convertView.findViewById(R.id.TextView02);
                holder.txtUser = (TextView) convertView.findViewById(R.id.TextView01);
                holder.txtColorValue = (TextView) convertView.findViewById(R.id.TextView03);

                convertView.setTag(holder);
            } else {
                holder = (ViewHolder) convertView.getTag();
            }

            holder.txtTitle.setText(searchArrayList.get(position).getTitle());
            holder.txtUser.setText(searchArrayList.get(position).getUser());
            String c = searchArrayList.get(position).getHex();
            holder.txtHex.setBackgroundColor(Integer.parseInt(c, 16)+0xFF000000);
            holder.txtColorValue.setText(searchArrayList.get(position).getHex());

            return convertView;
        }

        class ViewHolder {
            TextView txtTitle;
            TextView txtUser;
            TextView txtHex;
            TextView txtColorValue;
        }
    }

    class colorTask extends AsyncTask<String, Void, ArrayList<String>> {
        private final ProgressDialog dialog = new ProgressDialog(ColorsActivity.this);

        // can use UI thread here
        protected void onPreExecute() {
           this.dialog.setMessage("Contacting server...");
           this.dialog.show();
        }

        protected ArrayList<String> doInBackground(final String... args) {
            URL url = null;
            ParsedExampleDataSet parsedExampleDataSet = null;
            try {                       
                if (selection == 0) {
                     url = new URL ("http://www.colourlovers.com/api/colors/top?resultOffset=" + resultOffset);
                } 

            } catch (MalformedURLException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            }
                SAXParserFactory spf = SAXParserFactory.newInstance();
                SAXParser sp = null;
                try {
                    sp = spf.newSAXParser();
                } catch (ParserConfigurationException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                } catch (SAXException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }

                /* Get the XMLReader of the SAXParser we created. */
                XMLReader xr = null;
                try {
                    xr = sp.getXMLReader();
                } catch (SAXException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
                /* Create a new ContentHandler and apply it to the XML-Reader*/
                getColors myExampleHandler = new getColors(selection);
                xr.setContentHandler(myExampleHandler);

                /* Parse the xml-data from our URL. */
                try {
                    xr.parse(new InputSource(url.openStream()));
                    parsedExampleDataSet =
                        myExampleHandler.getParsedData();
                    myArr = parsedExampleDataSet.toArrayList();             
                } catch (IOException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                } catch (SAXException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }   
                    return myArr;               
        }

        // can use UI thread here
        protected void onPostExecute(final ArrayList<String> result) {
           if (this.dialog.isShowing()) {
              this.dialog.dismiss();
           }
           if (result != null) {    
               for (int i = 0; i < result.size(); i++) {
                   sr = new SearchResults();
                   sr.setTitle(result.get(i));
                   i++;
                   sr.setUser(result.get(i));
                   i++;
                   sr.setHex(result.get(i));
                   results.add(sr);
               }
               ListView lv = getListView();
               lv.setAdapter(new MyCustomBaseAdapter(ColorsActivity.this, results));
               resultOffset += 20;

           } else {
               Toast.makeText(ColorsActivity.this, "Error contacting server.", Toast.LENGTH_SHORT).show();
           }
        }

     }
    public class SearchResults {
         private String title = "";
         private String hex = "";
         private String user = "";

         public void setTitle(String title) {
          this.title = title;
         }

         public String getTitle() {
          return title;
         }

         public void setHex(String hex) {
          this.hex = hex;
         }

         public String getHex() {
          return hex;
         }
         public void setUser(String user) {
           this.user = user;
         }

         public String getUser() {
           return user;
         }
    }
4

4 に答える 4

2
for (int i = 0; i < result.size(); i++) {
    sr = new SearchResults();
    sr.setTitle(result.get(i));
    i++;
    sr.setUser(result.get(i));
    i++;
    sr.setHex(result.get(i));
    results.add(sr);
}

ここでやみくもにインクリメントiします。これが問題である可能性が最も高いです。問題の根本にたどり着くと、解析しているデータが不完全であったり、正しく解析されていなかったりして、必要な数のパラメーターが常に生成されるとは限らない可能性があります (つまり、タイトル、ユーザー、および色を期待して何かを解析しますが、タイトルとユーザーのみを取得します)。

のサイズListは 62 であるため、解析時に少なくとも 1 つのパラメーター (タイトル、ユーザー、または色) が欠落していることを示しています。すべてが正しく解析された場合List、サイズは 63 になります。

于 2012-07-11T19:08:25.900 に答える
0

i(i ++)を増やすと、配列のサイズが0から繰り返されるため、配列の境界を超えてしまいます。サイズ2に達すると、forループ内の2番目のi++は存在しないインデックスになります。

于 2012-07-11T19:09:03.597 に答える
0

あなたの問題はあなたのforループにあります。ループ内で変数iを増やしています。したがって、サイズが62のときにiが61の場合、サイズが62のときにインデックス62にアクセスしようとして、iも1つ追加します。

于 2012-07-11T19:09:13.403 に答える
0

問題は、存在しない arraylist 内のインデックスにアクセスしようとしていることです。問題は間違いなく for ループ内にあります。

for (int i = 0; i < result.size(); i++) {
   sr = new SearchResults();
   sr.setTitle(result.get(i));
   i++;
   sr.setUser(result.get(i));
   i++;
   sr.setHex(result.get(i));
   results.add(sr);
}

これは、ループ宣言で i をインクリメントするだけであれば問題ありません。問題が発生するのは、他の 2 つの i++ ステートメントのいずれかです。これは、それぞれの後に配列の長さを超えているかどうかを確認していないためです。これは、1 つ以上のパラメータが欠落しているトリプレット (Title、User、Hex) が 1 つ以上存在することを意味するに違いありません。3 つすべてのパラメーターが常に存在する必要がある場合は、配列が正しく設定されていません。

パラメータが一致しない場合にコードをフェールセーフにしたい場合は、配列をインクリメントするたびに i を配列の長さに対してチェックします。つまり、ループ内でさらに 2 回確認します。適切と思われる方法でエラーを修正します (ループから抜け出す、sr の残りの値をダミー データで埋めるなど)。

このようなもの:

for (int i = 0; i < result.size(); i++) {
   sr = new SearchResults();
   sr.setTitle(result.get(i));

   i++;
   if (i == result.size()) {
      sr.setUser("dummy");
   }
   else {
      sr.setUser(result.get(i));
   }

   i++;
   if (i == result.size()) {
      sr.setHex("#000000");
   }
   else {
      sr.setHex(result.get(i));
   }

   results.add(sr);
}
于 2012-07-11T19:21:45.340 に答える