NullReferenceException, при возврате структуры из функции?

Question

[Максим]

Фукция для создания структуры

private FormData CreateFormData()
        {
            int speed = 0;
            try
            {
                speed = Convert.ToInt32(cbSpeed.SelectedItem);
            } catch (Exception e) { if (ExceptionEvent != null) ExceptionEvent(e); }

//Создание структуры с передачей пареметров string, int, string.
            return new FormData(cbCOM.SelectedItem.ToString(), speed, tbSendText.Text);  //.NullReferenceException
        }

Структура

public struct FormData
        {
            public string Port;
            public int Baud;
            public string Messenge;
            public FormData(string port, int baud, string mess) { Port = port; Baud = baud; Messenge = mess; }
        }

Comments

[Артур Нуруллин]

Ловите однострочника!!!

[Максим]

Артур Нуруллин: Я пока учусь. но в

return new FormData(cbCOM.SelectedItem.ToString(), speed, tbSendText.Text);

нет ничего не понятного. Я уже заменил вызов, и сделал отдельную переменную.

Простой конструктор, в котором смотреть нечего.

public FormData(string port, int baud, string mess) { Port = port; Baud = baud; Messenge = mess; }

ИМХО, в однострочестве нет ничего страшного. Мне жалко выделять ЦЕЛУЮ переменную для такой простой цели, как передачи свойства другого класса в конструктор ( #чистыйC ).

P.S. если я неправ, то напишите в чём.

[Артур Нуруллин]

Максим: Вы еще учитесь, тут нет никакого ИМХО. За такое обычно по пальцам дают. Про одну строку я имел в виду ваш второй отрезок кода.

Answer 1

[Станислав Макаров]

скорее всего это cbCOM.SelectedItem, у которого вы пытаетесь вызвать ToString().

Comments

[Максим]

Спасибо, помогло. Появился ещё один вопросик. Как эту структуру привести к EventArgs?

[Станислав Макаров]

Максим саму структуру вы никак не приведёте, если по-нормальному, то создайте класс-наследник от EventArgs и поместите туда поле типа структуры. Впрочем, не совсем понятно зачем вам это)

[Максим]

Станислав Макаров: передать структуру в событие

[Максим]

Может поменять делегат EventHandler в Action?

[Станислав Макаров]

Максим по соглашению (сам язык этого не требует) у событий должна быть сигнатура вида (object sender, EventArgs args), поэтому лучше всё-таки унаследоваться от EventArgs. Вы можете создать простейший класс:

class FormEventArgs : EventArgs
{
	private readonly FormData formData;

	public FormEventArgs(FormData formData)
	{
		this.formData = formData;
	}

	public FormData FormData
	{
		get { return formData; }
	}
}

[Максим]

Станислав Макаров: Сделаю класс в классе. Спасибо!

[Максим]

Станислав Макаров: Неожиданно заметил, что можно EventHandler, но немного подумав, что структурам нельзя присвоить null, решил оставить класс.

Answer 2

[Роман]

смотрите параметры конструктора структуры, string это объект и может быть null.

Comments

[Станислав Макаров]

и кто в таком случае будет бросать исключение?

[Роман]

Станислав Макаров: ТСу кажется, что конструктор структуры кидает исключение, все в одной строчке, и отладчик на неё встает, а посмотреть кто вызвал исключение ему видать не судьба, как вы и написали скорее всего это cbCOM.SelectedItem

Answer 3

[Anatoliy Mikhailov]

return new FormData(cbCOM?.SelectedItem?.ToString(), speed, tbSendText?.Text);

https://msdn.microsoft.com/en-us/library/dn986595.aspx