Ошибка индекса символа при SubString

@Purple__Drunk · Регистрация: 26.12.2020

Студворк — интернет-сервис помощи студентам

Здраствуйте! Имеется задача написать метод к программе в учебных целях.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
namespace Pluralize
{
    public static class PluralizeTask
    {
        public static string PluralizeRubles(int count)
        {
            int preLastIndex = count.ToString().Length - 2;
            int lastIndex = count.ToString().Length - 1;
            string preLastChar = count.ToString().Substring(preLastIndex, 1); //ругается тут
            string lastChar = count.ToString().Substring(lastIndex, 1); //и тут
            
            if ((preLastChar != "1") && (lastChar == "1"))
                return "рубль";
            else if ((preLastChar != "1") && (lastChar == "2" || lastChar == "3" || lastChar == "4"))
                return "рубля";
            else return "рублей";
        }
    }
}

Ругается на 2 строки и пишет

Code
1
2
3
"Exception on: 0  
StartIndex cannot be less than zero.
Parameter name: startIndex"

Но при постановке стоп-поинта перед этими строками и построчным переключениями через F10 все значения принимаются верные и программа доходит до конца метода и даже выходит из него. А в ран-тайме крашит.

@EveKS · 27.03.2021, 20:07

Сообщение от Purple__Drunk

int preLastIndex = count.ToString().Length - 2;
int lastIndex = count.ToString().Length - 1;

Сообщение от Purple__Drunk

StartIndex cannot be less than zero.

Может где-то тут стоит что-то поменять?

@Purple__Drunk · 27.03.2021, 20:23 **[ТС]**

EveKS, я пробовал без них, указав все прямо внутри скобок сабстринга - тоже самое

@JustinTime · 27.03.2021, 20:40

Purple__Drunk,

C#
1
2
3
4
5
6
7
8
9
10
11
12
static string PluralizeRubles(int count)
{
    var d = count % 10;
    var r = count % 100 / 10;
    if (r != 1  && d == 1)
        return "рубль";
    else if (r != 1 && (d <=4 && d>=2))
        return "рубля";
    else return "рублей";
}
 
Console.WriteLine(PluralizeRubles(5));

@Purple__Drunk · 27.03.2021, 20:44 **[ТС]**

JustinTime, решил уже по другому:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
namespace Pluralize
{
    public static class PluralizeTask
    {
        public static string PluralizeRubles(int count)
        {
            int length = count.ToString().Length;
            int preLastIndex;
            int lastIndex = (length - 1);
            string preLastChar = null;
            string lastChar = null;
 
            if (length > 1)
            {
                preLastIndex = (length - 2);
                preLastChar = count.ToString().Substring(preLastIndex, 1);
            }
            else
            {
                preLastIndex = 0;
                preLastChar = "0";
            }
            lastChar = count.ToString().Substring(lastIndex);
            
            if ((preLastChar != "1") && (lastChar == "1"))
                return "рубль";
            else if ((preLastChar != "1") && (lastChar == "2" || lastChar == "3" || lastChar == "4"))
                return "рубля";
            else return "рублей";
        }
    }
}

В твоем, кстати на 11 будет выдавать "рубля"

Someone007 · 27.03.2021, 20:45

Purple__Drunk, А зачем такой говнокод то? Почему нельзя взять рандомный более эффективный кусок кода по одной из ссылок в гугле?

Зачем тут вообще Substring использовать, если можно получить символ по индексу напрямую и не создавать кучу мусора...

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
        public static string PluralizeRubles(int count)
        {
            const string one = "рубль";
            const string two = "рубля";
            const string five = "рублей";
 
            count %= 100;
            if (count >= 5 && count <= 20)
            {
                return five;
            }
            count %= 10;
            if (count == 1)
            {
                return one;
            }
            if (count >= 2 && count <= 4)
            {
                return two;
            }
            return five;
        }

@JustinTime · 27.03.2021, 20:51

Сообщение от Purple__Drunk

В твоем, кстати на 11 будет выдавать "рубля"

не будет.
Зачем Substring? если мат. вычисления быстрее работы со строками в разы...

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

@Purple__Drunk 27 / 26 / 2 Регистрация: 26.12.2020 Сообщений: 86
	27.03.2021, 20:23 [ТС]
	EveKS, я пробовал без них, указав все прямо внутри скобок сабстринга - тоже самое 0