ensios/tsoha-bootstrap

Tietokantasovellus-kurssin aloituspaketti

Tietokantasovelluksen esittelysivu

Yleisiä linkkejä:

• Linkki sovellukseeni(etusivulle)
• Linkki tuoteluokat sivulle
• Linkki tuoteluokan sivulle
• Linkki tuotteen sivulle
• Linkki dokumentaatiooni

Työn aihe

Huutokauppa (linkki valmiiseen aiheeseen)

(Mukaelma linkitetyllä sivulla esitetystä valmiista aiheesta)

Kirjautumiseen

Asiakas

Käyttäjätunnus: NAINEN
Salasana: Salasana3

Meklari (vain tälle näkyy tietokannan muutokseen tähtäävät toiminnot)

Käyttäjätunnus: MIES
Salasana: Salasana2

tsoha-bootstrap's Issues

Koodikatselmointi

Koodi ladattu pe 11. elokuuta klo 21:52

En saanut valitettavasti sovellustasi auki readme-tiedostossa löytyneestä linkistä.

Dokumentaatiosi on selkeä ja siitä saa hyvän kuvan toteutettavasta järjestelmästä.

Koodisi näytti päällisin puolin selkeältä. En itse ohjelmoi php:llä enkä tee omaa työtäni sillä. Koodisi oli hyvin jäsenneltyä ja muuttujien&metodien nimeämiset selkeitä.

Koodikatselmointi

Ladattu: 23/08/2017 22:11

Hyvännäköinen projekti, ulkoasuun on selvästi kiinnitetty huomiota. Jo luodut toiminnot tuntuvat pelaavan hyvin, tosin aikaleiman validointi kohteen muokkauksen yhteydessä kaataa sivun jos aikaleima ei ole tismalleen oikeassa muodossa. Mutta ilmeisesti siihenkin on tulossa jokin toinen syötetapa, kuin käyttäjän kirjoittama merkkijono.

Tietokannat on toteutettu järkevästi, ja varsinkin oma taulu henkilötiedoille on kiva yksityiskohta.

Itse koodiin: koodi on pääosin selkeää ja puhdasta ja selkeästi laajennettu tsohan mallien pohjalta. Jokaisella luokalla on oma, selkeä tehtävänsä, controllereita ja malleja on tullut lisää tarpeen vaatiessa. Asiat on nimetty järkevästi, ja niistä näkee suoraan, mitä ne tarkoittavat. Funktiot eivät ole liian pitkiä, eivätkä tee ylimääräisiä asioita. Sisennyksessäkään ei moitittavaa, mitä nyt välillä twigin kanssa:

(forms.html)

{% macro error_messages(errors) %}

{% if errors %}
<div class="alert alert-danger ">
    {% for error in errors %}
    <ul>
        <li>{{error}}</li>
    </ul>
    {% endfor %}
</div>
{% endif %}

{% endmacro %}

{% macro message(message, class) %}

{% if message|length > 1 %}
<div class="alert {{class}}">
    {{message}}
</div>
{% endif %}

{% endmacro %}

Näyttää kohtuu hirveältä, mutta minkäs teet. Ainakaan Sublime Textin autosisennys ei tuolle osaa mitään tehdä.

Vaikea löytää mitään oikeaa huomautettavaa. Projekti on hyvillä raiteilla, kunhan vain saat lisättyä kaikki puuttuvat toiminnot ajoissa. Erityistä plussaa ulkoasusta: simppeliä, puhdasta ja luettavaa. Jatka samaan malliin!