2023-05-29 Answers Code Review Tasks 23

Jautājumi

1. Nav saprotams, kādas vērtības jāliek šajā funkcijā

Funkcija paradzēta, lai azipildītu SelectionFileDAD.IntegrationDataSet, izmantojot ārēju Pascal kodu. Nedrīkst hard-coded bez nosaukumiem parametrus funkciju izsaukumiem.

SelectionFileDAD.Integrate funkcijas parametri

funkcijā tiek izveidots jauns peak table

Un uzstādīts baseline, kuru arī jārāda chromatogramā

2. Nekur kodā nedrīkst iekodēt Strings, kuri būs redzami lietotājam, nelietojot LocalizationUtils.Localize - programmai ir jāstrādā uzreiz Japāņu valodā, nevar būt netulkoti teksti

Code Review

1. Novērst šādu bīstamu kodu - un neatkārtot citur

2. Novērst, ka UIv2ChromatogramView strādā ar UIv2SelectionItem. Tas, ka treeview / sidebar strādā ar šo ir saprotams, bet nav saprotams kāpēc man nepietiek ar SelectionFileDAD kā datu avotu? Pie tam pašam Project view arī varētu iztikt ar UIv2SelectionItem, pievienojot IsSelected property. Šis ievērojami apgrūtinās darbu Kristianam K. peivienojot šo kontroli Automation programmām

Nosaukums OriginalSelectionFileDAD arī ir ļoti maldinošs, jo nemaz nav nekāds cits “non-original” SelectionFileDAD.

Tāpat nekādā gadījumā nedrīkst būt bezjēdzīgas kopijas datiem kā piemēram Peaks un Name, drīkst būt GETTER properties, bet nedrīkst būt datu kopijas!

4. Salabot, haotisku koda pierakstu - nepieciešams vienots funkciju nosaukumu stils nevis katru funkciju savā stilā un ar savu pierakstu, dažreiz this, dažreiz bez utt

   

5. Iznest kopīgo kodu funkcijās, neatkartot 3x

6. Visām funkcijām jābūt try..catch, lai pie vienas kļūdas visa lietotne nesabruktu, gandrīz nevienai tavai funkcijai nav try..catch un LoggingUtils.logException, kas ir svarīgi, lai vēlāk atrastu kļūdas [rpdilcok

7. a]

   

7. Funkcijās nedrīkst būt vairāk kā viens return, itsevišķi tādās, kur vairāki nosacījumi un jālieto ELSE IF. Katekoriski aizliegts ārpus try..catch RETURN rindiņā likt sarežģītus izsaukumus, kuriem ļoti augsta varbūtība crashot. Vispār return rindā nedrīkst likt funkciju izsaukumus

8. UIv2ChromatogramView pārsaukt UIv2MultiChromatogramsAndPeakTableView, ļoti apjucinošs nosaukums, nosaukumiem jābūt pēc iespējas aprakstošiem

    

TODO

1. Scrollable log, kur tiek fiksēts viss info kā tika iegūti dati (to pievienos Kristiana Kalniņa kods, kamēr chromatogram tiek ievākta)

Var pievienot no vecā, koda, taču jāpārbauda, ka strāda korekti kā arī kamēr kristiana kalniņa kods darbosies, šie dati mainīsies, tapēc būs nepieciešams, lai Log scrollotos uz leju, ja lietotājs pats nesāk scrollēt - vecajā kodā to dara m_timerLogScrollDown_Tick

if(Mode == UserControlViewDataMode.Realtime) vecajā kodā nozīmēja, ka chromatogram šobrīd tiek iegūta

xxxxxxxxxx
42
1
<ScrollViewer Name="uiScrollTextBoxLog">
2
    <StackPanel Orientation="Vertical">
3
        <!--  Online log  -->
4
        <TextBox Name="uiTextBoxLogRuntime"
5
                 Margin="10,10,10,0"
6
                 Background="White"
7
                 IsReadOnly="True"
8
                 Padding="0"
9
                 Style="{StaticResource ResourceKey=TextBoxClearReadOnly}"
10
                 Text="{Binding InstanceMainSelectionFileDAD.RefExperiment.MethodData.MetaRptFilePath,
11
                                UpdateSourceTrigger=PropertyChanged}"
12
                 TextWrapping="Wrap" />
13
        <TextBox Name="uiTextBoxLogRuntime2"
14
                 Margin="10,0,10,10"
15
                 Background="White"
16
                 IsReadOnly="True"
17
                 Padding="0"
18
                 Style="{StaticResource ResourceKey=TextBoxClearReadOnly}"
19
                 Text="{Binding InstanceMainSelectionFileDAD.RefExperiment.MethodData.MetaRptHeader,
20
                                UpdateSourceTrigger=PropertyChanged}"
21
                 TextWrapping="Wrap" />
22
        <!--  Offline log  -->
23
        <TextBox Name="uiTextBoxLog"
24
                 Margin="10,10,10,0"
25
                 Background="White"
26
                 IsReadOnly="True"
27
                 Padding="0"
28
                 Style="{StaticResource ResourceKey=TextBoxClearReadOnly}"
29
                 Text="{Binding InstanceMainSelectionFileDAD.RefFileDAD.MethodData.MetaRunHeaderSummary,
30
                                UpdateSourceTrigger=PropertyChanged}"
31
                 TextWrapping="Wrap" />
32
        <TextBox Name="uiTextBoxLog2"
33
                 Margin="10,0,10,10"
34
                 Background="White"
35
                 IsReadOnly="True"
36
                 Padding="0"
37
                 Style="{StaticResource ResourceKey=TextBoxClearReadOnly}"
38
                 Text="{Binding InstanceMainSelectionFileDAD.RefFileDAD.MethodData.MetaRunHeaderFull,
39
                                UpdateSourceTrigger=PropertyChanged}"
40
                 TextWrapping="Wrap" />
41
    </StackPanel>
42
</ScrollViewer>

2. “Info” tab, satur informāciju no selectionFileDAD.GetFormatedValuesList(isFullHeader: false) vajadzētu pārtaisīt, lai atkal nav lieks mainīgais m_itemsSourceDataGridInfo

   Jāievēro, ka nepieciešams dažiem no laukiem būtu funkcijai, lai var nomainīt saturu/tekstu IsEditable = true un visiem mainīt secību ar drag & drop

   Ar raita un Kristiana Kalniņa palīdzību jāuztaisa editing kontrole līdzīgi kā vecajā versijā bija (kristianam K. jau ir šāda kontrole mainot solvent names ChromSword DataSystem)