Question

我正在尝试使用强力技术进行简单的子串搜索，但是我收到了一个我看不到的错误。我对编程很陌生，所以请记住这一点。问题可能很简单。

using System;
using System.Collections;
using System.Collections.Generic;

namespace SubstringSearch
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.Write("Please enter some letters: ");
            string sequence = Console.ReadLine();
            Console.Write("Enter the sequence you want to search for: ");
            string pattern = Console.ReadLine();

            Console.WriteLine(Search(pattern, pattern.Length, sequence, sequence.Length));

            Console.ReadLine();
        }

        public static int Search(string pattern, int patternLength, string sequence, int stringLength)
        {
            int i;
            int j;

            if (stringLength >= patternLength)
            {
                for (j = 0; j <= (stringLength - patternLength); j++)
                {
                    for (i = 0; i < patternLength && pattern[i] == sequence[i + j]; i++);

                    if (i >= patternLength)
                        return j;
                    else
                        return -1;
                }
            }
            else
                return -1;
        }
    }
}

所以我收到一个错误和一个警告。首先它告诉我并非所有代码路径都返回一个值（在Search（）中）。我不明白为什么。其次，我得到一个警告，我的整数'j'在第一个for循环（'j ++'）中无法访问。

请帮忙！我确定答案很简单，但我看不到它。

Answer 1

据我所知，你得到的错误是因为如果第一个'for'循环甚至没有运行一次那么你就不会遇到一个return语句。它可能不太可能/不可能，但你仍然需要考虑它。解决这个问题的方法是删除末尾的'else'，这样如果它到达那么远，它肯定会击中'return -1'。

Answer 2

问题似乎在于你的第二个for循环。试试这个：

 if (stringLength >= patternLength)
        {
            for (j = 0; j <= (stringLength - patternLength); j++)
            {
                for (i = 0; i < patternLength && pattern[i] == sequence[i + j]; i++)
                {
                    if (i >= patternLength)
                        return j;                       
                }
            }
        }
 return -1;

那应该删除所有警告和错误并编译。你为什么不使用.Contains()方法？

包含

如果value参数出现在此字符串中，则为true，或者如果   value是空字符串（＆＃34;＆＃34;）;否则，错误。

Answer 3

不返回的代码路由是stringLength = patternLength。

Answer 4

替换

Console.WriteLine(Search(pattern, pattern.Length, sequence, sequence.Length));

与

sequence.IndexOf(pattern);

摆脱你的搜索功能。你正在重写（很差）框架中可用的内容。

简单的子串搜索（蛮力）

4 个答案: