Исходный код
if ((armid == "3504000036") || (armid == "3504000034"))
{
if (armid == "3504000036")
{
PlaceSave = "3243100001";
}
if (armid == "3504000034")
{
PlaceSave = "2311030001";
}
}Что не так в исходном коде
Очевидно, что первая проверка идентификаторов является избыточной, так как следом выполнятеся та же самая проверка. Соответственно, первый оператор if можно просто убрать.
Чтобы подчеркнуть альтернативность каждого сравнения, можно использовать else или вовсе перейти на оператор switch:
switch (armid) {
case '3504000036': PlaceSave = '3243100001'; break;
case '3504000034': PlaceSave = '2311030001'; break;
}