use std::optional<> to signify when there is no value
This commit is contained in:
parent
6a7678a6e9
commit
cba7e2dc36
1 changed files with 7 additions and 2 deletions
|
@ -125,7 +125,7 @@ struct Server {
|
||||||
bool m_checked = false;
|
bool m_checked = false;
|
||||||
bool m_up = false;
|
bool m_up = false;
|
||||||
QString m_motd; // https://mctools.org/motd-creator
|
QString m_motd; // https://mctools.org/motd-creator
|
||||||
int m_currentPlayers = 0;
|
std::optional<int> m_currentPlayers; // nullopt if not calculated/calculating
|
||||||
int m_maxPlayers = 0;
|
int m_maxPlayers = 0;
|
||||||
};
|
};
|
||||||
|
|
||||||
|
@ -146,6 +146,7 @@ class ServerPingTask : public Task {
|
||||||
qDebug() << "Resolved Addresse for" << domain << ": " << ip << ":" << port;
|
qDebug() << "Resolved Addresse for" << domain << ": " << ip << ":" << port;
|
||||||
McClient client(nullptr, domain, ip, port);
|
McClient client(nullptr, domain, ip, port);
|
||||||
int online = client.getOnlinePlayers();
|
int online = client.getOnlinePlayers();
|
||||||
|
qDebug() << "Online players: " << online;
|
||||||
m_server.m_currentPlayers = online;
|
m_server.m_currentPlayers = online;
|
||||||
|
|
||||||
client.close();
|
client.close();
|
||||||
|
@ -385,7 +386,11 @@ class ServersModel : public QAbstractListModel {
|
||||||
case 2:
|
case 2:
|
||||||
switch (role) {
|
switch (role) {
|
||||||
case Qt::DisplayRole:
|
case Qt::DisplayRole:
|
||||||
return m_servers[row].m_currentPlayers;
|
if (m_servers[row].m_currentPlayers) {
|
||||||
|
return *m_servers[row].m_currentPlayers;
|
||||||
|
} else {
|
||||||
|
return "...";
|
||||||
|
}
|
||||||
default:
|
default:
|
||||||
return QVariant();
|
return QVariant();
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue