Reply to thread

Message

<blockquote data-quote="CodeMonkey" data-source="post: 632247" data-attributes="member: 10548">Nice little client!There is a problem with the source though, it won't build the TvViewerSetupCore project because it is referencing some gfx (icons etc) files located in your local c:\media directory <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite1" alt=":)" title="Smile&nbsp; &nbsp; :)" loading="lazy" data-shortname=":)" />I have some code change suggestions as well.&nbsp; I was trying to run it at work with no MP installed anywhere so these are almost all error handling changes.In ServerStatusForm.Designer.csfrom:[code]this.timerRefreshStatus.Interval = 1000;[/code]to:[code]this.timerRefreshStatus.Interval = 10000;[/code]The 1 second refresh is a little harsh on the system if you are supposed to be a lightweight client <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite1" alt=":)" title="Smile&nbsp; &nbsp; :)" loading="lazy" data-shortname=":)" />In ServerStatusForm.cs&nbsp;&nbsp; timerRefreshStatus_Tick():Change the exception handling to:[code]&nbsp; &nbsp; &nbsp; &nbsp; timerRefreshStatus.Stop();&nbsp; &nbsp; &nbsp; &nbsp; MessageBox.Show(&quot;Error retrieving server states, maybe connection down. Exception: &quot; + ex.Message);[/code]No point keeping the timer going (particularly at once a second like it was <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite1" alt=":)" title="Smile&nbsp; &nbsp; :)" loading="lazy" data-shortname=":)" /> ) if there is no web service to talk to, is there?In SchedulesForm.cs&nbsp; constructor:add a try/catch:[code]&nbsp; &nbsp; public SchedulesForm(ServiceInterface _server)&nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; InitializeComponent();&nbsp; &nbsp; &nbsp; m_server = _server;&nbsp; &nbsp; &nbsp; try&nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; WebSchedule[] schedules = m_server.GetAllSchedules();&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; FillScheduleListView(schedules);&nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; &nbsp; catch (Exception ex)&nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; MessageBox.Show(&quot;Error retrieving schedules, maybe connection down. Exception: &quot; + ex.Message);&nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; }[/code]in MainForm.csAdd some error handling in ShowTvGuide()[code]&nbsp; &nbsp; private void ShowTvGuide()&nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; int index = m_showAllGroup ? cbGroups.SelectedIndex : cbGroups.SelectedIndex + 1;&nbsp; &nbsp; &nbsp; if (m_groups != null)&nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; WebChannelGroup selectedChannel = m_groups[index];&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; EpgForm frmEpg = new EpgForm(m_server, selectedChannel);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; frmEpg.Show();&nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; &nbsp; else&nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; MessageBox.Show(&quot;Error attempting to show EPG.&nbsp; Check your configuration.&quot;);&nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; }[/code]That's it, nice little client from the looks of it!</blockquote>

[QUOTE="CodeMonkey, post: 632247, member: 10548"] Nice little client! There is a problem with the source though, it won't build the TvViewerSetupCore project because it is referencing some gfx (icons etc) files located in your local c:\media directory :) I have some code change suggestions as well. I was trying to run it at work with no MP installed anywhere so these are almost all error handling changes. In ServerStatusForm.Designer.cs from: [code]this.timerRefreshStatus.Interval = 1000;[/code] to: [code]this.timerRefreshStatus.Interval = 10000;[/code] The 1 second refresh is a little harsh on the system if you are supposed to be a lightweight client :) In ServerStatusForm.cs timerRefreshStatus_Tick(): Change the exception handling to: [code] timerRefreshStatus.Stop(); MessageBox.Show("Error retrieving server states, maybe connection down. Exception: " + ex.Message);[/code] No point keeping the timer going (particularly at once a second like it was :) ) if there is no web service to talk to, is there? In SchedulesForm.cs constructor: add a try/catch: [code] public SchedulesForm(ServiceInterface _server) { InitializeComponent(); m_server = _server; try { WebSchedule[] schedules = m_server.GetAllSchedules(); FillScheduleListView(schedules); } catch (Exception ex) { MessageBox.Show("Error retrieving schedules, maybe connection down. Exception: " + ex.Message); } }[/code] in MainForm.cs Add some error handling in ShowTvGuide() [code] private void ShowTvGuide() { int index = m_showAllGroup ? cbGroups.SelectedIndex : cbGroups.SelectedIndex + 1; if (m_groups != null) { WebChannelGroup selectedChannel = m_groups[index]; EpgForm frmEpg = new EpgForm(m_server, selectedChannel); frmEpg.Show(); } else { MessageBox.Show("Error attempting to show EPG. Check your configuration."); } }[/code] That's it, nice little client from the looks of it! [/QUOTE]

Verification