projojoboy / becomefatshader Goto Github PK

Je hebt feedback gekregen van BerendWeij
URL: https://github.com/projojoboy/BecomeFatShader/blob/master/README.md

Feedback: Er ontbreekt nog een portfolio pagina voor dit project.

Ik wil graag meerdere afbeeldingen zien of bewegend beeld. Om zo goed te laten zien wat je hebt gemaakt.

Verder moet in de beschrijving duidelijk zijn:

voor welk platform je het hebt gemaakt
in welke taal je het hebt gemaakt
wat er speciaal is aan dit project (keywords)

Ook moet je door kunnen klikken naar de demo + GitHub repos.

Verder had je nog net iets meer eigens in de shader kunnen stoppen. Zo had je de waardes kunnen lerpen bijvoorbeeld. Of met een texture kunnen aangeven welke lichaamsdelen harder moeten schalen.

Je hebt feedback gekregen van BerendWeij
URL: https://github.com/projojoboy/BecomeFatShader/blob/master/Become Fat Shader/Assets

Feedback: Plaats je scripts in een Scripts folder. Dit is overzichterlijker.

Je hebt feedback gekregen van BerendWeij
op:

if(coll.tag == "food")
        {
            fatness += 10;
            body.GetComponent<Renderer>().material.SetFloat("_Amount", fatness);
            Destroy(coll.gameObject);
        }

        if (coll.tag == "SpecialFood")
        {
            fatness += 100;
            transform.localScale = new Vector3(3, 3, 3);
            body.GetComponent<Renderer>().material.SetFloat("_Amount", fatness);
            Destroy(coll.gameObject);
        }

        if (coll.tag == "void")
        {
            transform.position = new Vector3(0, 10, 0);
        } 

URL: https://github.com/projojoboy/BecomeFatShader/blob/master/Become Fat Shader/Assets/IncreaseFat.cs

Feedback: Ha Joey

Je if/if/if had een if, else if, else if kunnen zijn. Nu wordt er onnodig 3x gecheckt of tag een bepaalde string is.

Verder kan het ook wat DRYer. Ik zie veel dubbele code.

En hoe zou je interfaces kunnen gebruiken om dit nog generieker te maken? Het is wat mij betreft nog wel interessant dat je dat probeert door te voeren.