Что делаю не правильно с SSH?

Question

[Андрей Фомин]

Здравствуйте!
Работаю с SSH, надо через него реализовать подключение, возможность производить отключение, отправку команд и вывод результата команд на форму, из всего этого у меня только подключение получилось.
Я объявил метод InitSSH:

public void InitSSH()
        {
            int port = Int32.Parse(txtPort.Text);
            SshClient SSHclient = new SshClient(txtIP.Text, port, txtLogin.Text, txtPassword.Text);
        }

Потом реализовал подключение:

private void btnConnect_Click(object sender, EventArgs e)
        {
            if (txtIP.Text == "" ||  txtPort.Text == "" || txtLogin.Text == ""  || txtPassword.Text == "")
            {
                MessageBox.Show("Заполните все поля!","Ошибка",MessageBoxButtons.OK, MessageBoxIcon.Error);
            }
            else
            {
                InitSSH();
                TimerPoll.Start();
                lblStatus.Text = "Подключён";
                lblStatus.ForeColor = Color.Green;
                btnConnect.Enabled = false;
                btnDisconnect.Enabled = true;
            }
        }

А когда стал реализовывать отключение соединения по нажатию кнопки возникли проблемы:
1. У меня в форме используется tabpage, и из-за этого у меня стоит условие на сначала на выбранную вкладку, а потом на наличие установленного соединения.

