Индекс находится вне границ массива

@SteelWing_RC · Регистрация: 12.10.2018

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

Суть программы: считывание из файла и вывод на экран английского текста, заменяя первую букву слов, начинающихся с гласных, верхним регистром. Компилятор выдает исключение "Индекс находится вне границ массива" при сверке первой буквы(строка 27).

C#

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.IO;
 
namespace Lab7
{
    class Program
    {
        static void Main(string[] args)
        {
            string file_path = "C:/Users/1/source/repos/Lab7/Lab7/test.txt";
            StreamReader Read = new StreamReader(file_path);;
            char[] vow = { 'a', 'e', 'y', 'u', 'i', 'o', 'A', 'E', 'Y', 'U', 'I', 'O',};
            string Entered = Read.ReadToEnd();
            Read.Close();
            string[] StringArray;
            StringArray = Entered.Split(new[] {' ',',','/','\\','.',':',';','!','?','"','-', });
            StringBuilder str;
            for(int i=0;i < StringArray.Length;i++)
            {
                str = new StringBuilder(StringArray[i]);
                for (int j = 0; j < vow.Length; j++)
                {
                    if (str[0]==vow[j])
                    {
                        str.Replace(str[0], char.ToUpper(str[0]), 0, 1);
                        Console.WriteLine(str);
                        break;
                    }
                }
            }
            Console.ReadKey();
        }
    }
}

@Элд Хасп · 19.10.2018, 19:06

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

Компилятор выдает исключение "Индекс находится вне границ массива" при сверке первой буквы(строка 27).

Компилятор или при исполнении?

Добавлено через 11 минут
Проверил - всё нормально компилируется и исполняется. Возможно у Вас пустой файл "C:/Users/1/source/repos/Lab7/Lab7/test.txt" и из=за этого возникает ошибка.

Добавлено через 10 минут
И ещё рекомендация вместо подобной конструкции

C#

16	char[] vow = { 'a', 'e', 'y', 'u', 'i', 'o', 'A', 'E', 'Y', 'U', 'I', 'O', };

гораздо удобнее использовать

C#

16	char[] vow = @"aeyuioAEYUIO".ToCharArray();

А при разбивке Split в большинстве случаев (в том числе и Вашем) надо учитывать наличие переносов строк и исключать пустые элементы

C#

20	StringArray = Entered.Split("\r\n ,/\\.:;!?"-".ToCharArray(), StringSplitOptions.RemoveEmptyEntries);

Строки 15, 17, 20 можно сократить до одной

C#

20	StringArray = (new StreamReader(file_path)).ReadToEnd().Split("\r\n ,/\\.:;!?"-".ToCharArray(), StringSplitOptions.RemoveEmptyEntries);

Добавлено через 12 минут
Внутренний цикл for... со строки 25 проверяет только наличие символа в заданном массиве. Для этого можно использовать метод (Array.IndexOf в результате тело внешнего цикла сократится до 3 строк

C#

24
25
26

                str = StringArray[i];
                if (Array.IndexOf (vow, str[0])>=0)
                    Console.WriteLine(str.Remove(1).ToUpper() + str.Substring(1));

Добавлено через 5 минут
Внешний цикл у Вас перебирает слова больше у него нет функций поэтому логичнее его заменить на foreach... при этом переместив объявление переменной string str; в цикл. Весь код, с учётом вышесказанного, сократится до

C#

            string file_path = "test.txt";
            char[] vow = @"aeyuioAEYUIO".ToCharArray();
            foreach (string str in (new StreamReader(file_path)).ReadToEnd().Split("\r\n ,/\\.:;!?\"-".ToCharArray(), StringSplitOptions.RemoveEmptyEntries))
            {
                if (Array.IndexOf (vow, str[0])>=0)
                    Console.WriteLine(str.Remove(1).ToUpper() + str.Substring(1));
            }
            Console.ReadKey();

@SteelWing_RC · 19.10.2018, 19:52 **[ТС]**

Спасибо огромное! Однако не совсем понятна строчка:

if (Array.IndexOf (vow, str[0])>=0)

Можете, пожалуйста, объяснить?

@amr-now · 19.10.2018, 20:09

С регулярными выражениями задача естественнее смотрится. Они как раз и были созданы для работы с текстом.

C#

using System;
using System.IO;
using System.Text;
using System.Text.RegularExpressions;
 
namespace Lab7
{
    class Program
    {
        static void Main(string[] args)
        {
            string file_path = "C:/Users/1/source/repos/Lab7/Lab7/test.txt";
            string entered = File.ReadAllText(file_path, Encoding.UTF8);
 
 
            var matches = new Regex("[a-z]+", RegexOptions.IgnoreCase).Matches(entered);
            var stringArray = new string[matches.Count];
 
            for (int i = 0; i < stringArray.Length; i++)
            {
                var word = matches[i].Value;
                var first = word.Substring(0, 1);
                stringArray[i] = Regex.Replace(first, "[aeiouy]", first.ToUpper()) + word.Substring(1);
            }
            // Вывод
            for (int i = 0; i < stringArray.Length; i++)
            {
                Console.WriteLine(stringArray[i]);
            }
 
            Console.ReadKey();
        }
    }
}

@Элд Хасп · 19.10.2018, 21:14

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

Спасибо огромное! Однако не совсем понятна строчка:
if (Array.IndexOf (vow, str[0])>=0)
Можете, пожалуйста, объяснить?

Метод Array.IndexOf ищет в массиве указанном в первом параметре, значение указанное во втором параметре.
Если находит, то возвращает индекс найденного элемента.
Если такого элемента нет - возвращает -1.
Проверяя по условию >= 0, фактически мы проверяем был ли найден такой элемент.
Так как в массиве vow у нас все гласные, то получается это проверка первого символа строки str[0] - относится он к гласным или нет.

Добавлено через 2 минуты
В коде ещё и лишние скобки стоят - можно убрать

C#

            string file_path = "test.txt";
            char[] vow = @"aeyuioAEYUIO".ToCharArray();
            foreach (string str in (new StreamReader(file_path)).ReadToEnd().Split("\r\n ,/\\.:;!?\"-".ToCharArray(), StringSplitOptions.RemoveEmptyEntries))
                if (Array.IndexOf (vow, str[0])>=0)
                    Console.WriteLine(str.Remove(1).ToUpper() + str.Substring(1));
            Console.ReadKey();

@SteelWing_RC · 19.10.2018, 21:19 **[ТС]**

Спасибо Вам большое!

Индекс находится вне границ массива

Решение

Решение

Решение

@SteelWing_RC 1 / 1 / 0 Регистрация: 12.10.2018 Сообщений: 24
	19.10.2018, 19:52 [ТС]	3
	Спасибо огромное! Однако не совсем понятна строчка: if (Array.IndexOf (vow, str[0])>=0) Можете, пожалуйста, объяснить? 0

@SteelWing_RC 1 / 1 / 0 Регистрация: 12.10.2018 Сообщений: 24
	19.10.2018, 21:19 [ТС]	6
	Спасибо Вам большое! 1

Опции темы