編輯 | 言征

出品 | 51CTO技術(shù)棧（微信號：blog51cto）

近日，OpenAI一記重拳，推出了GPT-4o（“o”表示“omni”），將語音識別和對話方面的優(yōu)勢展示的淋漓盡致。幾乎可以肯定，它也是一個更強的LLM。但它能進行代碼審查嗎？本文通過一些測試來尋找答案。

1.長乘法：交給計算機就可以

首先，我檢查了它是否可以進行l(wèi)ong乘法運算：

圖片

答案是正確的，通過在線計算器上可以確認。但請注意，它并沒有試圖“計算出”結(jié)果——它只是將問題甩給了計算器。末尾的“查看分析”的藍色圖標基本上說明了這一點。這在之前的模型上都沒有做到。

2.代碼審查

我個人對正式嚴謹?shù)拇a審查沒有興趣。是的，有時初級開發(fā)人員可能確實需要這樣的審查，但單元測試和配對編程通常足以消除不良的編程習慣。但是，幾句含糊的評論可以幫助指出代碼中的潛在小問題，而不會顯得對容易重寫的代碼片段強加某種權(quán)威性的命令。

簡而言之，代碼審查更多的是一種social互動，而不是其他任何東西。請專注于工程問題，而不是代碼的語義細節(jié)。

然而，代碼審查是開發(fā)人員使用大型語言模型（LLM）的一個好案例。在這里，我將查看三段不同的代碼，每段代碼都只是繪制一個帶有對角線的矩形框。我將添加問題，并查看GPT-4o（通過ChatGPT）如何對它們進行審查。

接下來，我們從簡單的C#對角線繪制程序開始：

復制

using System;
 
public class Program
{
  public static void Main()
  {
    BoxWithDiagonalLine(5);
    BoxWithDiagonalLine(6,1);
    BoxWithDiagonalLine(6,-1);
    BoxWithDiagonalLine(2);
    BoxWithDiagonalLine(-1);
  }
 
  // Example 1
  public static void BoxWithDiagonalLine(int boxsize, int linepos = 0)
  {
    for(int i=0; i < boxsize; i++)
    {
      for (int j = 0; j < boxsize; j++)
      {
        if (j == linepos) Console.Write("x ");
        else Console.Write("o ");
      }
      linepos++;
      Console.Write("n");
    }
    Console.Write("n");
  }
}

將這個代碼丟進https://dotnetfiddle.net/（不過請將編譯器設置為至少.NET 6）并查看響應：

圖片

這段代碼相當簡單，也相對容易理解。外部循環(huán)控制行，內(nèi)部循環(huán)在正確的位置放置“x”。變量命名很簡潔，代碼很短。該方法有一個默認值，用于生成從左上角到右下角的完整對角線。雖然我們可以將起始線位置向左或向右移動，但擁有負的盒子大小（boxsize）并沒有多大意義。因此，我們可以通過將方法參數(shù)更改為無符號整數(shù)（unsigned int）來改進它。這將立即阻止上一個示例甚至無法編譯。一個非常大的linepos（起始線位置）將不可見，但它并不危險。

到目前為止，沒有什么可評論的。

接下來，將for循環(huán)替換成while循環(huán)：

復制

// Example 2
public static void BoxWithDiagonalLine(int boxsize, int linepos = 0)
{
  int i = 0;
  while(i++ < boxsize)
  {
    int j = 0;
    while (j++ < boxsize)
    {
      if (j == linepos + 1) Console.Write("x ");
        else Console.Write("o ");
    } linepos++;
    Console.Write("n");
  }
  Console.Write("n");
}

現(xiàn)在這段代碼可以運行，但值得稍微皺一下眉頭。通過在后置遞增循環(huán)變量（在i和j后面的++意味著值首先被讀取，然后再增加1）我們節(jié)省了一些空間，但我們需要那個笨拙的linepos + 1表達式。

while循環(huán)不如for循環(huán)那樣明確地表達開始和結(jié)束狀態(tài)——但我們并沒有陷入任何瘋狂的狀態(tài)。

我們進行最后一次更改。這次代碼中有一個變量來“搗亂”，它試圖使用linepos來控制外部循環(huán)：

復制

//Example 3
public static void BoxWithDiagonalLine(int boxsize, int linepos = 0)
{
  while(linepos < boxsize)
  {
    int j = 0;
    while (j++ < boxsize)
    {
      if (j == linepos + 1) Console.Write("x ");
        else Console.Write("o ");
    }
    linepos++;
    Console.Write("n");
  }
  Console.Write("n");
}

在試圖不必要地移除一個循環(huán)變量的過程中，矩形框本身將不再完整——盡管使用默認值時可以工作。事實上，這解決了另一個問題，即總是在底部角落可靠地以一個“×”結(jié)束。

圖片

這段糟糕的代碼可能是因為開發(fā)者不當?shù)乩^續(xù)使用了while循環(huán)。

