优化一下这段代码void maxValue(std::vector<int> number,int &max, int &index) { int m; m = number[0]; for (int i = 0; i < number.size(); ++i) { if(i<number.size()-1) m = MAX(m, number[i+1]); else if(i==number.size()-1) break; } max = m; for(int i = 0; i < number.size(); ++i) { if(m == number[i]) { index = i; } } }

size){
int max = number[0];
for(int i=1;i<size;i++){
if(number[i]>max){
max = number[i];
}
}
std::cout<<"最大值为："<<max<<std::endl;
}

可以将循环条件改为 i < size，这样可以避免数组越界的问题。同时，可以将函数参数改为 const std::vector<int>& number，这样可以避免不必要的拷贝。修改后的代码如下：

void maxValue(const std::vector<int>& number, int size){
int max = number[0];
for(int i=1; i<size; i++){
if(number[i] > max){
max = number[i];
}
}
std::cout << "最大值为：" << max << std::endl;
}

相关问题

优化这段代码public class Program { public static void Main() { List<int> scores = new List<int> { 20, 20, 21, 22, 98, 39, 98, 98 }; string concatenated = string.Join("", scores.Select(s => s.ToString()).Distinct()); List<int> digits = concatenated.Select(c => int.Parse(c.ToString())).ToList(); digits.Sort(); digits.Reverse(); int maxValue = int.Parse(string.Join("", digits)); Console.WriteLine("最大值: " + maxValue); } }

你可以对代码进行如下优化：

using System;
using System.Collections.Generic;
using System.Linq;

public class Program
{
    public static void Main()
    {
        List<int> scores = new List<int> { 20, 20, 21, 22, 98, 39, 98, 98 };

        int maxValue = GenerateMaxValue(scores);

        Console.WriteLine("最大值: " + maxValue);
    }

    public static int GenerateMaxValue(List<int> scores)
    {
        List<int> digits = scores.Distinct()
                                .OrderByDescending(s => s)
                                .SelectMany(s => s.ToString())
                                .Select(c => int.Parse(c.ToString()))
                                .ToList();

        return int.Parse(string.Join("", digits));
    }
}

这段代码将生成最大值的逻辑封装在了一个新的方法 `GenerateMaxValue` 中。在这个方法中，首先使用 LINQ 操作符 `Distinct` 删除重复的数字，并使用 `OrderByDescending` 按降序对数字进行排序。然后，通过 `SelectMany` 将每个数字转换为字符数组，再使用 `Select` 将字符转换为整数，并将结果存储在 `digits` 列表中。最后，使用 `string.Join` 将数字连接起来，并通过 `int.Parse` 将其转换为整数返回。

这样做的好处是将代码分解成更小的可读性更高的部分，并且可以在需要生成最大值的其他地方重复使用 `GenerateMaxValue` 方法。

QList<int>获取次大值位置

要获取 QList<int> 中的次大值位置，你可以先找到最大值的位置和值，然后再遍历一遍 QList 找到第二大的值及其索引。以下是一个示例代码：

int maxIndex = 0;
int maxValue = list[0];
for(int i = 1; i < list.size(); i++) {
    if(list[i] > maxValue) {
        maxValue = list[i];
        maxIndex = i;
    }
}

int secondMaxIndex = 0;
int secondMaxValue = list[0];
for(int i = 1; i < list.size(); i++) {
    if(list[i] > secondMaxValue && i != maxIndex) {
        secondMaxValue = list[i];
        secondMaxIndex = i;
    }
}

// 次大值的位置为 secondMaxIndex

请注意，如果 QList 中有多个最大值，则此代码将返回第一个找到的次大值的位置。如果 QList 的大小小于 2，此代码将引发异常。