[C#] Errore OleDbCommand.ExecuteReader()

[Andriy08]

Sapreste dirmi a cosa è dovuto questo errore?
Ho controllato e la connessione OleDBConnection è aperta. Metto una query SQL in OleDBCommand e evoco la funzione ExecuteReader e mi da questo errore... Sto sbattendo la testa già da 2 giorni.

System.Data.OleDb.OleDbException: Nessun valore specificato per alcuni parametri necessari.
in System.Data.OleDb.OleDbCommand.ExecuteCommandTextF orSingleResult(tagDBPARAMS dbParams, Object& executeResult)
in System.Data.OleDb.OleDbCommand.ExecuteCommandText( Object& executeResult)
in System.Data.OleDb.OleDbCommand.ExecuteCommand(Comm andBehavior behavior, Object& executeResult)
in System.Data.OleDb.OleDbCommand.ExecuteReaderIntern al(CommandBehavior behavior, String method)
in System.Data.OleDb.OleDbCommand.ExecuteReader(Comma ndBehavior behavior)
in System.Data.OleDb.OleDbCommand.ExecuteReader()
in TravianBot2.DataConnection.ExecuteSelect(String sqlCommand, QueryCallback callback) in C:\Users\Administrator\Desktop\Applications\Databa se\DataConnection.cs:riga 112

Qualcuno mi può aiutare?

[Andriy08]

Posto un può di codice sperando che si riesce a capire meglio l'errore.

public class DataConnection
{
    private readonly Queue[list=1] m_connectionPool = new Queue[list=1]();
    private string connString;

    public DataConnection(string connString)
    {
        this.connString = connString;
    }

    private OleDbConnection GetSqlConnection(out bool isNewConnection)
    {
        OleDbConnection conn = null;
        lock (m_connectionPool)
        {
            if (m_connectionPool.Count > 0)
            {
                conn = m_connectionPool.Dequeue();
            }
        }
        if (conn == null)
        {
            isNewConnection = true;
            conn = new OleDbConnection(connString);
            conn.Open();
        }
        else
        {
            isNewConnection = false;
        }
        return conn;
    }

    private void ReleaseConnection(OleDbConnection conn)
    {
        lock (m_connectionPool)
        {
            m_connectionPool.Enqueue(conn);
        }
    }

    public void ExecuteSelect(string sqlCommand, QueryCallback callback)
    {
        OleDbConnection conn = null;
        bool repeat = false;
        do
        {
            bool isNewConnection = true;
            try
            {
                conn = GetSqlConnection(out isNewConnection);
                OleDbCommand cmd = new OleDbCommand(sqlCommand, conn);
                OleDbDataReader reader = cmd.ExecuteReader();
                callback(reader);
                reader.Close();
                ReleaseConnection(conn);
                repeat = false;
            }
            catch (Exception e)
            {
                if (conn != null)
                    conn.Close();
                if (!isNewConnection)
                    repeat = true;
                System.Windows.Forms.MessageBox.Show(e.ToString(), "Error");
            }
        } while (repeat);
    }
} 

Qualche idea?

[Andriy08]

Da l'errore sul rigo 54 del codice che ho incollato.
Cioè su: OleDbDataReader reader = cmd.ExecuteReader();

[oregon]

Il problema non e' nel codice ma nella query contenuta in

sqlCommand

[Andriy08]

Questa è la query SQL:

SELECT `ID_Account`, `UserName`, `Password`, `Server`, `Tribe`, `Key` FROM `Accounts`; 

Utilizzo Access come database. Cosa c'è che non va in questa query?

[oregon]

Originariamente inviato da Andriy08
Questa è la query SQL:

SELECT `ID_Account`, `UserName`, `Password`, `Server`, `Tribe`, `Key` FROM `Accounts`; 

Utilizzo Access come database. Cosa c'è che non va in questa query?

Intanto non utilizzare gli apici. E poi, se questi campi sono tutti quelli presenti nella tabella usa il simbolo *

SELECT * FROM Accounts

[Andriy08]

Il problema è che la query viene generata in automatico.
Ho creato una classe base, a quale si appoggiano diciamo le classi tabelle, le quali hanno a sua volta le funzioni con dei Attributi che ho creato.
Quindi la query sql viene generata in questo modo:

object[] dataTable = objectType.GetCustomAttributes(typeof(DataTable), true);
StringBuilder sb = new StringBuilder("SELECT");

if (topLimit > 0)
    sb.Append(" TOP " + topLimit);

sb.Append(" `" + (dataTable[0] as DataTable).TableID + "`");

BindingInfo[] bindingInfo = GetBindingInfo(objectType);
for (int i = 0; i < bindingInfo.Length; i++)
    sb.Append(", `" + bindingInfo[i].Member.Name + "`");

sb.Append(" FROM `" + (dataTable[0] as DataTable).TableName + "`");

if (whereClause != null && whereClause.Trim().Length > 0)
    sb.Append(" WHERE " + whereClause.Trim());

sb.Append(';'); 

[oregon]

Allora, non usare gli apici per racchiudere i nomi dei campi e tabelle ma le parentesi quadre

[ ]

La query diventerà

SELECT [ID_Account], [UserName], [Password], [Server], [Tribe], [Key] FROM [Accounts];

[Andriy08]

Ok Provo

[Andriy08]

Bene, ho risolto. Che casino ho fatto sul forum. LOL
Per una query pakka di Access. Sono Abituato con MySQL.

Grazie oregon dell'aiuto.

PS. Secondo te è positivo gestire le connessioni in quel modo?