Почему не создается новый файл в C#?

Question

[Nirmanakaya61]

Помогите, пожалуйста, найти причину почему программа не создает новый файл. Суть проблемы: форма, где можно ввести ip и диапазон портов, кнопка "Scan" (запускает метод ScanPort из NetService.cs), в NetService.cs подключаемся к порту и если он открыт пишем в файл в корне С: строчку, что открыт (в блоке try), если не подключаемся, то в catch пишем в файл строчку, что порт закрыт. Имя файла создается на основе даты-времени в FileService.cs (работа с файлом идет через него). Проблема в том, что при запуске в VS первый раз файл создается, а дальше, если мы не перезапускаем приложение, а повторно нажимаем кнопку Scan, даже изменив ip/порты, либо идет дозапись в данный файл (если StreamWriter file = new StreamWriter(Settings.path, true) ), либо все стирается и пишутся результаты нового сканирования (если стоит false). Хотя по логике должен создаваться новый файл c другим именем. В чем может быть проблема? Может что-то с освобождением ресурсов?

NetService.cs :

using System;
using System.Net.Sockets;
using System.Threading.Tasks;

namespace WinPortScanner
{
    class NetService : IDisposable
    {
        public static void ScanPort(string ip, int startPort, int endPort)
        {
            int closedPort = 0;
            if (startPort > endPort)
            {
                int tempPort = startPort;
                startPort = endPort;
                endPort = tempPort;
            }

            using (FileService fileService = new FileService())
            {
                using (TcpClient client = new TcpClient())
                {

                    for (int port = startPort; port <= endPort; port++)
                    {

                        try
                        {

                            closedPort = port;
                            client.Connect(ip, port);
                            fileService.WriteOpen(ip, port);
                            client.Close();

                        }

                        catch (SocketException e)
                        {

                            fileService.WriteClose(ip, closedPort);

                        }

                    }
                    client.Close();

                }
                fileService.CloseFile();
            }
        }

        public void Dispose()
        {

        }
    }
}

FileSevice.cs :

using System;
using System.Threading.Tasks;

namespace WinPortScanner
{
    class FileService : IDisposable
    {
        private StreamWriter file = new StreamWriter(Settings.path, true);

        public void Dispose()
        {
           
        }

        public void WriteOpen(string ip, int port)
        {
            file.WriteLine($"{DateTime.Now}, {ip}, {port}, Open");
        }

        public void WriteClose(string ip, int port)
        {
            file.WriteLine($"{DateTime.Now}, {ip}, {port}, Close");
        }

        public void CloseFile()
        {
            file.Close();
        }
    }
}

Comments

[freeExec]

Непонятно: вы сформировать имя файла не можете? у вас файл не удаляется? вы не хотите в него дописывать?

[Nirmanakaya61]

freeExec, да, я хочу, чтобы при каждом нажатии кнопки Scan (то есть при запуске метода ScanPort из формы) происходило создание нового файла, даже если аргументы остались прежними (тот же ip и диапазон портов). Файл создается из даты и времени с секундами, так что повторяющихся названий в принципе не должно быть. Но создается только первый файл, а потом программа дописывает или переписывает все в него, это если не запускать программу заново через CTRL-F5, а работать с открытой формой. Хотя по логике (по крайней мере той, которую я сейчас вижу, а я могу ошибаться) при нажатии Scan должен запускаться новый цикл программы, в том числе и создание нового файла.

[freeExec]

Nirmanakaya61, так разберитесь в своём коде, новый вы файл создаёте или в старый пишете.

Answer 1

[Adler]

Что-то я не понял, где у вас Settings.path задается?
Не вижу, что бы этот параметр где-то в ScanPort менялся.

По видимом вы где-то присвоили значение Settings.path и оно больше не меняется.

Comments

[Nirmanakaya61]

Извиняюсь, забыл. Может проблема в static?

Settings.cs:

using System;

namespace WinPortScanner
{
    class Settings
    {
        
        public static string path = @$"c:\report{DateTime.Now.Day}{DateTime.Now.Month}{DateTime.Now.Year}_{DateTime.Now.Hour}{DateTime.Now.Minute}{DateTime.Now.Second}.csv";
    }
}

[Adler]

Ну так у вас и получается, что название файла создается только один раз при запуске программы и не меняется.
Вам нужно его где-то в FileService задавать или ScanPort.

[Nirmanakaya61]

Alexander, задал в FileService, не помогло.

using System;
using System.Threading.Tasks;

namespace WinPortScanner
{
    class FileService : IDisposable
    {
        
        static string path = @$"c:\report{DateTime.Now.Day}{DateTime.Now.Month}{DateTime.Now.Year}_{DateTime.Now.Hour}{DateTime.Now.Minute}{DateTime.Now.Second}.csv";
        private StreamWriter file = new StreamWriter(path, false);



        public void Dispose()
        {

            
        }

        public void WriteOpen(string ip, int port)
        {
            file.WriteLine($"{DateTime.Now}, {ip}, {port}, Open");
        }

        public void WriteClose(string ip, int port)
        {
            file.WriteLine($"{DateTime.Now}, {ip}, {port}, Close");
        }

        public void CloseFile()
        {
            file.Close();
        }

    }
}

[Роман]

Nirmanakaya61, убираем static у поля, и заодно гуглим о статических членах класса. Да реализация IDisposable и использование using явно лишнее, у вас нет понимания зачем это нужно в принципе.