Jump to content
php.lv forumi

bank.lv valūtas kursi


l27

Recommended Posts

PHP klase bank.lv valūtas kursu nolasīšānai un saglabāšanai DB: https://github.com/uldisn/bank.lv-currency-rate

 

Pieprasot valūtas kursu, tas tiek nolasīts no datubāzes. Ja nav iekš datubāzes,
tad nolasa no Latvijas Bankas mājas lapas un pie reizes saglabā datubāzē.
Tiek minimzēts pierasījumu skaits LB lapai, kā arī palielinās skripta izpildes ātrums.

Edited by l27
Link to comment
Share on other sites

Pārdomas par koda stilu.

 

Faili config.inc.php un constants.php:

* Kāpēc vienam ir "inc", bet otram nav "inc" nosaukumā?

* Kāpēc viens fails beidzas ar "?>", otrs fails - nē?

 

Jocīga (reti redzēta) nosaukuma konstrukcija "fCurrency"? Kas tas par "f" burtu? Tas pats arī parametru nosaukumos. Kāpēc $sError, nevis $bError? Tas domāts kā Hungarian notation?

Klasē daļa metožu ar private, protected, public, daļa bez. Klases atribūts definēts ar var.

SQL izteiksmēs daļā izmantoti lielie burti atslēgvārdiem, daļā mazie.

Ir if-else izteiksmes bez {}.

Konstantēm izmantoti mazie burti.

 

Vienā metodē ir šādi:

 

$aRow = MySQL::q($sSql);

 
        if (count($aRow) == 0) {

 

Bet citā metodē jau ir šādi:

 

$aC = MySQL::q($sSql);

        foreach ($aC as $aRow) {
 

Faila constants.php konstantes arī varētu pārlikt kā klases konstantes, ja šim mērķim vispār izmantot konstantes.

 
Es jau neko, tikai gribu saprast domāšanas veidu... Piemēram, ir pieņemts, ka automašīnai visi četri riteņi ir ar vienādiem diskiem un riepu modeļiem, un arī skrūves vienādas. Iekārtojot mājokli, pieņemts, ka flīzes liek vienādas ar noteikta platuma šuvi. Utt.
Link to comment
Share on other sites

 

Pārdomas par koda stilu.

 

.......

 

Paldies par recenziju. Daudzas īpatnības ir saistītas ar esošās sistēmas standartiem. Otrais faktors ir laiks.

Tākā sistēmā ieviesu autoloadingu, tad arī valūtas konstantes būt lietderīgi pārnest zem klases, kaut gan valūtas komplekti var atširties dažādām sistēmām.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...
×
×
  • Create New...