我认为您在这里混淆了两种不同的资源;您可能确实需要为每个登录的玩家打开一个线程(实际上是一个进程),但这并不意味着您应该为每个玩家不断打开与数据库的连接,因为数据库连接、文件句柄等内容不受管理资源,应该在你用完它们后立即释放。
C#线程here有很好的介绍和解释。
您应该只在需要时打开与数据库的连接。这意味着您的数据访问类可以多次实例化(这是最简单的设计),每个类都有自己的连接。如果您使用connection pooling,我觉得您实际上可能一直在询问,那么您将受益更多。转向静态数据库连接设计(其中许多人共享相同的 dao 类)可能会带来更多问题,因为您可能必须同步某些资源,确保某些变量只能通过锁定或类似的方式按顺序访问,等等。
例如,您可以阅读有关此here 的更多信息。微软还谈到了连接池@987654324@的好处。
上面的代码是连接数据库的一个很好的起点,通过包含你正在关闭的 using 语句,并在你完成连接后立即处理它;您还可以考虑以下改进:
using(var conn = new MySqlConnection(DatabaseHelper.getConnectionString()))
{
using (var cmd = conn.CreateCommand())
{
conn.Open();
cmd.CommandType = CommandType.Text;
//Parameterize your queries!
cmd.CommandText = "SELECT username FROM characters WHERE id=?"; //Use ? as placeholder for parameters, and add parameters in the order they appear in your query.
cmd.Parameters.Add(new MySqlParameter("@userid", MySqlDbType.Int, userid));
using(IDataReader reader = cmd.ExecuteReader())
{
//You should only expect one record. You might want to test for more than 1 record.
if (reader.Read())
{
user = reader.GetString(reader.GetOrdinal("username")); //Think also about null value checking.
}
}
}
}
您可能有一个 DAO 类或用户类上的一个方法来执行此操作。例如,如果它是用户上的方法,您可能会执行以下操作:
User myUser = new User(7);
myUser.Load();
在 Load 内部,您可能调用的方法之一是 PopulateFromDB(),它将包含上面的代码,并会加载该用户的所有属性。你可能还有一个做同样事情的 DAO 类:
UserLoader loader = new UserLoader();
string userName = loader.GetUserName(int userId);
这将使用上面示例中的代码返回用户名。我希望这种方法在像 User 这样的类上,因为它在逻辑上与之相连。但是,您会冒着将 DAO 逻辑与用户业务逻辑混合在一起的风险,这完全是一个独立的话题。
与其编写大量此类数据访问逻辑,不如考虑查看某种形式的框架,例如 ORM 或类似的框架 - SO 上的 question has already been answered for MySql。这也可以为您节省大量时间和精力,并让您专注于设计。