Question

我将数组Energy []设置为public时出现问题。我需要在Timer_Tick事件中使用该数组。我已经将数组设置为int并试图使它们公开但没有成功。

以下是代码：

using System;
using System.Windows.Forms;
using System.Collections.Generic;
using System.Linq;
using System.Data;
using System.Data.SQLite;

namespace Project
{
    public partial class FrmGame : Form
    {
        private string team;
        private string p;
        private List<DataGridViewRow> selected;
        public FrmPartida(string team, List<DataGridViewRow> selected)
        {
            this.selected = selected;
            InitializeComponent();
            this.team = team;

            this.ProgressBar.Value = 0;
            this.timer1.Interval = 100;
            this.timer1.Enabled = true;


            try
            {
                SQLiteConnection ocon = new SQLiteConnection(Config.stringConnect);
                dataSet1.Clear();
                SQLiteDataAdapter a = new SQLiteDataAdapter("Select * From players WHERE Teamplayers = '" + team + "'", ocon);
                a.Fill(dataSet1, "players");
            }
            catch (SQLiteException)
            {
                MessageBox.Show("There was an error");
            }

            Statistics(dataSet1, time);
        }

        public void Statistics(DataSet dataset, string team)
        {
            SQLiteConnection ocon = new SQLiteConnection(Config.stringConnect);
            DataTable table1 = dataset.Tables[0];

            string[] PlayerName = new string[6];
            int[] Strength = new int[6];
            int[] Energy = new int[6];
            try
            {
                if (tabela1.Rows.Count != 0)
                {
                    players = selected.Select(r => r.Cells[1].Value.ToString()).Take(6).ToArray();
                }
                if (table1.Rows.Count != 0)
                {
                    Strength = selected.Select(r => int.Parse(r.Cells[3].Value.ToString())).Take(6).ToArray();
                }
                if (table1.Rows.Count != 0)
                {
                    Energy = selected.Select(r => int.Parse(r.Cells[3].Value.ToString())).Take(6).ToArray();
                }

                int time = int.Parse(timer1.Interval.ToString());
                int count = int.Parse(ProgressBar.Value.ToString());
                int Strength1 = Strength[0], Strength2 = Strength[1], Strength3 = Strength[2], Strength4 = Strength[3], Strength5 = Strength[4], Strength6 = Strength[5];
                int Energy1 = Energy[0], Energy2 = Energy[1], Energy3 = Energy[2], Energy4 = Energy[3], Energy5 = Energy[4], Energy6 = Energy[5];
                int PlayerStatistics1, PlayerStatistics2, PlayerStatistics3, PlayerStatistics4, PlayerStatistics5, PlayerStatistics6;

                if ((count / 4) == time)
                {
                    for (Energy1 = 100; Energy1 > 1; Energy1--)
                    {
                        PlayerStatistics1 = Strength1 * Energy1 * time;
                    }
                }
                PlayerStatistics1 = Strength1 * Energy1 * time;
                PlayerStatistics2 = Strength1 * Energy1 * time;
                PlayerStatistics3 = Strength1 * Energy1 * time;
                PlayerStatistics4 = Strength1 * Energy1 * time;
                PlayerStatistics5 = Strength1 * Energy1 * time;
                PlayerStatistics6 = Strength1 * Energy1 * time;

                if (this.ProgressBar.Value == 180)
                {
                    MessageBox.Show(""+Energy1+"");
                }
            }
            catch (SQLiteException)
            {
                MessageBox.Show("There was an error");
            }
        }

        public void timer1_Tick(object sender, EventArgs e) 
        {
            if (this.BarraTempo.Value < 180)
            {
                this.BarraTempo.Value++; 
                if (BarraTempo.Value == 180)
                {
                    MessageBox.Show(""+Energy1+"" ); //"Energy1" needs to appear on this messagebox. 
                    FrmBreak f1 = new FrmBreak(this.time, selected);
                    f1.ShowDialog();
                    this.Hide();
                }
            }
            else
            {
                ProgressBar.Enabled = false;
            }
        }    
    }
}

如果有人知道如何解决这个问题，请告诉我。

谢谢，

Gianlucca。

Answer 1

看起来Energy是一个局部变量。局部变量没有访问修饰符，因为只有方法执行才能访问它们。请考虑将其改为字段，例如team和selected。

一些注意事项：遵循命名约定，变量以小写字母开头，例如energy。此外，一旦你把它变成一个字段，如果你只需要在类中使用另一个方法来访问它，就不需要它是private以外的任何东西。

Answer 2

因为您的Energy数组是在函数内声明的，所以只能从该函数内部访问它，即它不存在于public void Statistics之外。

要使数组公开，您需要在类的开头声明它，如下所示：

using System;
using System.Windows.Forms;
using System.Collections.Generic;
using System.Linq;
using System.Data;
using System.Data.SQLite;

namespace Project
{
    public partial class FrmGame : Form
    {
        private string team;
        private string p;
        private List<DataGridViewRow> selected;

        //Declare the array here with the public access modifier***
        public int[] Energy = new int[6];

        public FrmPartida(string team, List<DataGridViewRow> selected)
        {
            //Code removed for clarity
        }

        public void Statistics(DataSet dataset, string team)
        {
            //Code removed for clarity
        }

        public void timer1_Tick(object sender, EventArgs e) 
        {
            //Code removed
        } 
    }
}

那就是说，不应该将其声明为公共字段，而应该使用accessor（“获取”方法）。

如何制作公共阵列？

2 个答案: