Simplify: use an enum instead of magic constants.
This commit is contained in:
@@ -574,18 +574,18 @@ void MusicWheel::BuildWheelItemDatas( vector<WheelItemData> &arrayWheelItemDatas
|
||||
// default sort order; other songs may alter this
|
||||
SortCoursePointerArrayByDifficulty( apCourses );
|
||||
|
||||
if (PREFSMAN->m_iCourseSortOrder != 0)
|
||||
if (PREFSMAN->m_iCourseSortOrder != PrefsManager::COURSE_SORT_SONGS)
|
||||
{
|
||||
for(i=0; i<apCourses.size(); i++)
|
||||
apCourses[i]->UpdateCourseStats();
|
||||
|
||||
if (PREFSMAN->m_iCourseSortOrder == 1)
|
||||
if (PREFSMAN->m_iCourseSortOrder == PrefsManager::COURSE_SORT_METER)
|
||||
SortCoursePointerArrayByAvgDifficulty( apCourses );
|
||||
|
||||
if (PREFSMAN->m_iCourseSortOrder == 2)
|
||||
if (PREFSMAN->m_iCourseSortOrder == PrefsManager::COURSE_SORT_METER_SUM)
|
||||
SortCoursePointerArrayByTotalDifficulty( apCourses );
|
||||
|
||||
if (PREFSMAN->m_iCourseSortOrder == 3)
|
||||
if (PREFSMAN->m_iCourseSortOrder == PrefsManager::COURSE_SORT_RANK)
|
||||
SortCoursePointerArrayByRanking( apCourses );
|
||||
}
|
||||
|
||||
|
||||
@@ -110,7 +110,7 @@ PrefsManager::PrefsManager()
|
||||
m_bPickExtraStage = false;
|
||||
|
||||
// default to old sort order
|
||||
m_iCourseSortOrder = 0;
|
||||
m_iCourseSortOrder = COURSE_SORT_SONGS;
|
||||
|
||||
m_fLongVerSongSeconds = 60*2.5f; // Dynamite Rave is 2:55
|
||||
m_fMarathonVerSongSeconds = 60*5.f;
|
||||
|
||||
@@ -85,8 +85,7 @@ public:
|
||||
bool m_bTenFooterInRed;
|
||||
|
||||
// course ranking
|
||||
// by # songs = 0, avg diff=1, total diff=2, ranking=3
|
||||
int m_iCourseSortOrder;
|
||||
enum { COURSE_SORT_SONGS, COURSE_SORT_METER, COURSE_SORT_METER_SUM, COURSE_SORT_RANK } m_iCourseSortOrder;
|
||||
|
||||
/* 0 = no; 1 = yes; -1 = auto (do whatever is appropriate for the arch). */
|
||||
int m_iBoostAppPriority;
|
||||
|
||||
@@ -195,7 +195,7 @@ void ScreenAppearanceOptions::ExportOptions()
|
||||
PREFSMAN->m_bShowLyrics = !!m_iSelectedOption[0][AO_SHOW_LYRICS];
|
||||
PREFSMAN->m_bDancePointsForOni = !!m_iSelectedOption[0][AO_DANCE_POINTS_FOR_ONI];
|
||||
PREFSMAN->m_bTenFooterInRed = !!m_iSelectedOption[0][AO_TEN_FOOT_RED];
|
||||
PREFSMAN->m_iCourseSortOrder = m_iSelectedOption[0][AO_COURSE_SORT];
|
||||
(int&)PREFSMAN->m_iCourseSortOrder = m_iSelectedOption[0][AO_COURSE_SORT];
|
||||
|
||||
PREFSMAN->SaveGamePrefsToDisk();
|
||||
PREFSMAN->SaveGlobalPrefsToDisk();
|
||||
|
||||
Reference in New Issue
Block a user