review #2

Serferrr · 2024-01-26T17:43:57Z

review, don't merge

…e screen

There are a problem with undefined values of positions so I created TextParameters class to order this. Have to defined more values.

This class will be to create a grid in each Window to set every element on a screen clearly.

…etPosition

There was problems with distances in list of station when >4. Now it is fixed.

…kets displaying

In previous version on the top of the list of a queue was the last ticket. Now there is first ticket

No longer needed. I change my way of making an order with "magical numbers" - now there is WindowGrid.

smichajlidis added 30 commits January 3, 2024 20:16

add .gitignore

31cd43c

add main.cpp with sfml hello world

a01e273

add a queue, window and pick up 3 tickets

9981bc4

create QueuesManager with addQueue and pickATicket

b621e69

create WindowsManager and declare addWindow method

d29a587

define addQueue and pickATicket methods

f8f9116

define addWindow method

bbeaf78

redefine addRelatedQueue

b072cea

redefine the constructor

7992703

define constructors and pickaATicket method

862d955

change argument of addRelatedQueue method

5512bd1

declare display() and the constructor

83c8197

create Queue class for each methods and atributes

d906b5a

delete WindowsManager - Windows vector can be in QueuesManager

ebc0227

define some samples of creating queue and window

021c1b1

hide every creation into QueuesManager class

36dd63f

delete WindowsManager - no longer needed

e52d938

create vectors of Windows and Queues and all methods to connect them

e9b9c15

add processEvents(), update() and render() declarations

3eb2e7b

start definitions of processEvents, update and render methods

e5db0b3

add some methods to show and update sfml windows

3f66806

define run(), isOpen() universal methods

759a92b

add Lato font

6c2dca1

declare runAllWindows() method

8fc4ac6

define runAllWindows()

c7ff4c4

delete display() and update() methods declaration

271d011

delete display() and update() declaration and add title atribut

ef02bbc

change title to Station number 1

33a13a6

define processEvents, render and constructor to get two buttons on th…

14773fe

…e screen

delete update() from run() method

5a844b9

Repository owner deleted a comment from Serferrr Feb 4, 2024

smichajlidis added 10 commits February 4, 2024 13:16

Create TextParameters class and use few methods

5e848c6

There are a problem with undefined values of positions so I created TextParameters class to order this. Have to defined more values.

Create WindowGrid class

080409e

This class will be to create a grid in each Window to set every element on a screen clearly.

Create WindowGrid as an atribute of each Window

d81c9cd

Create a grid in ctor and put numbers of columns and rows into each s…

a54c5f4

…etPosition

Set proper distances in list of stations

4b7bbb2

There was problems with distances in list of station when >4. Now it is fixed.

Set (...) instead of following tickets when tickets are more than 10

1ece9ff

Add window_grid to Station class and set (...) instead of to many tic…

eb8e728

…kets displaying

Change last variable to counter and count from getFirstInQueue()

959a9a5

In previous version on the top of the list of a queue was the last ticket. Now there is first ticket

Delete TextParameters class

5e395b6

No longer needed. I change my way of making an order with "magical numbers" - now there is WindowGrid.

Set window_grid for Kiosk

f6605d6

Repository owner deleted a comment from Serferrr Feb 8, 2024

Change camelCase variables names to snake_case

2354241

Repository owner deleted a comment from Serferrr Feb 8, 2024

smichajlidis added 2 commits February 8, 2024 11:21

Change raw ints to std::uint32_t

fa86b9c

Delete unnecessary includes or replace it by class declarations

b609779

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

review #2

review #2

Serferrr commented Jan 26, 2024

review #2

Are you sure you want to change the base?

review #2

Conversation

Serferrr commented Jan 26, 2024