Vic br

[Roman-Koshelev]

Ревью

Audio::Audio(QString path) { this->path = path; ssize_t startIndexNameSong = path.lastIndexOf(QString::fromLocal8Bit("/"));

использовать auto

QVector<Audio> Audio::fromAudioPaths(const QStringList &paths){ QVector<Audio> result; for(auto iter = paths.begin() ; iter != paths.end() ; ++iter){

использовать for(const auto path: paths)

const QString& Audio::GetPath() const{
Все эти методы должны быть inline

const QString& GetAlbumTitle() const; bool operator == (const Audio& other) const; private: QString path;

Все переменке выглядят как const, но тогда LoadMetaData должен быть mutable (допустимо, если он вызывается только из конструктора)

QStringList tracksNames;

// добавим только новые треки в currentList
foreach(Audio song, tracks){`

с++11 уже умее т for(:)

public: MainController(){}; ~MainController(){};

мне кажется, что они виртуальный. Но в любом случае лучше оставить автогерируемый

MainWindow::~MainWindow() { delete audioListModel;

ЕМНП, то объекты будут удалены при удалении родителя.
Если хотите явного удаления, то используйте std:uniq_ptr

void MainWindow::setVolumeSlider() { if(!volumeSliderStatus){ volumeSlider = new QSlider(Qt::Horizontal, ui->volumeBox);

добавление / удаление объекта выглядит странным. его лучше скрывать

explicit MainWindow(QWidget *parent = 0);

nullptr (опять же C++11

public: explicit MainWindow(QWidget *parent = 0); ~MainWindow(); void setAudioListModel(QStringList tracks);

может по const ссылке?

void setPlayingPosition(int position); void setSelectedAudioPosition(int position); void addTracks(const QVector<Audio>& newTracks);

Надо выбрать один стиль для slots:

по значению
по указателю
Но на работу не влияет

class WorkingDirectory { public: WorkingDirectory(const QString& _path);

explicit

QString GetSong(const unsigned int i);

азвание аргумента не понятно, const избыточен (возможно)