1

最も近い 2 つのベクトルを見つけて合計を返すクラスを作成しようとしています。

一生懸命理解しようとしましたが、このメッセージが表示される理由がわかりません。これが唯一のエラーです。

java:93: 互換性のない型が見つかりました: void が必要です: EDU.gatech.cc.is.util.Vec2 結果 = one.add(two); ^

行 93 はコードの最後にあり、それを示すためにいくつかの矢印を付けました!

enter code here


package EDU.gatech.cc.is.clay;


import java.util.*;
import EDU.gatech.cc.is.clay.*;
import java.lang.*;
import EDU.gatech.cc.is.abstractrobot.*;
import EDU.gatech.cc.is.util.Vec2;
import EDU.gatech.cc.is.util.Units;


public class MAX_go_in_between extends NodeVec2
{
  public static final boolean DEBUG = /*true;*/ Node.DEBUG;
  private SocSmall abstract_robot;


  public MAX_go_in_between(SocSmall ar)
  {
    abstract_robot = ar;


  }

  long last_spott = 0;
  Vec2 result = new Vec2();



  public Vec2 Value(long timestamp)
  {


    if (DEBUG) System.out.println("MAX_Avoid_walls: Value()");

    if ((timestamp > last_spott) || (timestamp == -1))
    {
      if (timestamp != -1) last_spott = timestamp;


      Vec2 one;
      Vec2 two;

      //array of Vec2 of all the opponents
      Vec2[] list_opp = abstract_robot.getOpponents(timestamp);
      //empty array of vec2 where will be put the opponents in front of the robot
      ArrayList<Vec2> list_opp_in_front;

      Vec2 temp;


      // find which opponents are in front and put them in the arraylist
      for(int i=0; i<list_opp.length; i++)
      {
        temp = list_opp[i];

        if(temp.x >= 0.0)
        {
          list_opp_in_front.add(temp);
        }
      }

      //get closest opponent and sets it to index 0
      for(int i=1; i<list_opp_in_front.size()-1; i++)
      {
        temp = list_opp_in_front.get(i);

          if(list_opp_in_front.get(0).r<temp.r)
        {
          list_opp_in_front.set(i, list_opp_in_front.get(0));
          list_opp_in_front.set(0, temp);

        }
      }

      //get second closest opponent and sets it to index 1
      for(int i=2; i<list_opp_in_front.size()-1; i++)
      {
        temp = list_opp_in_front.get(i);

          if(list_opp_in_front.get(1).r<temp.r)
        {
          list_opp_in_front.set(i, list_opp_in_front.get(1));
          list_opp_in_front.set(1, temp);
        }

          // sum both vectors
          one = list_opp_in_front.get(0);
          two = list_opp_in_front.get(1);

 =============>>>>
 =============>>>>   result = one.add(two);
          }

      }

      return(result);
    }

  }



Here is the Vec2.add(Vec2) method:


 public void add(Vec2 other)
  {
  x = x + other.x;
  y = y + other.y;
  r = Math.sqrt(x*x + y*y);
  if (r > 0)
   t = Math.atan2(y,x);
  }
4

2 に答える 2

4
result = one.add (two);
public void add (Vec2 other)
//     ^^^^

このことから、メンバー関数addは に入れることができるものを何も返しませんresult。次のような行で:

x = x + other.x;

(xは「現在のオブジェクト」のメンバーであり、はそれに追加するオブジェクトです)、計算で使用するだけでなく、変更することを意図したother完全な確実性です。one.Add (two) one

したがって、ではなく:

one = list_opp_in_front.get (0);
two = list_opp_in_front.get (1);
result = one.add (two);

おそらく次のようなものが必要になるでしょう:

result = list_opp_in_front.get (0);
two = list_opp_in_front.get (1);
result.add (two);
于 2012-10-16T03:42:47.677 に答える
0

メソッド宣言に従って、public void add(Vec2 other)に追加twoしてoneいます。したがってone、それ自体があなたの結果なので、返す必要はありません。

return ステートメントを削除して、one結果オブジェクトとして扱います。

于 2012-10-16T03:48:29.717 に答える