[C++] Loading bar

Question

Witam,

Chciałbym zaciągnąć od Was rady. Napisałem programik z loading barem i dodatkami typu kolorki. Wiem, że tutaj może być kwestia sporna, ponieważ to będzie raczej dotyczyło własnego upodobania, chociaż i tak chciałbym poznać co o tym sądzicie. Czy uważacie, iż taki loading bar jest okej? Czy może coś dodać np animację napisu Loading. Chciałbym, aby taki loading podobał się nie tylko mnie, ale innym (może wprowadzę kiedyś do gry czy coś). Więc jestem otwarty na wszelkie krytyki . Oczywiście, mamy do czynienia tylko z konsolą okienkową. A może macie kompletnie inną wizję? Podzielcie się ze mną .

Tutaj kod:

#include <iostream>
#include <windows.h>


void loading(void)
{
		
	ShowWindow(GetConsoleWindow(), SW_MAXIMIZE);
	system("color 1F");
	
	HANDLE screen = GetStdHandle(STD_OUTPUT_HANDLE);
	
	COORD pos;
	pos.X = (168/2)-3;
	pos.Y = (58/2)-5;
	SetConsoleCursorPosition(screen, pos);
	std::cout << "Loading\n";
	
	pos.X = 0;
	pos.Y = (58/2);
	SetConsoleCursorPosition(screen, pos);
	for(int i{}; i<168; i++)
	{
		if(i == 50 || i == 90 || i== 100 || i == 120) Sleep(1000/3);
		std::cout << "\xb1";
		for(int j=0; j<16000000; j++) 
		{}
	}
	std::cout << "\n"; 
}

int main()
{

	loading();	
}

P.S. Standard C++11.

Answer 1

Moja opinia?
system(...) - tej funkcji nie używaj. Problem z bezpieczeństwem i wieloma innymi rzeczami jak multiplatformowość. Użyj SetConsoleTextAttribute instead.
Ustawiasz pozycję kursora na sztywno (linie 13-16). WinApi ma funkcje do pobierania wielkości bufora konsoli - tego prawdziwego jak i widzialnego.
Sleep(...) - tego też nie używaj.. includuj thread i chrono i zrób sobie std::this_thread::sleep_for(std::chrono::JEDNOSTKA_CZASU(ILOSC))

 

----WRAŻENIA WIZUALNE----

:V
Trochę za długie to jest. Pod cmder'em konsola wariuje, napis loading jest na środku, a na dole leci jakiś pasek z kwadracików (kodowanie złe) do 1 1/8 (na oko) paska. Wygląda to tak
Ja bym zrobił mały pasek, który składałby się z max 10-12 kratek.

---Á PROPOS KODU----

Zamiast bawić się z jakimiś pętlami itd pobawiłbym się z operatorem dzielenia i modulo. Chodzi o to, że jak masz np. 50kB danych do wysłania (załóżmy, że masz lekkiego klienta FTP na konsoli), to dzielisz sobie to na 12, wychodzi Ci 4.xxxxx, co oznacza, że co 4kB wysłane musisz zapalić jedną kratkę z loading bara i tutaj właśnie jest zabawa z modulo i dzieleniem. Dodatkowo trzeba to jeszcze gdzieś zapisywać, bo jakby np. przerwało nagle połączenie, to przecież nie może być tak, że połowa danych wysłana, a loading bar leci od początku (ale to już zależne od przypadku)

PS

Zapomniałem. Jeszcze trzeba się zmagać z tym, że przecież data może mieć NP te 50kB, a ty chcesz mieć loading bar szerokości 12kratek. Wtedy trzeba robić jakieś align'y czy inne cuda.

Comments

Protokołów w konsoli? Jakich protokołów w konsoli? :0

 

No widzisz, u Ciebie wygląda to tak. I to jest to o czym pisałem - ustawianie na sztywno wielkości bufora. Zobacz to

---

Wybacz xD FTP skojarzyłem z File Transfer Protocol haha

---

No tak, FTP to File Transfer Protocol, ale protokoły nie są zależne od sposobu wyświetlania

---

@Hiskiel,

Sleep(...) - tego też nie używaj.. includuj thread i chrono i zrób sobie std::this_thread::sleep_for(std::chrono::JEDNOSTKA_CZASU(ILOSC))

Co za różnica? I tak używa WinAPI.

---

Piszę to tak, bo tu mi chodzi o osobne złe praktyki. Już lepiej, żeby używał tam gdzie może dobrych praktyk, a tam gdzie musi WinApi.