Hallo, kann man diesen Code irgendwie kompakter schreiben?

@C1_ID, etc. sind Parameter vom Typ VARCHAR(50) einer sp.

markieren

Code:

DECLARE @Counter INT
SET @Counter = 1

WHILE (@Counter < 5)
BEGIN
   IF (@Counter = 1)
   BEGIN
      SET @C_ID = @C1_ID
   END
   ELSE IF (@Counter = 2)
   BEGIN
      SET @C_ID = @C2_ID
   END
   ELSE IF (@Counter = 3)
   BEGIN
      SET @C_ID = @C3_ID
   END
   ELSE IF (@Counter = 4)
   BEGIN
      SET @C_ID = @C4_ID
   END
   SET @Counter = @Counter + 1

   -- some more code.....
END

Geht es darum, ein Verarbeitungs-Feld auszutauschen abhängig vom Counter?
Ich denke nicht, dass es besser geht, außer ein Case Statement, was aber im Prinzip kein Unterschied ist.

Eigentlich deutet diese Feld1- 5 Geschichte auf ein unsauberes Datenmodell hin, würde ich mal so ohne weiteren Hintergrund sagen.
Die Feld1- Feld5 Infos gehören vielleicht eher in 5 verschiedene Datensätze und würden so problemlos zu bearbeiten sein.

If then else if then else if then else if then else if then else ...

Case?

hallo,

ein Array?

mfg

Also ich hätte das jetzt so gemacht:

markieren

Code:

DECLARE @Counter INT
SET @Counter = 1

WHILE (@Counter < 5)
BEGIN
  Set @C_ID =
   Case @Counter
   When 1 then @C1_ID else
   When 2 then @C2_ID else
   When 3 then @C3_ID else
   When 4 then @C4_ID end;
END
   SET @Counter = @Counter + 1

MFG Devil

Danke Devil1925, genau so habe ich es mir vorgestellt.

PS: Die else musste ich weglassen, sonst zeigte es Fehler an.

Zitat:

Eigentlich deutet diese Feld1- 5 Geschichte auf ein unsauberes Datenmodell hin, würde ich mal so ohne weiteren Hintergrund sagen.

Das sind 4 Parameter einer gespeicherten Prozedur.

Zitat von API:

Zitat:

Eigentlich deutet diese Feld1- 5 Geschichte auf ein unsauberes Datenmodell hin, würde ich mal so ohne weiteren Hintergrund sagen.

Das sind 4 Parameter einer gespeicherten Prozedur.

Ja, schon klar. Merkmale, die in dieser Form durchnummeriert sind, deuten halt auf schwächen im Datenmodell. Ich habe ja dazugeschrieben, dass ich das ohne Hintergrundinformationen schreibe. Es war einfach nur ein Hinweis.

Sorry wegen den "Else"'s, ich hatte das einfach mal so runter geschrieben und nicht vorher getestet, sonst wäre es vermutlich aufgefallen naja, ist ja gut das ich dein Problem Beheben konnte