University-of-Basel-Studentprojects/Programmierprojekt
Archived
1
0
Fork 0
Code Issues 17 Pull Requests Actions Wiki Activity

Add additional getters to SessionRegistry with UserId and username as key #230

Merged
lars.winzer merged 4 commits from feat/47-sessionregistry-get-by-userid-and-username into main 2026-04-02 14:23:43 +02:00
Conversation 1 Commits 4 Files Changed 1
+6 -5
1 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 415ad754df - Show all commits
src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/domain/user/UserRegistry.java
+6 -5
View File
@@ -1,8 +1,6 @@
package ch.unibas.dmi.dbis.cs108.casono.server.domain.user; package ch.unibas.dmi.dbis.cs108.casono.server.domain.user;
import ch.unibas.dmi.dbis.cs108.casono.server.network.sessions.SessionId; import ch.unibas.dmi.dbis.cs108.casono.server.network.sessions.SessionId;
import java.lang.classfile.attribute.UnknownAttribute;
import java.util.Collection; import java.util.Collection;
import java.util.Optional; import java.util.Optional;
import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.ConcurrentHashMap;
@@ -91,7 +89,8 @@ public class UserRegistry {
* Looks up a user by their {@link SessionId}. * Looks up a user by their {@link SessionId}.
* *
* @param sessionId the SessionId to look up * @param sessionId the SessionId to look up
* @return an Optional containing the {@link User}, or empty if no user is associated with this SessionId * @return an Optional containing the {@link User}, or empty if no user is associated with this
* SessionId
*/ */
public Optional<User> getBySessionId(SessionId sessionId) { public Optional<User> getBySessionId(SessionId sessionId) {
return Optional.ofNullable(bySessionId.get(sessionId)); return Optional.ofNullable(bySessionId.get(sessionId));
@@ -101,7 +100,8 @@ public class UserRegistry {
* Looks up a user by their {@link UserId}. * Looks up a user by their {@link UserId}.
* *
* @param userId the UserId to look up * @param userId the UserId to look up
* @return an Optional containing the {@link User}, or empty if no user is associated with this UserId * @return an Optional containing the {@link User}, or empty if no user is associated with this
* UserId
*/ */
public Optional<User> getByUserId(UserId userId) { public Optional<User> getByUserId(UserId userId) {
return Optional.ofNullable(byId.get(userId)); return Optional.ofNullable(byId.get(userId));
@@ -111,7 +111,8 @@ public class UserRegistry {
* Looks up a user by their username. * Looks up a user by their username.
* *
* @param username the username to look up * @param username the username to look up
* @return an Optional containing the {@link User}, or empty if no user is associated with this username * @return an Optional containing the {@link User}, or empty if no user is associated with this
* username
*/ */
public Optional<User> getByUsername(String username) { public Optional<User> getByUsername(String username) {
return Optional.ofNullable(byName.get(username)); return Optional.ofNullable(byName.get(username));