private void tabControl1_SelectedIndexChanged(object sender, EventArgs e, object client)
        {
            if (tabControl1.SelectedIndex == 0)
            {
                if (SSHclient.Connected)
                {
                    SSHclient = null;
                    TimerPoll.Stop();
                    lblStatus.Text = "Не подключён";
                    lblStatusPO.Text = "Не подключён";
                    OVEN_info.Items.Clear();
                }
            }

В обоих случаях у меня ошибка "Имя "SSHclient" не существует в текущем контексте"
2. Посылать команды насколько я понял можно с помощью "RunCommand" и саму команду через textbox задаю, а вот как обрабатывать результат? Его парсить надо или в переменную завернуть в listbox вывести? я именно сам механизм не понимаю, как это должно работать.
Заранее спасибо.

Comments

[Vindicar]

Правила ресурса запрещают скриншоты кода. Кнпока оформления кода (</>) в помощь.

Answer 1

[Vindicar]

Учи язык, что тут скажешь. Точнее, читай про области видимости и время жизни объекта.
У тебя SSHClient - локальная переменная в методе InitSSH(). Она существует, пока метод выполняется, и видна только внутри этого метода. По завершению метода она уничтожается.
Если тебе нужно, чтобы время жизни объекта превышало время выполнения метода, т.е. чтобы объект не уничтожался по его завершении - объект должен лежать в атрибуте класса. Тогда его время жизни будет (как правило) равно времени жизни экземпляра класса (например, формы), а область видимости будет покрывать все методы класса.

Comments

[Андрей Фомин]

Vindicar, я правильно понял что по хорошему надо InitSSH поместить в класс, а потом уже пробовать делать с ним то, что нужно мне?
И еще момент - что скажете насчет механизма реализации вывода результата работы команды?

[Vindicar]

Андрей Фомин,

по хорошему надо InitSSH поместить в класс

Ну вообще в C# вроде нет свободно-стоящих функций, там весь код - методы, т.е. члены того или иного класса. Хотя я давненько с ним не работал, и мог подзабыть.
Но сам по себе перенос InitSSH() внутрь класса ничего не даст, тебе также надо переделать локальную переменную на атрибут этого класса.

[Андрей Фомин]

Vindicar,
Ну я тут попробовал переделать немного код и получилось следующее:
Я создал класс SSH и в качестве атрибутов задал параметры которые нужны для подключения:

public class SSH
        {
            public string ip;
            public int port;
            public string login;
            public string password;
        }

Я вообще в нужную сторону двигаюсь?

[Vindicar]

Андрей Фомин, у тебя проблема-то была не с этим. Я фз что тебе ещё сказать, тебе прямым текстом вроде говорят: вынеси объявление SshClient SSHclient; за пределы метода InitSSH().

[Андрей Фомин]

Vindicar,
Ладно, сделаем то что посоветовали:

SshClient SSHclient = new SshClient(txtIP.Text, port, txtLogin.Text, txtPassword.Text);

InitSSH вообще убрал, оставил только это, ругается на параметры:
"Инициализатор поля не может обращаться к нестатическому полю, методу или свойству".

[Vindicar]

Андрей Фомин, ну вот тут у тебя ещё один пробел.
Ты путаешь объявление поля и его инициализацию.
Тебе никто не запрещает объявить поле

SshClient SSHclient;
// или, если очень хочется иметь значение по умолчанию
SshClient SSHclient = null;

А потом, в конструкторе класса или в отдельном методе, уже его инициализировать

void InitSSH()
{
    this.SSHclient = new SshClient(txtIP.Text, port, txtLogin.Text, txtPassword.Text);
}

Вот кроме шуток, отложи свой проект и поучи основы языка. Без них ты далеко не уедешь.

Answer 2

[Alexander]

Объяви SshClient за пределами InitSSH, а в нем лишь присвой ему значение.

Comments

[Андрей Фомин]

Alexander,
Это вы касаемо отключения рекомендуете так сделать, или в каком месте нужно это реализовать?

[Alexander]

Как вам уже сказали, читайте азы языка про области видимости объектов, тогда все поймете.
В двух словах, у вас SshClient существует только в методе InitSSH, существует пока тот выполняется и больше не доступна ни откуда. Что бы с ней что-то можно было сделать с другого метода, нужно поменять ее область видимости.

Answer 3

[Василий Банников]

Тот SSHClient, который ты инициализируешь в InitSSH - это локальная переменная.
Тебе следует обращаться к this.SSHclient => нужно ещё создать поле в классе с таким именем и типом.

Comments

[Андрей Фомин]

Василий Банников,
То есть выглядеть это должно что-то около этого?

public void InitSSH()
        {
            int port = Int32.Parse(txtPort.Text);
            this.SSHclient = new SshClient(txtIP.Text, port, txtLogin.Text, txtPassword.Text);
        }

И еще:

нужно ещё создать поле в классе с таким именем и типом

А не могли бы объяснить поподробнее для чего это нужно и привести пример?

[Василий Банников]

Андрей Фомин, ну типа

public class /*Какая-то там форма*/
{
  private SshClient SSHclient;
  
  // ...

  private void InitSSH()
  {
    int port = Int32.Parse(txtPort.Text);
    this.SSHclient = new SshClient(txtIP.Text, port, txtLogin.Text, txtPassword.Text);
  }

  // ...
  private void btnConnect_Click(object sender, EventArgs e)
  {
    if (txtIP.Text == "" ||  txtPort.Text == "" || txtLogin.Text == ""  || txtPassword.Text == "")
    {
      MessageBox.Show("Заполните все поля!","Ошибка",MessageBoxButtons.OK, MessageBoxIcon.Error);
    }
    else
    {
      InitSSH();
      TimerPoll.Start();
      lblStatus.Text = "Подключён";
      lblStatus.ForeColor = Color.Green;
      btnConnect.Enabled = false;
      btnDisconnect.Enabled = true;
    }
  }

  // ...

  private void tabControl1_SelectedIndexChanged(object sender, EventArgs e, object client)
  {
    if (tabControl1.SelectedIndex == 0)
    {
      if (SSHclient.Connected)
      {
        // Перед тем как делать = null следует ещё закрыть соединение и вызывать Dispose,
        // иначе у тебя будет утечка ресурсов
        SSHclient = null;
        TimerPoll.Stop();
        lblStatus.Text = "Не подключён";
        lblStatusPO.Text = "Не подключён";
        OVEN_info.Items.Clear();
      }
    }
  }
}