C# - Gemensam kod för alla case i ett Switch statement - Programmering och digitalt skapande

2012-04-13, 02:36

#1

Murloc

Medlem

Plats: Kiruna

Registrerad: maj 2010

C# - Gemensam kod för alla case i ett Switch statement

Om jag har ett Switch statement med väldigt många olika case och varje case har några rader gemensam kod, finns det då något sätt att få den gemensamma koden att köras vid varje case utan copy paste:a det överallt?

__________________
ASUS P6T Deluxe V2 - i7-930 @ 3,85Ghz - Gigabyte GTX 580 - 12GB Corsair DDR3 1600Mhz - 2x WD Caviar® Black 1TB 7200RPM 32MB Cache i RAID-0 - Corsair TX750M

Anmäl

Citera

Gilla

2012-04-13, 02:47

#2

swehunter2000

Medlem

Plats: östersund

Registrerad: jan 2011

en function

__________________
CPU:Intel Core I7 930 (Socket 1366) MOBO:Asus P6T-SE GPU:Sapphire Radeon HD6870 FleX RAM:Corsair XMS3 Vengeance 1600 MHz 12GB (Tripple channel) SSD:Force GT 120GB HDD:Samsung Spinpoint F3 1TB Chassi:NZXT M59 PSU:Antec 550 Watt Modulärt Ljud:Creative Sound Blaster X-Fi Titanium Fatal1ty Champion Series Hörlurar: AKG K 240 MK II Högtalare: Logitech X-530 Mus: Logitech MX518 Matta: steelseries QcK Mini Skärmar: en BenQ 22' och en Samsung 19'. V.v. citera mig för att få svar.

Anmäl

Citera

Gilla (1)

2012-04-13, 02:50

#3

swehunter2000

Medlem

Plats: östersund

Registrerad: jan 2011

Public Static Void ngtnamn{
kod..
kod...
kod..
}

case habbla habbla
ngtnamn; <-- kör funktionen "ngtnmn"

Om jag inte kommer ihåg helt fel..

__________________
CPU:Intel Core I7 930 (Socket 1366) MOBO:Asus P6T-SE GPU:Sapphire Radeon HD6870 FleX RAM:Corsair XMS3 Vengeance 1600 MHz 12GB (Tripple channel) SSD:Force GT 120GB HDD:Samsung Spinpoint F3 1TB Chassi:NZXT M59 PSU:Antec 550 Watt Modulärt Ljud:Creative Sound Blaster X-Fi Titanium Fatal1ty Champion Series Hörlurar: AKG K 240 MK II Högtalare: Logitech X-530 Mus: Logitech MX518 Matta: steelseries QcK Mini Skärmar: en BenQ 22' och en Samsung 19'. V.v. citera mig för att få svar.

Anmäl

Citera

Gilla

2012-04-13, 03:31

#4

Murloc

Medlem

Plats: Kiruna

Registrerad: maj 2010

Var inte så jag menade, funktionen måste jag också copy paste:a på alla case. Vi säger att jag har hundra case som alla ska köra koden i funktionen, kan jag undvika att klistra in funktionen på hundra ställen?

__________________
ASUS P6T Deluxe V2 - i7-930 @ 3,85Ghz - Gigabyte GTX 580 - 12GB Corsair DDR3 1600Mhz - 2x WD Caviar® Black 1TB 7200RPM 32MB Cache i RAID-0 - Corsair TX750M

Anmäl

Citera

Gilla

2012-04-13, 04:52

#5

tgwallenborg

Medlem

Registrerad: sep 2011

Jag kan vara helt ute och cykla men kan du inte, det första du gör köra en default case där du skiter i break.
Är inte helt säker på att den väljer rätt case efter eller om det automatiskt hoppar in i case 1 men du kan ju testa.
T.ex
case (vilkor)
case default: koden()
case 1: blabla;
break;
osv

__________________

Anmäl

Citera

Gilla

2012-04-13, 05:48 #6

jovnas

Medlem

Plats: Göteborg

Registrerad: maj 2004

jag har förmodligen missuppfattat problemet, men om ALLA case skall köra vissa rader som är samma, så ska ju den koden inte ligga i switchen alls...

alternativt kan man ju köra en default, som tidigare nämnts, eller sätta så att flera olika case kör samma kod. observera att "fallthrough" inte fungerar i C# utan man måste explicit ange "goto" istället.


var result = string.Empty;

switch (i)

{

    case 1:

        result = functionA(i);

        break;



    case 2: 

        // Do nothing

        break;



    case 3:

    case 4:

        result = functionB(8); // Körs för 3 och 4

        break;



    case 5:

        result = functionC(50);

        goto case 1;    // Hoppar till case 1



    case 42:

        throw new Exception("Whoa! 42!");



    default:    // Körs om inget tidigare matchar

        result = functionX();

        break;

}

mer om switch på MSDN

__________________
as far as we can tell, the massacre went well...

Anmäl

Citera

Gilla

2012-04-13, 07:58

#7

ministerkrister

Medlem

Plats: Uppsala

Registrerad: apr 2007

Du kanske ska göra två fall?

...
if(condition) {
       switchMethod1(...);
} else {
       switchMethod2(...);
}
...

private switchMethod1(...)
{
    gemensamMetod(...);
    switch (i)
   {
    case 1:
       ....
        break;

    default:
    ...
   }
} 
private switchMethod2(...)
{
    switch (i)
   {
    case 1:
       ....
        break;

    default:
    ...
   }
}

__________________
The statement below is true.
The statement above is false.

Anmäl

Citera

Gilla

2012-04-13, 08:41

#8

Mikael07

Medlem

Plats: Göteborg

Registrerad: mar 2011

Citat:

Ursprungligen inskrivet av Murloc

Var inte så jag menade, funktionen måste jag också copy paste:a på alla case. Vi säger att jag har hundra case som alla ska köra koden i funktionen, kan jag undvika att klistra in funktionen på hundra ställen?

Ja, du kan skriva den utanför switchen.

__________________
.:: Linux Mint :: i5 2500k :: Gainward GTX580 Phantom :: XMS3 8Gb :: Seagate 2Tb ::.

Anmäl

Citera

Gilla

2012-04-13, 08:47

#9

ErDraug

Medlem

Plats: Borås

Registrerad: sep 2008

Använd inte GoTo... Snälla... Men annars ja du måste lägga den på 100 ställen är nog enklaste sättet.

Anmäl

Citera

Gilla

2012-04-13, 13:08

#10

JasPlan

Medlem

Plats: Stockholm

Registrerad: sep 2009

Citat:

Ursprungligen inskrivet av ministerkrister

Du kanske ska göra två fall?

...
if(condition) {
       switchMethod1(...);
} else {
       switchMethod2(...);
}
...

private switchMethod1(...)
{
    gemensamMetod(...);
    switch (i)
   {
    case 1:
       ....
       break;

    default:
    ...
   }
} 
private switchMethod2(...)
{
    switch (i)
   {
    case 1:
       ....
        break;

    default:
    ...
   }
}

varför krångla till det? han kan väl bara köra:

...
if(condition) 
{
      gemensamMetod(...);
}

switch (i)
{
    case 1:
       ....
        break;

    default:
    ...
}

__________________
CPU: Intel 3930k @ 4.5GHz (Vattenkyld) MB: Asus Rampage Extreme IV X79 (Vattenkyld) GPU: 2st GTX680 2GB @ 1200MHz (Vattenkyld) RAM: 32GB @ 1866MHz Chassi: Enermax Fulmo GT PSU: Silverstone Platinum 1000W SSD: 2st Vertex 4 256GB (RAID 0) HDD: 2st WD Green 2TB Skärmar: 2st Dell U2412m Ljudinterface: Propellerhead Balance Högtalare: 2st Genelec 8020BPM

Anmäl

Citera

Gilla

2012-04-14, 08:19

#11

DeluxXxe

Medlem

Plats: Karlskrona

Registrerad: feb 2007

Citat:

Ursprungligen inskrivet av JasPlan

varför krångla till det? han kan väl bara köra:

... if(condition) { gemensamMetod(...); } switch (i) { case 1: .... break; default: ... }

Den sista var ganska bra, men han nämnde inte att det var något särskilt förhållande, bara att koden var tvungen att användas i varje case. Lägg det efter switch satsen:

switch (i)
{
    case 1:
       ....
        break;

    default:
    ...
}
gemensamMetod(...);

Att man har massa logik i switch satser kan även tyda på att man kan bryta ut den logiken till klasser och subklasser. Istället för koden ovan hade du då kunnat använda något liknande detta:

http://pastie.org/3785146

Nu gjorde jag koden ovan i Java, men det är samma princip och nästan samma syntax i C#. =)

