1

文字を見つけるためにチェックし、「C」を追加して大文字かどうかをチェックし、文字列の残りの文字を続行するループを他にどのように実行できるかわかりません。私は C# に慣れていないので、ここで何が起こっているのかを説明するのはあまり得意ではありません。しかし、'for' ループの 'i' は 0 にリセットされないため、間違った出力を受け取ることになります。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace Encrypt
{
    class Program
    {
        static char[] alphabet = new char[52] { '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', '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' };
        static void Main(string[] args)
        {

            string input = Console.ReadLine();
            string output = Decode(input);
            Console.WriteLine(output);
            Console.ReadLine();

        }

        static string Decode(string input)
        {
            string output = "";
            foreach (Char c in input)
            {
                for (int i = 0; i < alphabet.GetLength(0); i++)
                {
                    if (c == alphabet[i])
                    {
                        int intoutput = i + 1;
                        if (intoutput > 26)
                        {
                            intoutput = intoutput - 26;
                            output = "C" + intoutput + " ";
                        }
                        else
                        {
                            output = intoutput + " ";
                        }
                    }
                }
            }
        return output;
        } 
    }
}
4

5 に答える 5

0

あなたの問題はリセットではなくi、あなたの問題です

output = "C" + intoutput + " ";
...
output = intoutput + " ";

これらは に追加されずoutput、 に割り当てられoutputます。つまり、以前に割り当てられた値は失われます。あなたは次のようなものが欲しいでしょう

output = output + "C" + intoutput + " ";
...
output = output + intoutput + " ";
于 2013-02-20T09:13:48.087 に答える
0

各反復で出力をオーバーライドしています。

出力 += "C" + intoutput + " "; 出力 += intoutput + " ";

于 2013-02-20T09:22:33.720 に答える
0
  var output = new StringBuilder();
  foreach (Char c in input)
  {
    for (int i = 0; i < alphabet.Length; i++)
    {
      if (c == alphabet[i])
      {
        if (Char.IsUpper(c))
          output.AppendFormat("C{0} ", i-26); 
        else
          output.AppendFormat("{0} ",i);
      }
    }
  }
  return output.ToString();

また

  return string.Join(" ", input.ToCharArray().Select(c =>
  {
    for (int i = 0; i < alphabet.Length; i++)
    {
      if (c == alphabet[i])
        return Char.IsUpper(c) ?
          string.Format("C{0}", i - 26)
          :
          i.ToString();
    }
    return "";
  }).ToArray()).Trim();

またはループなし

  //static List<char> alphabet = new List<char> { 'a', 'b', 'c'...};
  int i;
  return string.Join(" ", input.ToCharArray().Select(c =>
    (i = alphabet.IndexOf(c)) != -1 ?
      (Char.IsUpper(c) ?
        string.Format("C{0}", i - 26)
        :
        i.ToString()
      )
      :
     ""
  ).ToArray());
}

そして最後にアルファベット配列なし

  return string.Join(" ", input.ToCharArray().Select(c =>
    Char.IsLetter(c) ?
      (Char.IsUpper(c) ?
        string.Format("C{0}", c-'A')
        :
        (c-'a').ToString()
      )
      :
     ""
  ).ToArray()).Trim();
于 2013-02-20T09:35:39.377 に答える
0

あなたは、別の値に再割り当てすると言いますが、それにoutput =追加する のではなく、あなたが望む.outputoutput +=

また、アルファベット文字は文字セット ( 'b' = 'a' + 1) に順番に格納されます。これは機能するはずです。

foreach (Char c in input)
{
  if ('A' <= c <= 'Z')
    output += "C" + (int)(c - 'A') + " ";
  else if ('a' <= c <= 'z')
    output += (int)(c - 'a') + " ";
}

編集:別のバージョン:

foreach (Char c in input)
{
  if (Char.IsUpper(c))
    output += "C" + (int)(c - 'A') + " ";
  else if (Char.IsLower(c))
    output += (int)(c - 'a') + " ";
}

上記の両方は、1 ~ 52 個の値をループする必要がなく、1 ~ 4 個の比較を行うだけなので、アルゴリズムよりもはるかに高速です。

于 2013-02-20T09:27:04.327 に答える
0

ここであなたの答えは完成します

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace Encrypt
{
  class Program
  {
        static char[] alphabet = new char[52] { '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', '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' };

       static void Main(string[] args)
       {
            string input = Console.ReadLine();
            string output = Decode(input);
            Console.WriteLine(output);
            Console.ReadLine();
       }

       static string Decode(string input)
       {
            string output = "";
            foreach (Char c in input)
            {
               for (int i = 0; i < alphabet.GetLength(0); i++)
               {
                  if (c == alphabet[i])
                  {
                     int intoutput = i + 1;
                     if (intoutput > 26)
                     {
                         intoutput = intoutput - 26;
                         output = output + "C" + intoutput + " ";
                     }
                     else
                     {
                         output = output + intoutput + " ";
                     }
                   }
                }
            }
         return output;
       } 
     }
  }
于 2013-12-06T20:04:53.650 に答える