Invert the if

This commit is contained in:
Claus Vium 2019-08-16 19:58:44 +02:00
parent 26b4fb21fe
commit 15f7a2078b

View file

@ -205,38 +205,41 @@ namespace MediaBrowser.Providers.TV.TheTVDB
var currentActor = episode.GuestStars[i]; var currentActor = episode.GuestStars[i];
var roleStartIndex = currentActor.IndexOf('('); var roleStartIndex = currentActor.IndexOf('(');
string name = currentActor; if (roleStartIndex == -1)
string role = string.Empty;
if (roleStartIndex != -1)
{ {
var roles = new List<string> {currentActor.Substring(roleStartIndex + 1)}; result.AddPerson(new PersonInfo
name = name.Substring(0, roleStartIndex).Trim();
// Fetch all roles
for (var j = i + 1; j < episode.GuestStars.Length; ++j)
{ {
var currentRole = episode.GuestStars[j]; Type = PersonType.GuestStar,
var roleEndIndex = currentRole.IndexOf(')'); Name = currentActor,
Role = string.Empty
});
continue;
}
if (roleEndIndex != -1) var roles = new List<string> {currentActor.Substring(roleStartIndex + 1)};
{
roles.Add(currentRole.TrimEnd(')'));
// Update the outer index (keep in mind it adds 1 after the iteration)
i = j;
break;
}
roles.Add(currentRole); // Fetch all roles
for (var j = i + 1; j < episode.GuestStars.Length; ++j)
{
var currentRole = episode.GuestStars[j];
var roleEndIndex = currentRole.IndexOf(')');
if (roleEndIndex != -1)
{
roles.Add(currentRole.TrimEnd(')'));
// Update the outer index (keep in mind it adds 1 after the iteration)
i = j;
break;
} }
role = string.Join(", ", roles); roles.Add(currentRole);
} }
result.AddPerson(new PersonInfo result.AddPerson(new PersonInfo
{ {
Type = PersonType.GuestStar, Type = PersonType.GuestStar,
Name = name, Name = currentActor.Substring(0, roleStartIndex).Trim(),
Role = role Role = string.Join(", ", roles)
}); });
} }