Detta gör att du isolerar antalet switchar till ett ställe, dvs där du skapar objekten. Det är bra om du tex har switch satser utspridda lite här och där, som switchat på samma sak, men gör olika saker. För varje ny sak du vill att alla switch-fall ska utföra, så lägger du istället till en metod och lägger beteendet i objektet istället för ett switch-case. Lycka till!

Anmäl

Citera

Gilla

2012-04-14, 08:45

#12

SolidReactor

Medlem

Plats: göteborg

Registrerad: nov 2002

Varför inte try/finally?

try
{
    switch (i)
    {
        case 1:
        .......;
        break;
    
        case #:
        ........;
        break;

        default:
        ........;
        break;
    }
}
finally
{
    gemensamKod();
}

Har inte testat koden, kommer från C++ och nyligen börjat med C# men de bör fungera galant!

Anmäl

Citera

Gilla

2012-04-14, 09:07

#13

DeluxXxe

Medlem

Plats: Karlskrona

Registrerad: feb 2007

Citat:

Ursprungligen inskrivet av SolidReactor

Varför inte try/finally?

try
{
    switch (i)
    {
        case 1:
        .......;
        break;
    
        case #:
        ........;
        break;

        default:
        ........;
        break;
    }
}
finally
{
    gemensamKod();
}

Har inte testat koden, kommer från C++ och nyligen börjat med C# men de bör fungera galant!

Try, catch, finally är till för återhämtande från fel. Här förekommer ingen felhantering och därför bör det inte heller användas. Om man vill göra det enkelt, men samtidigt kan tänka sig att ha logiken utspridd i flera switchar så är det mest logiska att placera "gemensamkod()" direkt efter switchen, men utan try, finally.

Anmäl

Citera

Gilla

2012-04-14, 15:48

#14

KurreKula

Medlem

Plats: Bromma

Registrerad: aug 2008

Det är sällan man ser så många dåliga förslag i samma tråd.

Ska koden köras för varje case så har den inget i switchen att göra. Hela syftet med en switch är ju till för att köra viss kod för vissa case, inte för alla. Default är endast till för de fallen när inget case har uppnåtts.

JasPlans lösning är den enda jag anser värdig

__________________
Asus Striker II Extreme / XFX Geforce GTX 280 / Q9450 @ 3.6GHz/ TRUE Noctua 120/ 4x1GB Corsair TWIN3X2048-1333C9DHX / X25-M G2 80gb Velociraptor / Win 7 Ultimate x64/ Antec P190

MovieDatabase

Anmäl

Citera

Gilla (2)

2012-04-14, 22:10

#15

ministerkrister

Medlem

Plats: Uppsala

Registrerad: apr 2007

Citat:

Ursprungligen inskrivet av JasPlan

varför krångla till det? han kan väl bara köra:

...
if(condition) 
{
      gemensamMetod(...);
}

switch (i)
{
    case 1:
       ....
        break;

    default:
    ...
}

Såklart jag var inte vid mina sinnens fulla bruk när jag skrev det där. Skämskudde framför ansiktet.

Skickades från m.sweclockers.com

__________________
The statement below is true.
The statement above is false.

Anmäl

Citera

Gilla (1)

Verktyg
Visa utskriftsvänlig version E-posta den här sidan
Visningsval
Linjär visning Växla till hybrid-visning Växla till trådad visning

Sök innehåll

Preliminärt sökresultat

Murloc

C# - Gemensam kod för alla case i ett Switch statement

swehunter2000

swehunter2000

Murloc

tgwallenborg

jovnas

ministerkrister

Mikael07

ErDraug

JasPlan

DeluxXxe

SolidReactor

DeluxXxe

KurreKula

ministerkrister

Dela

Senaste nyheterna

Hårdvara

Mjukvara

Övrigt

Nytt i forumet

Senaste privatannonserna

Säljes

Köpes

Skänkes

Nytt i testlabbet

Nya IT-jobb

Nya IT-jobb

Sök jobb

In English

Annonsera

Kontakt

RSS

Sök innehåll

Medlemskap

Preliminärt sökresultat

C# - Gemensam kod för alla case i ett Switch statement

C# - Gemensam kod för alla case i ett Switch statement

Dela

Senaste nyheterna

Hårdvara

Mjukvara

Övrigt

Nytt i forumet

Senaste privatannonserna

Säljes

Köpes

Skänkes

Nytt i testlabbet

Nya IT-jobb

Nya IT-jobb

Sök jobb

In English

Annonsera

Kontakt

RSS