Question

我是C＃的新程序员。我遇到的问题是我的字符频率读取程序没有正确显示ASCII值。它应该做的是从文本文件中读取，将全部大写转换为小写，显示ASCII值，出现频率，以及每个字符出现在文件中的字符总数百分比，然后按频率对列表进行排序。以下是我目前的代码：

using System;
using System.Collections.Generic;
using System.Collections;
using System.Text;
using System.IO;
using LibUtil;
using LibDate;

namespace Ch8Prb3
{
  class Program
  {
    const string INPUT_FILE_NAME = "\\CIS210\\Ch8Prb3\\TextDat.Txt";
    const string OUTPUT_FILE_NAME = "\\CIS210\\Ch8Prb3\\Ch8Prb3Rpt.Txt";

    int count;
    static StreamReader fileIn;
    static StreamWriter fileOut;

    static void Main()
    {
      ConsoleApp.ClrScr(); IdentifyApplication();
      OpenFiles(); LetterFreq();
      CloseFiles();
    }

    static void IdentifyApplication()
    {
      Console.WriteLine();
      Console.WriteLine("Application: Ch8Prb3 -- Find and display a character-frequency  ");
      Console.WriteLine("                        report of only letter chracters and their");
      Console.WriteLine("                        ASCII values from a text file.");
      Console.WriteLine();
    }

    static void OpenFiles()
    {
      try
      {
        fileIn = File.OpenText(INPUT_FILE_NAME);
        Console.WriteLine("{0} was opened", INPUT_FILE_NAME);
      }
      catch
      {
        Console.WriteLine("Error: {0} does not exist\n", INPUT_FILE_NAME);
        ConsoleApp.Exit();
      }

      try
      {
        fileOut = File.CreateText(OUTPUT_FILE_NAME);
        Console.WriteLine("{0} was created\n", OUTPUT_FILE_NAME);
      }
      catch
      {
        Console.WriteLine("Error: {0} could not be created\n", OUTPUT_FILE_NAME);
        ConsoleApp.Exit();
      }
    }

    static void LetterFreq()
    {
      int[] c = new int[(int)char.MaxValue];
      int total = 0;
      int j = 0;

      string s = File.ReadAllText("\\CIS210\\Ch8Prb3\\TextDat.Txt");
      string l = Convert.ToString(Encoding.ASCII.GetBytes(s));

      s = System.Threading.Thread.CurrentThread.CurrentCulture.TextInfo.ToLower(s.ToLower());      
      double percent;

      foreach (char t in s)
      {
        c[(int)t]++;
      }

      PrintHeader();

      for (int i = 0; i < (int)char.MaxValue; i++)
      {
        if (c[i] > 0 && char.IsLetter((char)i))
        {
          total += c[i];
          percent = c[i] / total * 100;          
          fileOut.WriteLine("  {0}  {1,3}  {2,3}  {3,2:f2}", (char)i, l, c[i], percent);
        }        
      }

      fileOut.WriteLine();
      fileOut.WriteLine("Number of Characters: {0}", total);
    }   

    static void PrintHeader()
    {
      fileOut.WriteLine("   Chapter 8 Problem 3");
      fileOut.WriteLine("Character Frequency Report");
      fileOut.WriteLine("      {0:MM/dd/yyyy}", Date.Today);
      fileOut.WriteLine();
      fileOut.WriteLine("      ASCII               ");
      fileOut.WriteLine("Char  Value  Freq  Percent");
      fileOut.WriteLine("----  -----  ----  -------");
    }

    static void CloseFiles()
    {
      fileIn.Close(); fileOut.Close();
    }
  }
}

我没有获得每个字符的ASCII值，而是将System.Byte []一直放到ASCII列中。

帮助！

Answer 1

这是问题所在：

Convert.ToString(Encoding.ASCII.GetBytes(s));

Encoding.GetBytes(string)会返回byte[]，并在其上调用Convert.ToString()只会返回System.Byte[]。

目前尚不清楚为什么你在这里使用Encoding.ASCII - 你已经将文件的内容作为字符串添加到上一行：

string s = File.ReadAllText("\\CIS210\\Ch8Prb3\\TextDat.Txt");

只需使用它而不是l。说实话，目前还不清楚你对l的期望是什么。如果您指的是角色的Unicode代码点，请使用“ASCII值”，只需将i投射到int：

fileOut.WriteLine("  {0}  {1,3}  {2,3}  {3,2:f2}", (char)i, (int) i, c[i], percent);

哦，你可能也希望改变你的计算方式percent - 你正在使用整数运算，所以结果总是为0。

顺便说一句，还不清楚为什么你要将琴弦两次套管。你不觉得一次就够了吗？你为什么要低下套管呢？

哦，你打开一个输入文件，但从不对它做任何事情。为什么呢？

关于你的代码还有其他一些我要改变的东西 - 尤其是布局（每行一个语句，几乎总是！） - 但是这样做是为了一个开始。

Answer 2

你可以使用list和groupby来大大压缩你的程序：

class Program
{
    const string INPUT_FILE_NAME = "\\CIS210\\Ch8Prb3\\TextDat.Txt";

    static void Main(string[] args)
    {
        var s = System.IO.File.ReadAllText(INPUT_FILE_NAME).ToLower();
        var list = s.ToList();
        var group = list.GroupBy(i => i);

        foreach (var g in group)
        {
            Console.WriteLine("{0} {1}", g.Key, g.Count());
        }

        Console.ReadLine();
    }
}

获取System.Byte []而不是ASCII值

2 个答案: