Skip to content

Commit 64a31fb

Browse files
committed
Fixed bug where a NRE would be thrown if no homework was available
1 parent b301497 commit 64a31fb

File tree

1 file changed

+16
-12
lines changed

1 file changed

+16
-12
lines changed

classeviva-net/Classeviva.cs

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -332,20 +332,24 @@ public async Task<Homework[]> GetHomeworkAsync(DateTime startDate, DateTime endD
332332
startDate.Subtract(new DateTime(1970, 1, 1)).TotalSeconds.ToString() + "&end=" +
333333
endDate.Subtract(new DateTime(1970, 1, 1)).TotalSeconds.ToString());
334334
HomeworkBody[] homeworkBody = JsonConvert.DeserializeObject<HomeworkBody[]>(await msg.Content.ReadAsStringAsync());
335-
Homework[] homework = new Homework[ homeworkBody.Length ];
336-
for (int i = 0; i < homeworkBody.Length; i++)
335+
if (homeworkBody != null)
337336
{
338-
homework[ i ] = new Homework(
339-
homeworkBody[ i ].id,
340-
homeworkBody[ i ].title,
341-
DateTime.ParseExact(homeworkBody[ i ].start, "yyyy-MM-dd HH:mm:ss", null),
342-
DateTime.ParseExact(homeworkBody[ i ].end, "yyyy-MM-dd HH:mm:ss", null),
343-
homeworkBody[ i ].allDay,
344-
DateTime.ParseExact(homeworkBody[ i ].data_inserimento, "dd-MM-yyyy HH:mm:ss", null),
345-
homeworkBody[ i ].autore_desc,
346-
homeworkBody[ i ].nota_2);
337+
Homework[] homework = new Homework[homeworkBody.Length];
338+
for (int i = 0; i < homeworkBody.Length; i++)
339+
{
340+
homework[i] = new Homework(
341+
homeworkBody[i].id,
342+
homeworkBody[i].title,
343+
DateTime.ParseExact(homeworkBody[i].start, "yyyy-MM-dd HH:mm:ss", null),
344+
DateTime.ParseExact(homeworkBody[i].end, "yyyy-MM-dd HH:mm:ss", null),
345+
homeworkBody[i].allDay,
346+
DateTime.ParseExact(homeworkBody[i].data_inserimento, "dd-MM-yyyy HH:mm:ss", null),
347+
homeworkBody[i].autore_desc,
348+
homeworkBody[i].nota_2);
349+
}
350+
return homework;
347351
}
348-
return homework;
352+
else return null;
349353
}
350354
}
351355
}

0 commit comments

Comments
 (0)