imo, you can simplify this bit var assessments = _model.GetDataList("Assessments", "assessment_name"); if (assessments.Any()) { HashSet<string> items = new HashSet<string>(assessments); assessments = items.ToList(); for (int i = 0; i < assessments.Count; i++) { assessments[i] = (assessments[i] + ": " + _model.CheckStatus(assessments[i])); } _view.ExistingAssessments = assessments; } to this var assessments = _model.GetDataList("Assessments", "assessment_name").Distinct().ToList(); if (assessments.Any()) { assessments.ForEach(a => a += $":{_model.CheckStatus(a)}"); _view.ExistingAssessments = assessments; }