A24E5Z
- A README.md fájlba érdemes lenne írni néhány dolgot a repository
tartalmáról. Ide kellene majd kerüljön a féléves feladat leírása
is.
- A többi elkészített feladatot is érdemes lenne feltölteni.
A86ANU
- Elég furcsák aktuálisan a feltöltött üres fájlok.
- Fel kellene tölteni az eddig elkészített munkálatok eredményét.
ABCHLD
- Fel kellene tölteni az eddig elkészített munkálatok eredményét.
ACC02G
- Úgy kellene szervezni a repository-t, hogy több C projekt is helyet
kaphasson benne.
- A static kulcsszó használata ilyen
formában nem tűnik indokoltnak.
ACUEBO
- A fájlszerkezet kiindulásnak jó, de fel kellene tölteni az
elkészített programok forráskódját.
ADVLIC
- Célszerű készíteni hozzá Makefile-okat.
- A header fájlokba tetszetős, hogy vannak legalább egy-egy soros
dokumentációs kommentek.
- Jópár kódrészt érdemes lehet kiszervezni függvénybe.
B8GU52
- Az exe fájlokat nem kellett volna feltölteni. A pad.o-t szitén nem. (Törölni kellene a
repository-ból.)
- Érdemes lehet minden gyakorlatot külön mappába tenni (vagy akár
azokon belül a programokat szintén). Mindegyikhez jó, hogy ha lesz Makefile is.
BQLOTW
- A feladat-leiras.txt helyett inkább valamilyen szabványos readme
fájlt lenne érdemes használni.
- A repository gyökerébe is érdemes betenni egyet.
- Praktikus lehet minden projekthez, lefordítandó programhoz külön
Makefile-t készíteni. (A runCwithgcc.txt-re így nincs szükség.)
CNY8MP
- A README.md-be érdemes jobban részletezni a projekt céljával, az
elkészítendő szoftver funkcióival kapcsolatos dolgokat.
- Mivel nagyobb projektről van szó, ezért érdemes lehet egy külön
(például) docs nevű jegyzéket létrehozni,
amibe a dokumentációk kerülnek majd.
- A névterek neve jellemzően inkább kisbetűs szokott lenni.
- A friend kulcsszó kapcsán többször
meg kell gondolni, hogy tényleg szükség van-e rá, vagy nincs-e esetleg
szebb megoldás.
- Azon metódusokat, amelyek nem változtatják meg az objektum
állapotát, érdemes explicit módon jelölni (const a definíciós sor végén).
- A szingleton minta használatát (például Console esetében) érdemes lehet meggondolni,
hogy nincs-e elegánsabb megoldás.
- A kód formázása többségében rendben van. Néhol a tagolást, hosszú
sorok tördelését még érdemes lehet alakítani.
- Praktikus ugyan, de rövid függvények esetében sem tartanám
szerencsésnek az implementációt a hpp
fájlba tenni. (A template-eknél van amikor nincs más megoldás.)
CXG5RG
- Az exe és dll fájlokat semmiképpen sem kellene feltölteni.
- Az SDL library-t magát szintén nem kellene feltölteni.
- Úgy kellene szervezni a fájlokat, hogy több C projekt is helyet
kaphasson benne.
- A program forráskódját több C fájlra kellene szétbontani.
D0H157
- További feladatokat is el kellene még készíteni, és fel kellene
tölteni.
D1H8VP
- A jegyzéknevekben a számok kiírása helyett számjegyeket célszerű
használni.
- A README-kbe jó, hogy ha van valamilyen részletezés, leírás.
D56J6J
DWCPH5
- Az exe fájlokat nem kellene feltölteni.
- A féléves feladathoz praktikusan majd egy összetettebb példából
lehet érdemes kiindulni.
DY98R5
- Az exe fájlokat nem kellene feltölteni.
- A logikai értékekhez lehet használni a true és false
kulcsszavakat.
EK298H
- A repository így egyáltalán nem jó. Sem az OpenCL-es példákat, sem
az összes egyetemi tárgyamhoz kapcsolódó anyagot, sem a MinGW-t, sem
pedig a Notepad++-t nem kellett volna feltölteni.
- Mivel git-ről törölni ténylegesen nem olyan egyszerű, ezért
ugyanilyen néven lenne célszerű létrehozni egy új repository-t.
F2UJS6
- A jegyzékszerkezetből már látszik valami, de lényegi tartalom még
nem került feltöltésre.
F4IIYA
- Egyelőre csak egy feladatkiírás került fel a repository-ba.
- A leírásban részletezni kellene arra vonatkozóan néhány dolgot, hogy
például a pontok hogy kerülnek megjelenítésre, illetve a virágra
kattintást milyen módszerrel tervezi majd megvalósítani.
FG6XJF
- Csak egy közel üres readme fájl van a repository-ban.
FNJ1Q7
- Az SDL library-t magát nem kellett volna feltölteni.
- Az exe fájlok és DLL-ek feltöltésére szintén nincsen szükség.
- A gyakorlatokon készített munkálatoknak viszont fel kellene
kerülniük.
FZ1AKR
- A repository-ba még éppen csak néhány fájl került fel, azok is
üresek.
G2JY7L
- A gyakorlatokon készített feladatok szépen, rendszerezetten
felkerültek.
- Az exe fájlokat nem kellene feltölteni. (Érdemes hozzá .gitignore
fájlt készíteni.)
- A forráskódokban helyenként kisebb, főként csak formai hibák
előfordulnak, de összességében szépen kidolgozott feladatok vannak.
G50NCO
- Az exe fájlokat nem kellene feltölteni. (Érdemes hozzá .gitignore fájlt készíteni.)
- Rendben vannak az elkészített feladatok.
G6HV2W
- A repository gyakorlatilag üres.
GKIU70
- A DLL fájlok feltöltésére úgy általában nincs szükség.. bár ez
esetben külön köszönöm! (Az eddig használt PNG függvénykönyvtár sajnos
teljesen elavult. Bízok benne, hogy ez a fájl átmenetileg megoldja majd
a problémát.)
- A gyakorlatokon készített további feladatok forráskódját is érdemes
lehet feltölteni.
H22TQ1
- A jegyzékek és forráskódok szépen rendszerezettek.
- A futtatható binárisokat nem szükséges feltölteni.
- A header fájlokban jó, hogy ha van dokumentációs komment, de
elegánsabb, hogy ha angolul van, illetve sortörésekkel formázott szükség
esetén. A paraméterezés és a visszatérési érték is részletezhető
benne.
HDK6NX
- A 3_Gyakorlat esetében a teljes,
fordítható programot érdemes feltenni.
- A 4_Gyakorlat esetében
hasonlóképpen.
HK74CE
- `gyak2/matrices`: A dokumentációs kommenteket elegendő lehet csak a
header fájlba beírni.
- Minden projekthez, programhoz jó, hogy ha van Makefile.
HZ60V1
- A repository gyakorlatilag üres.
I03URB
- Van néhány félbehagyott kódrész.
- A futtatható állományokat nem szükséges feltölteni.
- A pong-ban előfordulnak C++-os kódrészek. (Hogy ha C++-t szeretne
használni, akkor az egészet abban kellene írni.)
I89JMB
- Az elkészített programkódok szépen rendszerezettek.
- Az exe fájlokat nem kellene feltölteni. (Érdemes .gitignore fájlt használni hozzá.)
IOMAG5
- Az exe fájlokat nem kellene feltölteni. (Érdemes .gitignore fájlt használni hozzá.)
- A utils-t ilyen formában szintén nem
szükséges feltölteni.
ISXBF3
- A gyakorlatokon elkészített feladatokat is érdemes lenne
feltölteni.
- A feladathoz először majd egy rövid specifikációt kellene készíteni,
és elegendő csak utána hozzákezdeni.
JDMZZK
- A gyak1.c kiindulásnak jó, de a többi
elkészített feladatot is fel kellene tölteni.
JH3V7T
- A projektekhez/programokhoz érdemes Makefile-okat készíteni, hogy
egyszerűbb legyen lefordítani.
- A header fájlokba a függvénydefiníciók elé dokumentációs kommenteket
kellene írni.
- A README.md-be egy rövid leírást
kellene majd írni a repository céljára/tartalmára vonatkozóan. (Ide
kellene majd felkerülnie a féléves feladat rövid
specifikációjának.)
JSBPC1
- A repository gyakorlatilag üres. (Egy generikus README.md van csak benne.)
K1IFAB
- Sok kifejezetten szép és érdekes megoldás van benne.
- A magyar nyelvű dokumentációs sztringek használata nem hiba. Az
angol nyelvű leírás tetszetősebb, közérthetőbb lehet.
- A forráskódban a külön aljegyzék szerinti felosztás szintén
tetszetős.
- A lényegretörő, informatív README.md
szintén dícsérendő.
KA38LU
- A repository gyakorlatilag üres.
MO0VEY
- A repository nem érhető el.
NORS38
- A repository gyakorlatilag üres.
OIKGCW
- A feladatba még korainak tűnne így belekezdeni. (Nem idő
tekintetében, hanem hogy még nincs specifikálva, hogy mit is szeretne
majd elkészíteni.)
- A gyakorlatok feladatai szépen rendszerezettek.
- Érdemes minden programhoz Makefile-t
készíteni majd, hogy egyszerűbb legyen lefordítani.
- Az exe fájlokat nem szükséges feltölteni.
P1UV07
- A feladathoz az után lenne érdemes hozzákezdeni, hogy már
specifikálva lett, hogy mi fog majd elkészülni.
- A line programban (és úgy általában) az extern használatát kerülni kellene.
QTMKZZ
- A repository kiindulásnak jó, de aktuálisan gyakorlatilag üres.
R8H3FT
- A repository gyakorlatilag üres.
RHYYLH
- Az elkészített programokat mindenképpen jegyzékekbe kellene
szervezni.
- A programok fordításához szükséges forrásfájloknak (beleértve a
Makefile-t is) elérhetőeknek kellene
lenniük.
S7KTW0
- A readme.txt-k lehetnek
informatívabbak.
- ZIP fájlokat nem célszerű feltölteni.
- A repository szerkezetileg és tartalmilag alapvetően rendben
van.
SZHYRK
- Az első heti feladat is csak elég törmelékesen került
feltöltésre.
T1TNWL
- Kifejezetten sok és érdekes dolog felkerült a repository-ba.
- A DLL fájlok feltöltésére nincs szükség.
- A kulcsszavak (if, while) után néha lemaradt a szóköz.
- A jegyzékek elnevezése nem egészen egységes.
- Az AdvancedSDL/src/main.c-ben a clean
up-os goto hívást érdemes lehet átgondolni, hogy van-e elegánsabb
megoldás.
- A Cuboid/src/Brick/brick.c-ben az
ok változó esetében lehet bool típust is használni.
- A Going3D/include/Window/window.h
esetében a dupla include guard-ot nem látnám indokoltnak.
- Az include-ok sorrendjére vonatkozóan C-ben a "saját -> 3rd party
-> standard" sorrend általában a konvenció.
- A Line projekt esetében a struktúrákban a nevek kezdőbetűje
változatosan kis- és nagybetűs.
- A header fájlokban érdemes lehet dokumentációs kommenteket
használni.
TLZ12Y
- Az exe fájlokat nem szükséges feltölteni.
- A jegyzékek nevében a szám kiírása helyett számjegyeket praktikusabb
lehet használni.
- A feltett anyagok így még kissé törmelékesek.
U03GDK
- Az exe fájlokat nem kellene feltölteni. (Érdemes .gitignore fájlt használni hozzá.)
- Minden programhoz érdemes Makefile-t
készíteni.
- A README.md-t célszerű MarkDown
formázással formázni.
V204JT
- A repository-ban három, gyakorlatilag üres fájl van csak.
V64E5R
- A Readme.md fájlokban a leírásokat
érdemes részletezni.
- Minden programhoz kellene Makefile-t
készíteni.
- A gyakorlatokon elkészített további programok forráskódját is fel
kellene tölteni.
W0E0RY
- A repository gyakorlatilag üres.
WO02D7
- Kifejezetten sok szép és érdekes dolog van a repository-ban.
- `include/core/Camera.h`: Az enum
helyett érdemes lehet enum class-t
használni. Formalitás, de a default értékek egy külön névtérbe is
kerülhetnek (mert így elég generikus a nevük).
- A dokumentációs kommentekben érdemes a paraméterezést is
részletezni.
- A glm függvénykönyvtárat elegendő
lett volna jelezni, hogy az is függőség. (Feltételezve, hogy abban nem
történt változtatás.)
- Nagyobb mennyiségű resource esetén azt inkább külön fájlként érdemes
hivatkozni (például resources.zip egy
külön URL-en).
- A Mesh osztály konstruktorában a
vektorokat referenciaként célszerű átadni (vagy move szemantika
szerint).
- A for és az if kulcsszavak után elegánsabb, hogy ha mindig
van szóköz.
WSI08I
- A repository kiindulásnak jó, de a többi elkészített feladatot is
fel kellene tölteni.
WY9VBO
- A repository éppen csak néhány, közel üres fájlt tartalmaz.
WZZTLL
- A repository nem érhető el.
XBG3S6
- Előfordul néhány furcsa, üres fájl (include, origin,
donut.c).
- Minden programhoz kellene készíteni Makefile-t.
- A gyakorlatokhoz tartozó feladatok jegyzékeinek nevében jobb, hogy
ha számjegyek szerepelnek a számok kiírása helyett.
XPGMCH
- Szép, előrehaladott állapotban lévő munkálatokról van szó.
- A README.md-ben a feladatkiírásban
úgy szerepel, hogy "MS Paint with openGL". Az aktuális változat SDL2
renderert használ, és SDL függvényeket úgy általában. Az aktuális
megközelítés sem rossz, de OpenGL-el talán egyszerűbb és elegánsabb
lehet.
XPUYJX
- Egyetlen line.c fájl került
feltöltésre a repository-ba.
YE6BLB
- A repository éppen csak néhány, közel üres fájlt tartalmaz.