另存为使用提供的文件名,但稍后在代码中更改为 null - 这导致文件永远不会被创建

Posted

技术标签:

【中文标题】另存为使用提供的文件名,但稍后在代码中更改为 null - 这导致文件永远不会被创建【英文标题】:Save as is taking the filename provided, but is changing to null later in the code - which is causing the file never to be created 【发布时间】:2012-11-21 15:40:15 【问题描述】:

所以,我上次有一个关于语法的问题,现在语法错误已修复,我有一个问题,即使我的教授看过之后,他也不知道如何解决。我们逐行浏览了我的代码,并通过初始的另存为对话框,一切看起来都很好,并且文件名/路径显示在调试器中。它传递到创建文件行,然后传递到我必须添加以使我的语法正常工作的代码 - 然后继续到我试图打开文件的位置,以便能够使用带有我的随机数的 writeline 命令生成器 - 而不是打开适当的文件,它变成“空”作为一个值!但它并没有就此停止,它继续到随机数生成器,并滚动所需数量的随机数,但当然,由于开始值显示为“null”,它不会像预期的那样保存到文件中。哦,我教科书中的代码是产生第一个语法错误的原因,但没有提供修复它的方法。这是代码,如果它很长/难以阅读,请见谅。

using System.IO; // Added to be able to use StreamWriter variable type

namespace Random_Number_File_Writer

public partial class Form1 : Form
 
    StreamWriter randomNumberFile; //Name streamwriter
    decimal numbers; //Variable to insert the number up down value into
    Random rand1 = new Random(); //Random number generator
    int writeitem; // Variable to insert random number into, to write.

    public Form1()
    

        InitializeComponent();
    
    public void saveFileDialog1_FileOk(object sender, CancelEventArgs e)
    
    

    private void generateButton_Click(object sender, EventArgs e)
    
        try
        
            //Initial opening point for save file dialogue
            saveFileDialog1.InitialDirectory = @"C:\Users\Heather\Documents\Visual Studio 2010\Projects\Random Number File Writer";
            //Save As popup - Opening the file for writing usage once it's created.
            if(saveFileDialog1.ShowDialog() == DialogResult.OK)
            
                randomNumberFile = File.CreateText(openFileDialog1.FileName);
            
            else // Popup informing user that the data will not save to a file because they didn't save.
            
                MessageBox.Show("You elected not to save your data.");
            

            numbers = numericUpDown1.Value; //Gathering the number of numbers to generate from the number box.

            while (numbers > 0) // Loop counting down to 0 to give the user the appropriate number of requested random numbers.
            

                writeitem = rand1.Next(101); // Random number generated.
                randomNumberFile.WriteLine(writeitem); //Random number written to file
                numbers--; // Initial number for user input decremented so that loop will have an ending and user only gets the amount of randoms asked for.
            
            randomNumberFile.Close();
        

我只包括了相关的部分——我确实有一点点,但那只是用于退出/清除按钮,调试器根本不会跳转到它们,所以我剪掉了多余的部分。

【问题讨论】:

您正在尝试写入文件,即使用户说不这样做。当您打印“您选择不保存数据”时,您应该从该方法返回... 【参考方案1】:

您将openFileDialog1.FilenameFile.CreateText 一起使用,但在上面您使用saveFileDialog1

【讨论】:

另外,打你的教授的脸,因为他显然不明白范围的重要性。如果编写得当,您可以避免这种情况。 是的,说真的,如果教授逐行通过调试器看不到这个,我会感到害怕...... 马里奥,想想自己被喜悦的尖叫所拥抱。有用!谢谢! @HeatherT 看看我的答案,以及您目前设置方法的方式遇到问题。尝试取消保存文件对话框,你的程序仍然会尝试将随机数写入一个不好的文本文件【参考方案2】:

对我来说,您的 while 数字循环位于保存文件对话框的 if else 逻辑之外,这对我来说毫无意义。如果他们没有选择文件,那么您为什么还要尝试将随机数写入文件?在 if 语句中移动 while 循环。

此外,正如马里奥指出的那样,您在 2 个不同的对话框中使用了不匹配的文件名,因此这是问题的根本问题,但我建议您同时修复这两个问题以避免将来出现问题。

try
    
        //Initial opening point for save file dialogue
        saveFileDialog1.InitialDirectory = @"C:\Users\Heather\Documents\Visual Studio 2010\Projects\Random Number File Writer";
        //Save As popup - Opening the file for writing usage once it's created.
        if(saveFileDialog1.ShowDialog() == DialogResult.OK)
        
            randomNumberFile = File.CreateText(saveFileDialog1.FileName);
            numbers = numericUpDown1.Value; //Gathering the number of numbers to generate from the number box.
            while (numbers > 0) // Loop counting down to 0 to give the user the appropriate number of requested random numbers.
            
                writeitem = rand1.Next(101); // Random number generated.
                randomNumberFile.WriteLine(writeitem); //Random number written to file
                numbers--; // Initial number for user input decremented so that loop will have an ending and user only gets the amount of randoms asked for.
                randomNumberFile.Close();
            
        
        else // Popup informing user that the data will not save to a file because they didn't save.
        
            MessageBox.Show("You elected not to save your data.");
        
    

【讨论】:

虽然您解决了一个有效问题,但不建议使用大的 if 正文,并且不会解决最初的问题。 我意识到我的回答并没有解决问题的根本问题并解决了这个问题。但是,将 while 循环添加到检查有效保存文件的 if 肯定不会创建“大” if 块,更不用说这里的逻辑需要它。如果他通过了用户没有选择保存文件的 else ,他仍在尝试将随机数写入文本文件! 这可以通过制作elsereturn来解决。任何大于一行的if 块都可以被认为是大的。 虽然这可能是一个偏好问题,但我认为 else 语句中的随机返回会阻止进一步的代码执行是非常糟糕的形式,我认为这里的很多人都会同意我的观点。 我认为您指的是单进单出,这在 60 年代确实被认为是一件好事。今天它更像是快速失败。大if 块不仅仅是风格问题,它们是等待发生的意外。如果还有其他条件呢?嵌套另一个 if?

以上是关于另存为使用提供的文件名,但稍后在代码中更改为 null - 这导致文件永远不会被创建的主要内容,如果未能解决你的问题,请参考以下文章

VBA 另存为结果未定义的文件类型

保存前不适用于另存为

另存为提示位置但使用单元格中的文件名

将excel另存为pdf,将其方向更改为水平

VBA 另存为 PDF 文件名作为单元格值

“另存为”对话框文件名