Jump to content
php.lv forumi

Recommended Posts

Posted (edited)

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
Posted

Agrāk kodos rakstīja māzak rindiņu un dabūja to pašu rezultātu.

Posted

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.
Posted

 

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.

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...