2016-11-14 83 views
0

我是Android编程的初学者,寻找建议/建设性的批评。我正在使用下面附加的代码,以便根据strings.xml文件的内容生成一个三维数组。一旦数组生成后,我有额外的代码(未附加),从数组中选取一定的值并使用它执行一些计算并在文本视图中显示结果。我的方法是好还是有更好的方法来实现我的目标?先谢谢你。strings.xml中的三维数组

这里是strings.xml中的内容物的少量样品(大约有50串阵列,从4至约150的任何地方的项目,用含有10个整数或十进制值的每个项目):

<string-array name="state1"> 
    <item>686, 33, 22, 93, 76, 38, 45, 52, 2, 0.55</item> 
    <item>600, 34, 25, 92, 76, 36, 43, 50, 2, 0.51</item> 
    <item>774, 33, 28, 91, 74, 28, 35, 42, 2, 0.43</item> 
</string-array> 

<string-array name="state2"> 
    <item>20, 52, 23, 57, 53, -19, -13, -6, 1, 90</item> 
    <item>289, 67, -38, 73, 58, -26, -19, -12, 2, 24</item> 
    <item>2156, 68, -35, 66, 53, -36, -29, -22, 2, 77</item> 
    <item>131, 61, -4, 68, 57, -19, -12, -6, 1, 14</item> 
    <item>138, 61, -7, 70, 58, -19, -12, -5, 1, 11</item> 
    <item>194, 61, -9, 71, 58, -23, -16, -9, 1, 13</item> 
    <item>342, 61, -13, 71, 58, -20, -14, -7, 2, 13</item> 
    <item>112, 55, 20, 69, 59, -13, -6, 0, 1, 8</item> 
</string-array> 

这是我的MainActivity.java:

public class MainActivity extends AppCompatActivity { 

    @Override 
    protected void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 
     setContentView(R.layout.activity_main); 
     Resources res = getResources(); 
     int i, j, k; 
     float[][][] weatherData = new float[60][200][10]; 
     for (i = 1; i < 3 ; i++) 
     { 
      int arrayID = res.getIdentifier("state" + i,"array",getPackageName()); 
      String[] stateData = res.getStringArray(arrayID); 
      int m = stateData.length; 
      for (j = 0; j < m ; j++) 
      { 
       String[] dataValues = stateData[j].split(","); 
       int n = dataValues.length; 
       for (k = 0; k < n; k++) 
       { 
        weatherData[i-1][j][k] = Float.parseFloat(dataValues[k]); 
       } 
      } 
     } 
     Log.v("value", ""+weatherData[1][14][9]); 
    } 
} 
+0

,如果你使用的是Java,你需要学习面向对象的东西,比如封装......这将帮助您使代码更易于阅读。 – Lucas

+0

优化技巧:看起来您正在对一些静态数据进行一些计算。相反,您可以将计算结果存储起来,以免再次发生。你可以使用java类来存储这些数组作为常量。 –

+0

@KNeerajLal三维数组中的数据由所有常量值组成,并且包含此代码的活动只会从数组中选取一个值并且只是一次。 – KJG

回答

0

我的建议是不要在这里使用Android字符串数组。它看起来很乱,而且显然不是为使用3D数据而设计的。我宁愿在你的案例中使用纯文本文件。

文件的格式可能是类似的东西x,y,z:value如:

0,0,1:0.5 
0,0,2:2 
1,1,1:12.1 

原始数据,可以被放置在/res/raw/data.txt,并访问是这样的:

InputStream inputStream = context.getResources().openRawResource(R.raw.data); 
BufferedReader reader = new BufferedReader(new InputStreamReader(inputStream)); 
String line; 

while ((line = reader.readLine()) != null) { 
    // process single line here 
} 
+0

我认为一点附加信息可能会有用。在此之前的活动中,用户有机会选择一个国家(使用微调器)。此选择会更新第二个微调控制器,并显示该国家的状态列表。一旦用户选择一个状态,第三个微调控制器将更新为该州的城市列表。我计划为每个国家制作一个3D阵列。根据你的建议,看起来我需要50个原始数据文本文件,其中包含对应于每个州/城市(美国)的数字数据。那是对的吗? – KJG

+0

谢谢!我尝试了这一点,我认为这是一个更好的选择。 – KJG