2

次のように結果を表示しようとしています。

-.-|-    [tab]  kt   
-.|--    [tab]  nm  
-.|-|-   [tab]  ntt

しかし、これは私の現在の出力です

-.-|-|  kt  
-.|--|  nm  
-.|-|-|  [tab]ntt

|すべてのモールス信号の最後にあるので、これを削除したいと思います。

また、ユーザーはドットとダッシュの間にスペースを入れてモールス信号を入力できるため、文字の配置に影響し、すべてが適切にタブ化されるわけではないことに気付きました。タブという単語は、実際のタブの配置方法がわからなかったので、書き込んだだけで表示されるはずがありません。

private static readonly IDictionary<char, string> morseCode_alpha = new Dictionary<char, string>
{
    {'a', ".-"},{'b',"-..."}, {'c',"-.-."}, {'d',"-.."}, {'e',"."},
    {'f',"..-."}, {'g',"--."}, {'h',"...."},{'i',".."}, {'j',".---"}, 
    {'k',"-.-"}, {'l',".-.."}, {'m',"--"}, {'n',"-."}, {'o',"---"}, 
    {'p',".--."}, {'q',"--.-"}, {'r',".-."}, {'s',"..."}, {'t',"-"}, 
    {'u',"..-"}, {'v',"...-"}, {'w',".--"}, {'x',"-..-"}, {'y',"-.--"}, {'z',"--.."}
};

private static string ConvertMorseToText(string symbolCode)
{
    var builder = new StringBuilder(4 * symbolCode.Length);

    foreach (char c in symbolCode)
        builder.Append(morseCode_alpha[c]);
    return builder.ToString();
}

private static string ConvertTextToMorse(char ch)
{
    if (morseCode_alpha.Keys.Contains(ch))
        return morseCode_alpha[ch];
    else
        return string.Empty;
}

private static string ConvertStringToMorse(string letters)
{
    StringBuilder sb = new StringBuilder();
    foreach (char ch in letters)
    {
        if (sb.Length != 0 && sb[sb.Length - 1] != ' ')
            sb.Append("|");
        sb.Append(ConvertTextToMorse(ch));
    }
    return sb.ToString();
}

private static IEnumerable<string> Permutations( string symbolCode)
{
    int n = symbolCode.Length;
    if (n == 0 || symbolCode.Length == 0)
        yield return " ";
    else
        foreach (var entry in morseCode_alpha)
            if (symbolCode.StartsWith(entry.Value))
                foreach (string next in Permutations(symbolCode.Substring(entry.Value.Length)))
                    yield return entry.Key + next;
}

private static void Write( string rest)
{
    string result = ConvertStringToMorse(rest);
    Console.Write(result+"\t");
    Console.WriteLine(rest);
}

static void Main(string[] args)
{
    string morseInput;
    string entered = "";
    do
    {
        Console.WriteLine("Enter Morse Code: \n");
        morseInput = Console.ReadLine().Replace(" ","");
        bool isValid = Regex.IsMatch(morseInput, @"^[-.]+$");
        if (isValid)
        {
            Console.WriteLine("\nAll permutations:\n");
            string morse = ConvertMorseToText(entered);
            string permutations = morseInput.Substring(morse.Length);
            Write(permutations);
            var nexts = new List<string>(Permutations(permutations));
            foreach (string next in nexts)
                Write(next);
        }
        else
        {
            Console.WriteLine("\nFormat of morse must be only dots and dashes.");
            Console.WriteLine("Parameter name: "+morseInput+"\n");
        }
    }
    while (morseInput.Length != 0);
}
4

3 に答える 3

3

そして、質問の他の部分に答えるために...

タブストップはコンソール書き込み用に固定されているため、String.PadRight のようなものを使用することをお勧めします。

したがって、コードは次のようになります。

private static void Write(string rest)
{
    string result = ConvertStringToMorse(rest);
    Console.Write(result.PadRight(20));
    Console.WriteLine(rest);
}
于 2013-01-28T21:53:55.310 に答える
2

メソッドのドラフト版:

private static string ConvertStringToMorse(string letters)
{
    var result = string.Join("|",
        letters
            .Select(ConvertTextToMorse)
            .Where(morse => !string.IsNullOrEmpty(morse)));
    return result;
}

アップデート：

entered変数は一度だけ使用されることに注意してください: 定義されたとき -空の文字列が割り当てられます。次に、ConvertMorseToText(entered)メソッドが呼び出されます。空の文字列引数に対して常に空の文字列が返されます。この割り当てstring permutations = morseInput.Substring(morse.Length);の後、permutations変数は変数とまったく同じ値を格納しmorseます (morse.Length常に 0 であるため)。

したがって、entered変数とConvertMorseToText()メソッドは役に立たないようです (どちらも安全に削除できます)。

static void Main(string[] args)
{
    do
    {
        Console.WriteLine("Enter Morse Code: ");

        string morseInput = Console.ReadLine();
        if (string.IsNullOrWhiteSpace(morseInput))
        {
            // Empty or consists only of white-space characters
            break;
        }

        morseInput = morseInput.Replace(" ", "");

        bool isValid = Regex.IsMatch(morseInput, @"^[-.]+$");
        if (isValid)
        {
            Console.WriteLine("All permutations:");
            Console.WriteLine();
            var nexts = Permutations(morseInput).ToList();
            foreach (string next in nexts)
                Write(next);
        }
        else
        {
            Console.WriteLine();
            Console.WriteLine("Format of morse must be only dots and dashes.");
            Console.WriteLine("Parameter name: {0}", morseInput);
        }
    }
    while (true);
}

更新 2: and (インデクサー) の代わりにTryGetValue()メソッドを使用することを検討してください。つまり、ルックアップを 2 回実行しないでください。Dictionary<TKey, TValue>Keys.Contains[]

private static string ConvertTextToMorse(char ch)
{
    string result;
    return morseCode_alpha.TryGetValue(ch, out result) ? result : string.Empty;
}
于 2013-01-28T21:55:06.017 に答える
1

代わりにこのコード:

Console.Write(result+"\t");
Console.WriteLine(rest);

使用する

Console.WriteLine("{0,-10}{1,-10}", result, rest);

次に、左揃えの 2 つの列 (それぞれ最大 10 文字) が表示されます。または、右揃えが必要な場合は「-」記号を削除します。

于 2013-01-28T21:53:20.807 に答える