Code Monkey home page Code Monkey logo

oop's Introduction

oop

Лабораторные работы по ООП

oop's People

Contributors

panic-torra avatar

Watchers

James Cloos avatar Nick Yegorov avatar  avatar

oop's Issues

Замечания по Лабораторной работе №1 "invert"

  • #define замените на const int, const double и т.д.
  • используя только cpp файлы - бессмысленно разделять объявление функций и их реализацию
  • простые типы не нужно передавать по константным ссылкам, памяти они занимают немного, а передача по ссылке замедлит работу программы, передавайте в зависимости от назначения или по простой ссылке или по значению
  • не очень хорошо получается, когда в функции TryToGetInvertMatrix происходит PrintMatrix, назавание функции TryToGetInvertMatrix не соответствует выполняемым внутри действиям, советую изменить архитектуру. не нужно использовать TryToGetInvertMatrix как замену main
  • не смысла объявлять матрицу Matrix dstMatrix; до того как прошли проверки. объявляйте - после них.
  • TryToGetInvertMatrix если возвращаемых значения 2 используйте тип bool
  • попробуйте использовать вместо CStyle массивов std::vector или std::array
  • вместо типа short внутри циклов нужно использовать тип std::size_t(если индексы используются для доступа к элементам какого-либо контейнера)

Замечания по Car

  • Навести порядок в именовании тестов
      if car engine is turned off
2>        can not select gear when engine turned off
2>        engine can be turned on
2
  • DontRepeatYourself
work with reverse gear
  • имя теста не является описанием требований, а является всего лишь названием тестов

Замечания по программе Replace

  • Не используйте exit в программе. Об ошибках рапортуйте через коды возврата (или исключения, если с ними знакомы)

Замечания по FindMaxEx

  • Неэффективно реализован выбор максимального значения. Всякий раз происходит копирование следующего кандидата на победу

Замечания по HttpURL

http://localhost:65536
URL: http://localhost:0/5536
Protocol: HTTP
Domain: localhost
Port: 0
Document path: /5536
  • Криво обрабатывается порт. Нет тестов граничных условий
http://localhost:0
URL: http://localhost:0/
Protocol: HTTP
Domain: localhost
Port: 0
Document path: /
  • Нулевой номер порта должен быть недопустимым

Замечания по HtmlDecode

  • сделать замену за один проход
  • исходники не должны лежать в файлах ресурсов
  • тесты должны содержать человекочитаемые строки
  • вероятно, в финальной версии в функции Replace не будет необходимости
  • можно сразу зарезервировать место в строке-результате, достаточное для хранения исходной строки (результат будет не длинеее)
  • структура тестов.

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.