køre et tabel igennem og liste dem i en listbox

for det første er brug af database objekterne en smule rodet, skriver det lige på min egen måde, håber du forstår hvad jeg vil ud fra det:

public string hentKlientTabel(int tabelid) {
    OleDbConnection conn = new OleDbConnection("dsn");
    OleDbCommand cmd = conn.CreateCommand();
    cmd.CommandText = "SELECT klientid FROM KlientTabel WHERE tabelid = '" + tabelid + "'";
   
    OleDbDataReader dr = cmd.ExecuteReader();
    string returnString = "";
    while(dr.Read()) {
          returnString += dr.GetInt32(0) + ", ";
    }
    return returnString;
}

jeg undskylder hvis det ikke umiddelbart virker, har ikke testet det.

Min hentklienttabel virker godt nok og jeg forstår godt det du har lavet, men sådan kan jeg ikke lave det, da jeg bruger metoden flere gange, hvor der ikke skal være komma med. Jeg har prøvet med en for sætning, men den returnere kun et id selv om der er flere i tabellen og jeg skal gerne have returneret alle med det tabelid

int tableid = hentKlientTabel(tabelid);

for(int i = 0; i < medmoedeid(); i++)
{
}

det stykke kode jeg har skrevet til dig vil returnere alle klientid's hvor tabelid er det angivne.
Jeg forstod ikke helt det du skrev, men hvis det du mente var at der ikke skal være et komma helt til sidst i strengen kan du jo lave et count på hvor mange rækker den returnerer og så lave en tæller i while løkken, og så længe tælleren er mindre end det antal rækker der er, så skal den påføre et komma.

jeg tror jeg har forstået det du skrev bedre, det du vil er at returnere en array
det gør du ved at sige at metode skal returnere en int array.
public int[] hentKlientTabel(int tabelid)

du laver så først i din kode en query til databasen der tæller hvor mange rækker den vil returnere, og herefter laver du en int array med det antal rækker:
int[] returnArray = new Int32[rowCount];

herefter laver du så det kode jeg tidligere skrev for at returnere alle rækker, og så i din while løkke, bruger du en tæller til at angive hver enkelt række i din int array:
int = 0;
while(dr.Read()) {
    returnArray[i] = dr.GetInt32(0);
    i++;
}

return returnArray;

når du så skal bruge din funktion laver du en int array ud af funktionen:
int[] klientids = hentKlientTabel(id);

og så looper du gennem arayen med en foreach:
foreach(int KlientId in klientids) {
    Response.Write(KlientId);
}

tælleren i while løkken skal self. hedde: int i = 0;

Jeg har lavet min metode lige som din nu, men jeg får følgende fejl:

CS0029: Cannot implicitly convert type 'object'to 'System.Data.OleDb.OleDbDataReader'

jeg tror du bør sætte dig ned og sætte dig lidt bedre ind i hvordan objekt orienteret programmering fungerer. især bør du nok sætte dig ind i hvad data typer er og gør.

Ja, det er nok rigtigt nok, men min sql metode, som jeg viste i spørgsmålet, virker fint nok, og kan jeg ikke bare kører en foreach eller en for sætning på den for at løbe tabellen igennem men hvordan?

Du kan læse det ind i et dataset i ét hug, og efterfølgende iterere over den ene tabel du får i dit dataset.

du vil så skulle bruge en sql som :
SELECT klientid FROM KlientTabel

og det nemmeste er, hvis du opretter en adapter som du så bruger til at fylde et dateset med :

OleDbDataAdapter a = new OleDbDataAdapter(<dit commandobjekt);
DataSet ds = new DataSet();
a.Fill(ds);

foreach(DataRow dr in ds.Tables[0].Rows)
{
  Response.Write(dr[0].ToString());
}

Var det sådan noget du var ude efter ?

Ja, og så har prøvet at lave et dataset som det du har lavet, men så kommer den med følgende fejl:

CS0246: The type or namespace name 'DataRow' could not be found (are you missing a using directive or an assembly reference?)


    System.Data.DataSet hentKlientTabel()
        {
            string connectionString = "Provider=Microsoft.Jet.OLEDB.4.0; Ole DB Services=-4; Data Source=D:\\Fam\\db1.mdb";
            System.Data.IDbConnection dbConnection = new System.Data.OleDb.OleDbConnection(connectionString);

            string queryString = "SELECT [klientTabel].[klientid] FROM [klientTabel]";
            System.Data.IDbCommand dbCommand = new System.Data.OleDb.OleDbCommand();
            dbCommand.CommandText = queryString;
            dbCommand.Connection = dbConnection;

            System.Data.IDbDataAdapter dataAdapter = new System.Data.OleDb.OleDbDataAdapter();
            dataAdapter.SelectCommand = dbCommand;
            System.Data.DataSet dataSet = new System.Data.DataSet();
            dataAdapter.Fill(dataSet);

            foreach(DataRow dr in dataSet.Tables[0].Rows)
            {
                Response.Write(dr[0].ToString());
            }

            return dataSet;
        }

Hvad mener du med "dit commandobjekt"?

DataRow ligger i namespace System.Data, så du skal enten skrive en
using System.Data;

eller skrive det fuldt ud hver gang du bruger det :
System.Data.DataRow, så du får sådan en her :

foreach(System.Data.DataRow dr in dataSet.Tables[0].Rows)
    // osv..

Hvad jeg mener med dit commandbjekt... jeg mener det du laver med
new System.Data.OleDb.OleDbCommand();

yep, det virker som det skal nu, men hvordan får jeg dem lagt ind i en listbox, så de står samlet på en linje.

Giv lige et svar snepnet og prec, så får i 30 point hver

Jeg har fået den listbox til at virke, men hvordan får jeg dataene ind i et felt i en datagrid?

Hvad skal man skrive i stedet for ToString(), for at få det til en int, der hvor der står "Response.Write(dr[0].ToString());"?

k

lol jeg klovner :P

Svar her :o)
Du kan få det til en int på flere måder :

(int)dr[0];
int.Parse(dr[0].ToString());
Convert.ToInt(...