Как по другому сделать?

Question

[D3nch1kk]

int characterChoose = Convert.ToInt32(Console.ReadLine());
 switch(characterChoose)
 {
     case 0:
         int PlayerDamage = 50;
         int PlayerHealth = 150;
         int PlayerResist = 25;
         break;
     case 1:
         int PlayerDamage = 75;
         int PlayerHealth = 125;
         int PlayerResist = 15;
         break;

 }

Пишет

Переменная PlayerDamage уже объявлена в этой области

Comments

[Василий Банников]

А что сделать то?

[Shavadrius]

Если чисто по ошибке говорить, то SWITCH в данном случае является областью видимости и внутри него нельзя объявлять переменные с одинаковым именем.
Если хотите объявлять одни и те же переменные внутри CASE, то просто создайте еще один локальный Scope, добавив фигурные скобки.
Как-то так:

switch(characterChoose)
 {
     case 0: {
         int PlayerDamage = 50;
         int PlayerHealth = 150;
         int PlayerResist = 25;
         break;
     }
}

[#]

Shavadrius, а их потом получится использовать? ))))))))))))))

[#]

Shavadrius, реально вспоминается Хатабыч с мраморным телефоном )))

[Shavadrius]

#, ну это уже другой вопрос) внутри кейса можно без проблем)

[#]

Shavadrius, ну.. раз пошла такая пьянка.. может все таки вызов функции/метода с тремя аргументами? )))

[#]

Shavadrius, или кортежи )))

[#]

Shavadrius, а что то рулить "внутри кейса" это жесть как утяжеляет код )))

Answer 1

[Василий Банников]

А что сделать то?
Вообще кажется, что вот эти характеристики лучше в класс объединить и держать где-нибудь словарь/массив со всеми вариантами.

Тогда и switch не понадобится и вот так дублировать тоже не придётся

Answer 2

[MVV]

Вынесите описания переменных наверх, над swicth.

int PlayerDamage ;
int PlayerHealth;
int PlayerResist ;

 switch(characterChoose)
 {
     case 0:
         PlayerDamage = 50;
         PlayerHealth = 150;
         PlayerResist = 25;
         break;
     case 1:
         PlayerDamage = 75;
         PlayerHealth = 125;
         PlayerResist = 15;
         break;

 }

PS Согласен с предыдущим комментатором - инциализация через оператор switch выглядит тяжеловесно. В нынешнем C# можно записать то же самое сильно короче, через деконструкцию, например.
PPS Держите пример с вашеми данными, как это должно выгдядеть стильно-модно-молодежно:

int characterChoose = Convert.ToInt32(Console.ReadLine());
(int PlayerDamage, int PlayerHealth , int PlayerResist)=characterChoose switch
{
    0 => (50, 150, 25),
    1 => (75, 125, 15),
   //... дописать все остальные варианты
} ;

Answer 3

[#]

а что говорит компилятор? и справка по ошибке?
.. там и делайте выводы