Problem med att översätta ifrån VB-script till Asp.net 2.0 C#

Du hamnar i en evig loop. antalValda är ju 0 från början och i for-loopen kollar du om i<antalValda, vilket det inte är. Alltså kommer antalValda aldrig att bli antalValda+1, och man kommer aldrig ur while-loopen.

BTW. Om du bara ska hämta ett enda värde från databasen så är det ohygglig overkill att köra via en adapter och ett dataset! Använd bara ett OleDbCommand och metoden ExecuteScalar.

Tack för svar emission, och jag förstår dina synpunkter, dok inte hur jag skall lösa det.
Jag gjorde en liten test:

for (int i = 0; i < antalValda; i++)
            {
                Random randomNumber = new Random();
                slumpTal = randomNumber.Next(1, Convert.ToInt32(id));
                Response.Write(slumpTal.ToString());
                if (slumpTal != Urval[i])
                {
                    antalValda += 1;
                    Urval[antalValda] = slumpTal;
                }
                    
            }

Jag flyttade in min random, då jag tänkte att den då skulle fylla på antalValda, men
det blev samma resultat.
Hur skall jag komma förbi problemet?

PS
Jag har ändrat databasanropet:

sql = "SELECT MAX(id) AS maxid FROM Quize";
        OleDbCommand myCommand = new OleDbCommand(sql, DBConn);
        DBConn.Open();
        int id = (int)myCommand.ExecuteScalar();

DS

Du sitter fortfarande med samma problem. For-satsen säger at den ska gå in i satsen om i är mindre än noll (antalValda). Om du vill att den ska gå in i for-satsen måste du sätta ett annat vilkor, alternativt ändra värdet på antalvalda till något mer än noll.

Gjorde en lottonummer generator för ett tag sen som borde fungera.

string connectionstring = "";
string sql = "SELECT MAX(id) AS maxid FROM Quize";
OleDbConnection conn = new OleDbConnection(connectionstring);
OleDbCommand myCommand = new OleDbCommand(sql, conn);

int max = 0;
using(conn)
{
   conn.Open();
   max = (int)myCommand.ExecuteScalar();
}

if(max == 0)
   throw;

List<int> col = new List<int>();
Random rnd = new Random();
for(int i = 0; i < 7; i++)
{
   int x;
   do 
   {
      x = rnd.Next(1, max);
   }
   while(col.Contains(x));
   col.Add(x);
}            

foreach(int i in col)
   Response.Write(i.ToString() + "<br/>");

Dino, det ser väldigt intressant ut, men jag åker på ett par errors.

if (max == 0)
            throw;
'error: Error	1	En throw-sats utan argument tillåts inte utanför en catch-sats	F:\Inetpub\wwwroot\ASP_Net-Test\Saw-Asp.net\Quize\Quize.aspx.cs	52	13	F:\...\Saw-Asp.net\

'om jag bortkommenterar raden med throw;
 List<int> col = new List<int>();
'error
Error	1	En inbäddad sats kan inte vara en deklaration eller etiketterad sats	F:\Inetpub\wwwroot\ASP_Net-Test\Saw-Asp.net\Quize\Quize.aspx.cs	54	9	F:\...\Saw-Asp.net\

Det känns som om jag är i närheten, men jag vet inte hur jag skall fixa till detta.

Om du bortkommenterar raden med throw så blir ju den if-satsen helt uppf***ad. Ta bort hela det blocket (inte rätt) eller kasta en Exception med throw.

Helt riktigt emission.
Jag tror att jag har suttit lite för länge med det här, och såg inte den enkla lösningen, att lägga det i ett try/catch block.

Nu fungerar detta, men jag skulle behöva ha något råd för fortsättningen.
Tidigare (i min gammla applikation) så la jag detta i sessions, och gjorde en ny
databas-sökning för varje fråga. Det känns nog inte bra att göra så.
Min tanke är, skulle man kunna göra bara en sql, och lägga resultatet (i.tostring) som
IN i den sql-sats, lägga detta i ett dataset, och skicka med datasetet som ett
agrument till en class?
Finns det något bättre sätt?

Jag har gjort så här nu:

sql="SELECT Quize.Id, Quize.Fraga, Quize.Allt1, Quize.Allt2, Quize.Allt3, Quize.Svar, Quize.Ratt FROM Quize ";
        sql +="  WHERE Quize.Id  IN( ";
        foreach (int i in col)
            sql += i.ToString() + ",";
        sql += " )";
        Response.Write(sql);

När jag skriver ut min sql så blir det:

SELECT Quize.Id, Quize.Fraga, Quize.Allt1, Quize.Allt2, Quize.Allt3, 
Quize.Svar, Quize.Ratt FROM Quize WHERE Quize.Id IN( 
49,39,421,94,69,409,239,53,290,34, )

Som ni ser, så får jag med ett "," för mycke, och jag antar att jag skall
ha en if-sats för att känna av om jag är på sista posten.
Men jag vet inte hur den bör utformas. Något tips? :stud

Jag såg svårigheterna som större än lösningen:

int k = 1;
        foreach (int i in col)
        {
            if (k != 10)
                sql += i.ToString() + ",";
            else
                sql += i.ToString();
            k += 1;
        }

Tack för all hjälp.

Antingen så gör du en if-sats som struntar i att lägga på kommat om du är på den sista posten (du får göra en for-loop i stället för en foreach), eller så tar du helt enkelt bort sista bokstaven från SQL-satsen.

sql=sql.Substring(0,sql.Length-1);
sql += " )";

PS. Det stavas "Quiz", utan "e" :) DS