我从一个网站的表中获取了我正在创建的android中的总统民意调查应用程序的值。我将四个列存储在四个不同的arraylists中。当我尝试使用arraylist.get()
获取值并将其存储到数组中时,arraylist.get()
会返回null
。我在ArrayList
上显示了ListView
,并显示了所有值,因此我知道值已正确存储。
如果您查看我正在使用的网站,它会包含轮询开始时的所有轮询数据。该数组将包含8个连续的民意调查,我将其平均,然后在图表上绘图。我将采取八个民意调查中第一个的开始日期和八个民意调查中最后一个的结束日期,并在两者之间准确找到日期,并将该日期用作我的投票点的x坐标。然后我将对接下来的8个民意调查做同样的事情。这就是为什么我首先检查网站上的民意调查总数是否可以被8整除。如果网站上的民意调查数不能被8整除,我会转到其他地方,然后减去余数。循环完成后,我将单独平均剩余的。
这是我的代码:
package com.pollapp.presidentialelectionpolls;
import java.io.IOException;
import java.text.DateFormat;
import java.text.SimpleDateFormat;
import java.util.ArrayList;
import java.util.Date;
import org.jsoup.Jsoup;
import org.jsoup.nodes.Document;
import org.jsoup.nodes.Element;
import org.jsoup.select.Elements;
import android.net.ConnectivityManager;
import android.net.NetworkInfo;
import android.os.Bundle;
import android.os.StrictMode;
import android.annotation.SuppressLint;
import android.app.Activity;
import android.content.Context;
import android.view.Menu;
import android.widget.Toast;
@SuppressLint("NewApi")
public class MainActivity extends Activity {
Document doc;
Element table;
Elements rows;
Element row;
Elements cols;
Elements link;
int remainderNum = 0, i = 0;
String startingDate, endingDate;
Date start, end;
DateFormat df = new SimpleDateFormat("MM/dd");
String[][] average = new String[8][3];
ArrayList<String> nameOfSrc = new ArrayList<String>();
ArrayList<String> dateWidth = new ArrayList<String>();
ArrayList<String> Trump = new ArrayList<String>();
ArrayList<String> Clinton = new ArrayList<String>();
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);
StrictMode.ThreadPolicy policy = new StrictMode.ThreadPolicy.Builder()
.permitAll().build();
StrictMode.setThreadPolicy(policy);
try {
ConnectivityManager cm = (ConnectivityManager) this
.getSystemService(Context.CONNECTIVITY_SERVICE);
NetworkInfo ni = cm.getActiveNetworkInfo();
Boolean isConnect = ni == null ? false : ni
.isConnectedOrConnecting();
if (isConnect) {
doc = Jsoup
.connect(
"http://www.realclearpolitics.com/epolls/2016/president/us/general_election_trump_vs_clinton-5491.html")
.get();
table = doc.select("table.data.large").get(1);
rows = table.select("tr");
}
} catch (IOException ioe) {
ioe.printStackTrace();
}
for (int i = 2; i < rows.size(); i++) {
row = rows.get(i);
cols = row.select("td");
link = cols.get(0).select("a");
nameOfSrc.add(link.get(0).text());
dateWidth.add(cols.get(1).text());
Clinton.add(cols.get(4).text());
Trump.add(cols.get(5).text());
}
if ((dateWidth.size() % 8) == 0) {
for (i = (dateWidth.size()-1); i <= 7; i = i-8) {
average[i][0] = dateWidth.get(i);
average[i][1] = Clinton.get(i);
average[i][2] = Trump.get(i);
average[i-1][0] = dateWidth.get(i-1);
average[i-1][1] = Clinton.get(i-1);
average[i-1][2] = Trump.get(i-1);
average[i-2][0] = dateWidth.get(i-2);
average[i-2][1] = Clinton.get(i-2);
average[i-2][2] = Trump.get(i-2);
average[i-3][0] = dateWidth.get(i-3);
average[i-3][1] = Clinton.get(i-3);
average[i-3][2] = Trump.get(i-3);
average[i-4][0] = dateWidth.get(i-4);
average[i-4][1] = Clinton.get(i-4);
average[i-4][2] = Trump.get(i-4);
average[i-5][0] = dateWidth.get(i-5);
average[i-5][1] = Clinton.get(i-5);
average[i-5][2] = Trump.get(i-5);
average[i-6][0] = dateWidth.get(i-6);
average[i-6][1] = Clinton.get(i-6);
average[i-6][2] = Trump.get(i-6);
average[i-7][0] = dateWidth.get(i-7);
average[i-7][1] = Clinton.get(i-7);
average[i-7][2] = Trump.get(i-7);
startingDate = average[i][0].substring(0, average[i][0].indexOf(" "));
endingDate = average[i-7][0].substring(average[i-7][0].lastIndexOf(" ") + 1);
}
} else {
remainderNum = (dateWidth.size() % 8);
for (i = (dateWidth.size() - 1); i <= (remainderNum + 7); i = i-8) {
average[i][0] = dateWidth.get(i);
average[i][1] = Clinton.get(i);
average[i][2] = Trump.get(i);
average[i-1][0] = dateWidth.get(i-1);
average[i-1][1] = Clinton.get(i-1);
average[i-1][2] = Trump.get(i-1);
average[i-2][0] = dateWidth.get(i-2);
average[i-2][1] = Clinton.get(i-2);
average[i-2][2] = Trump.get(i-2);
average[i-3][0] = dateWidth.get(i-3);
average[i-3][1] = Clinton.get(i-3);
average[i-3][2] = Trump.get(i-3);
average[i-4][0] = dateWidth.get(i-4);
average[i-4][1] = Clinton.get(i-4);
average[i-4][2] = Trump.get(i-4);
average[i-5][0] = dateWidth.get(i-5);
average[i-5][1] = Clinton.get(i-5);
average[i-5][2] = Trump.get(i-5);
average[i-6][0] = dateWidth.get(i-6);
average[i-6][1] = Clinton.get(i-6);
average[i-6][2] = Trump.get(i-6);
average[i-7][0] = dateWidth.get(i-7);
average[i-7][1] = Clinton.get(i-7);
average[i-7][2] = Trump.get(i-7);
startingDate = average[i][0].substring(0, average[i][0].indexOf(" "));
endingDate = average[i-7][0].substring(average[i-7][0].lastIndexOf(" ") + 1);
}
}
Toast.makeText(getApplicationContext(), startingDate + ", " + endingDate, Toast.LENGTH_SHORT).show();
}
@Override
public boolean onCreateOptionsMenu(Menu menu) {
// Inflate the menu; this adds items to the action bar if it is present.
getMenuInflater().inflate(R.menu.main, menu);
return true;
}
}
答案 0 :(得分:3)
我想给出一个不同的答案。您的真正的问题是您的代码缺乏合理的抽象。
含义:您的程序是关于“候选人”的“数据”。但是你在许多不同的列表中都拥有所有这些信息。那是完全错误的。
相反,您可以创建一个类 Candidate ;候选人有一个名字,可能还有与他有关的数据点。
换句话说:你应该努力摆脱“低级程序”编程风格,其中索引1表示“克林顿的数据点”,2表示“特朗普的数据点”。
相反,你必须得到该候选人类的实例;并将数据点推入它们。这样做可能会让您摆脱很多的代码;从而使您更容易处理您有兴趣解决的真实问题。
长话短说:您的代码没有使用合理的抽象;因此,阅读,理解和发现其中的错误是如此“抽象”。