Любые идеи, почему эта сортировка в алфавитном порядке не работает?

Я делаю программу, которая многое делает с списком автомобилей, проверяя все, и последнее, что мне нужно сделать, это сортировать эти машины по алфавиту по строкам. Вот код, который не работает, как это должно быть.Любые идеи, почему эта сортировка в алфавитном порядке не работает?

for (int j = 0; j < towns[townNo].townVehicleCount-1; j++) { 
    for (int k = 1; k < towns[townNo].townVehicleCount; k++) { 
     int identical; 
     identical = towns[townNo].Vehicles[j].Brand.CompareTo(towns[townNo].Vehicles[k].Brand); 
     if (identical == 1) { 
      towns[townNo].Vehicles[towns[townNo].townVehicleCount + 1] = towns[townNo].Vehicles[j]; 
      towns[townNo].Vehicles[j] = towns[townNo].Vehicles[k]; 
      towns[townNo].Vehicles[k] = towns[townNo].Vehicles[towns[townNo].townVehicleCount + 1]; 
     } 
     if (identical == -1) { 
      towns[townNo].Vehicles[towns[townNo].townVehicleCount + 1] = towns[townNo].Vehicles[k]; 
      towns[townNo].Vehicles[k] = towns[townNo].Vehicles[j]; 
      towns[townNo].Vehicles[j] = towns[townNo].Vehicles[towns[townNo].townVehicleCount + 1]; 
     } 
    } 
}

источник

2015-11-05 Lukas Baranauskas

Что происходит? несортированное, неизмененное, исключение? –

@HenkHolterman он сортирует, но кажется случайным образом. здесь https://i.imgur.com/XC05W6t.jpg –

Если «города [городNo] .Vehicles» имеет тип «IEnumerable », вы можете просто позвонить в OrderBy (v => v.Brand) –

Я думаю, что здесь есть две критические проблемы.

for (int j = 0; j < towns[townNo].townVehicleCount-1; j++) { for (int k = 1; k < towns[townNo].townVehicleCount; k++) {

изменить его

for (int j = 0; j < towns[townNo].townVehicleCount; j++) { 
    for (int k = j+1; k < towns[townNo].townVehicleCount; k++) {

2. Теперь, почти в каждой строке языка программирования функция сравнения возвращает положительное или отрицательное значение в зависимости от сравнения. Итак, я думаю, вы должны изменить if (identical == 1) { и if (identical == -1) { линии, такие как if (identical > 0) { или if (identical < 0) { согласно вашей логике.

Надеюсь, это поможет вам немного.

источник

2015-11-05 13:03:33 Ultraviolet

Это то, что я получаю сейчас ... Solaris; MAN; Scania; Alfa; Jeep; Peugeot; Volvo; –

Ответ был очень простым!

«Кажется, вы всегда меняете элементы j и k, когда они разные. Удалите одно из операторов if, чтобы их менять, когда элементы не в правильном порядке».

Большое спасибо, @Henrik

источник

2015-11-05 13:15:42

Любые идеи, почему эта сортировка в алфавитном порядке не работает?

ответ

Смежные вопросы