現(xiàn)在讓我們總結(jié)一下我們?nèi)绾慰创@些不同版本：

第一個例子是可行的，但我們可以強制參數(shù)為正數(shù)。然而，沒有注釋的話，這段代碼的功能并不十分清晰。

第二個例子使用了while循環(huán)，它不像for循環(huán)那樣具有確定性。同時，由于后置遞增，還需要考慮一個丑陋的表達式。

第三個例子完全是錯誤的，盡管在默認情況下它得到了正確的結(jié)果。

我們對AI代碼審查有什么期望？首先，我沒有給出任何注釋，那么它能否弄清楚這段代碼的作用？只有方法的名稱揭示了這一點。

這是GPT-4o的回應：

復制

“Your code is functional and effectively generates a box with diagonal lines based on the provided parameters. However, there are a few improvements and edge cases to consider:
1.Input Validation: Ensure that the box size is positive and greater than 1, since a box with a size less than 2 doesn’t make much sense.
2.Handling linepos: Improve handling of diagonal positions to ensure the diagonals are correctly drawn when linepos is negative or greater than the box size.
3.Code Readability: Add comments and improve variable naming for better readability and maintenance.”

【譯】“您的代碼是功能性的，并且能夠有效地根據(jù)提供的參數(shù)生成帶有對角線的矩形框。但是，有幾個改進點和邊緣情況需要考慮：

1.輸入驗證：確保矩形框的大小是正數(shù)且大于1，因為大小小于2的矩形框并沒有太大意義。

2.處理linepos：改進對角線位置的處理，以確保當linepos為負數(shù)或大于矩形框大小時，對角線能夠正確繪制。

3.代碼可讀性：添加注釋并改進變量命名以提高代碼的可讀性和可維護性?！?/p>

所以它確實理解了代碼的作用——即使它認為程序畫的是“l(fā)ines”而不是“l(fā)ine”。這仍然很好。

它建議的代碼如下：

復制

// ChatGPT4o suggested code
public static void BoxWithDiagonalLine(int boxSize, int linePos = 0)
{
  if (boxSize < 2)
  {
    Console.WriteLine("Box size must be greater than 1.");
    return;
  }
 
  // Adjust linePos if it's out of the box boundaries
  if (linePos < 0) linePos = 0;
  if (linePos >= boxSize) linePos = boxSize - 1;
  for (int i = 0; i < boxSize; i++)
  {
    for (int j = 0; j < boxSize; j++)
    {
      if (j == linePos) Console.Write("x ");
        else Console.Write("o ");
    }
    linePos++;
 
    // Reset linePos if it goes out of the box boundaries
    if (linePos >= boxSize) linePos = 0;
    Console.Write("n");
}
Console.Write("n");

以下是將其放入dotnetfiddle.net的結(jié)果：

圖片

我認為大多數(shù)人都會對小于2的boxSize感到滿意。同時，它也很好地使用了駝峰命名法來命名參數(shù)，這在C#中是正確的。但遺憾的是，它忽略了改變位置以使線條從更低處開始的想法。而由于重置了linepos，我們在第二個例子的底部得到了那個糟糕的錯誤。

不過，從代碼審查的角度來看，它確實抓住了要點。

3.第二個例子結(jié)果

讓我們給它輸入第二個例子，看看它的表現(xiàn)如何。下面是一些有趣的評論：

“The use of while loops is functional but could be less intuitive compared to for loops for such iterations.”

Excellent. This is exactly the right note.

“The condition j == linepos + 1 is used to determine where to place “x”. This works but can be made more intuitive by aligning with the index i.”

再次強調(diào)，這是很好的。它理解了功能性和可讀性之間的區(qū)別。

生成的代碼限制了linepos的值，并且相當奇怪地使用了取模函數(shù)來改變這個值。但最終得到了相同的結(jié)果。它回歸到了for循環(huán)，這是很棒的。

4.第三個例子結(jié)果

最后，讓我們給出糟糕的例子3，看看它的表現(xiàn)如何。奇怪的是，它沒有發(fā)現(xiàn)這個錯誤，但提出了以下建議：

“Edge Cases:

– When linepos is negative, it works but may not be intended.”

這表明它完全沒有理解代碼——當然，我也沒有明確說明我在做什么。所以，也許“wrapping line ”可能是一個合理的猜測？

5.結(jié)論

總的來說，審查的性質(zhì)和結(jié)果給我留下了深刻的印象。GPT-4o生成的代碼雖然不太理想，但也還可以。在對代碼的批評中，沒有“書呆子的笨拙腳步”；相反，它輕輕地引導讀者走向更好的方向。

結(jié)果可能沒有達到預期——態(tài)度很好但代碼略有不準確。然而，新手應該會很喜歡；在這方面，它至少比部分人做得更好。

參考鏈接：??https://thenewstack.io/reviewing-code-with-gpt-4o-openais-new-omni-llm/??

本文轉(zhuǎn)載自??51CTO技術(shù)棧??，作者：言征