Changes between Initial Version and Version 1 of Ticket #486, comment 1


Ignore:
Timestamp:
03/21/21 10:29:18 (3 years ago)
Author:
Антон Филатов
Comment:

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #486, comment 1

    initial v1  
    11
    2 {{{
    3 $ make
    4 mkdir -p obj
    5 g++ -O2 -Wall -Werror -std=c++11 -Iinclude -c -MMD -o obj/main.o src/main.cpp
    6 g++ -O2 -Wall -Werror -std=c++11 -Iinclude -c -MMD -o obj/employees.o src/employees.cpp
    7 src/employees.cpp: In member function ‘void EmployeesArray::list()’:
    8 src/employees.cpp:52:23: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
    9      for (int i = 0; i < _employees.size(); ++i) {
    10                      ~~^~~~~~~~~~~~~~~~~~~
    11 cc1plus: all warnings being treated as errors
    12 Makefile:18: recipe for target 'obj/employees.o' failed
    13 make: *** [obj/employees.o] Error 1
    14 }}}
     2~~Да, по структуре у вас сейчас всё правильно: виртуальный print/scan, который вы правильно дёргаете от базового класса в наследниках, а также в операторах << и >>,
     3фабричный метод тоже правильный. (его, правда, можно в EmployeeArray засунуть - это всё же часть класса по-хорошему, а не совсем отдельная функция)~~
    154
    16 Да, по структуре у вас сейчас всё правильно: виртуальный print/scan, который вы правильно дёргаете от базового класса в наследниках, а также в операторах << и >>,
    17 фабричный метод тоже правильный. (его, правда, можно в EmployeeArray засунуть - это всё же часть класса по-хорошему, а не совсем отдельная функция)
     5~~И сделайте, наверное, pragma once - это короче, чем такие вот дефайны~~
    186
    19 И сделайте, наверное, pragma once - это короче, чем такие вот дефайны
     7~~Пока без оценки, но вы идёте в правильную сторону ~~
    208
    21 Пока без оценки, но вы идёте в правильную сторону
     9Это комментарий к старой версии