Java到C++转换代码

Question

我有一个 Java 代码；这是我的 Poin Java 代码:

import java.util.Scanner;

public class Poin {
    private int X;
    private int Y;

    public Poin() {
        X = 0;
        Y = 0;
    }

    public Poin(int X, int Y) {
        this.X = X;
        this.Y = Y;
    }

    public int getX() {
        return X;
    }
    public int getY() {
        return Y;
    }
    public boolean InRect(Poin TopLeft, Poin BottomRight) {
        if (this.X < BottomRight.getX() && this.X > TopLeft.getX()
                && this.Y < BottomRight.getY() && this.Y > TopLeft.getY()) {
            return true;
        } else {
            return false;
        }
    }

    public static void main(String [] args) {
        int N;
        int i;
        int x,y;
        int count = 0;
        Poin TopLeft, BottomRight;

        Scanner sc = new Scanner(System.in);
        N = sc.nextInt();

        Poin[] a = new Poin[N];
        int x_top = sc.nextInt();
        int y_top = sc.nextInt();
        int x_bot = sc.nextInt();
        int y_bot = sc.nextInt();
        TopLeft = new Poin(x_top, y_top);
        BottomRight = new Poin(x_bot, y_bot);
        for (i = 0; i < N; i++) {
            x = sc.nextInt();
            y = sc.nextInt();
            Poin p = new Poin(x, y);
            a[i] = p;
            if (p.InRect(TopLeft, BottomRight)) {
                count += 1;
            }
        }
        System.out.println(count);
        for (i = 0; i < N; i++) {
            System.out.println(a[N-1-i].getX()+","+a[N-1-i].getY());
        }
    }
}

我想将其转换为 C++。这是我的 C++ 代码:

#include <iostream>
#include <cstdlib>

using namespace std;
class Poin
{
    private :
    int x;
    int y;
    public :
    Poin() {
        x = 0;
        y = 0;
    }

    Poin(int x, int y) {
        this->x = x;
        this->y = y;
    }   

    int get_x() {
        return x;
    }

    int get_y() {
        return y;
    }

    bool InRect(Poin& TopLeft, Poin& BottomRight) {
        if (this->x < BottomRight.get_x() && this->x > TopLeft.get_x() && this->y < BottomRight.get_y() && this->y > TopLeft.get_y()) 
        {
            return true;
        } 
        else 
        {
            return false;
        }
    }
};

int main ()
{
    int N;
    int i;
    int x,y;
    int count = 0;
    int x_top;
    int y_top;
    int x_bot;
    int y_bot; 
    Poin TopLeft, BottomRight;


    Poin** a = new Poin*[N];
    cin>>x_top;
    cin>>y_top;
    cin>>x_bot;
    cin>>y_bot;
    TopLeft = new Poin(x_top, y_top);
    BottomRight = new Poin(x_bot, y_bot);
    for (i = 0; i < N; i++) {
        x = cin>>x;
        y = cin>>y;
        px = p.poin(x, y);
        a[i] = p;
        if (px.InRect(TopLeft, BottomRight)) {
            count += 1;
        }
    }
    cout<<count;
    for (i = 0; i < N; i++) {
        cout << a[N-1-i].getX() << "," << a[N-1-i].getY();
    }   
    return 0;
}

我很难将此代码从 Java 转换为 C++。 Poin 是一个类，在主程序中使用它作为参数:

Poin TopLeft, BottomRight;
TopLeft = new Poin(x_top, y_top);
BottomRight = new Poin(x_bot, y_bot);

有人有办法解决我的问题吗？

Answer 1

虽然您已经获得了一些关于您所看到的表面问题的建议，但遵循该建议(本身)的结果仍然是我认为糟糕的 C++ 代码。到目前为止，您基本上是从 Java 音译到 C++，对让 C++ 编译器接受仍然基本上是 Java 代码的语法进行大致必要的最小更改。

相反，我建议编写真正的 C++。它将与您的 Java 代码完全不同，因为从根本上说，这两种语言确实是完全不同的语言，并且使用 C++(完全)编写的代码与使用 Java 完成相同工作的代码明显不同。

让我们考虑一下您的代码真正做的事情的基础，并编写一些 C++ 来做这件事。

1. 从文件中读入点(每个点都是一对int)
2. 计算矩形内的点数
3. 打印出计数
4. 以相反的顺序打印出点

我们希望编写代码以尽可能干净地支持它，所以让我们从为作业正确定义的 point 类开始。必要的能力是:

1. 从整数构造。
2. 从流中读取
3. 写入流
4. 检查是否在矩形中

这应该就是它所需要的，所以让我们来实现它:

class Poin {
    int x;
    int y;
public:
    // construct a point from two ints. Note that we prefer to use the member 
    // initialization list over assigning inside the body of the ctor
    Poin(int x=0, int y=0) : x(x), y(y) {}

    bool operator<(Poin const &other) const {
        return x < other.x && y < other.y;
    }

    bool in_rect(Poin const &TL, Poin const &BR) const {
        return (*this < BR) && (TL < *this);
    }

    // read a Poin from a stream:    
    friend std::istream &operator>>(std::istream &is, Poin &p) {
        return is >> p.x >> p.y;
    }

    // display a Poin on a stream. Note: operator>> won't read what this writes
    friend std::ostream &operator<<(std::ostream &os, Poin const &p) {
        return os << p.x << "," << p.y;
    }
};

注意几点(请原谅双关语):我们没有/不需要/想要 getX 或 getY 或任何类似的东西。 point 知道如何处理它自己的 I/O，因此外部代码只读取或写入 point —— 它不必处理 x 和 y 坐标构成一个点。同样，一个点“知道”如何将自己与另一个点进行比较(以一种适合我们需要的方式——当然不是唯一可能的定义方式) .然后我们使用它来获得 in_rect 的一个相当简单的定义。

使用它，连同标准库中的算法、迭代器和容器，我们的 main 变得彻底更简单和更清晰:

int main() {
    // read points defining the rectangle from standard input:
    Poin TL, BR;
    std::cin >> TL >> BR;

    // read points from standard input to initialize collection:    
    std::vector<Poin> points{ 
        std::istream_iterator<Poin>(std::cin),
        std::istream_iterator<Poin>() };

    // display count of points inside rectangle:
    std::cout << "Count: "
              << std::count_if(points.begin(), points.end(), 
                               [&](Poin const &p) { return p.in_rect(TL, BR); })
              << "\n";

    // display all points in reverse order:
    std::copy(points.rbegin(), points.rend(),
        std::ostream_iterator<Poin>(std::cout, "\n"));
}

我不得不努力工作以抵制阐述这段代码的优越性和简单性的诱惑，以及相对于 Java 等低级语言的优势，但我想我会让代码自己说明这一点主题。

尽管如此，我还是要添加最后一点:这段代码是用 C++11 编写的。如果您使用的是较旧的编译器，您可能会遇到一些问题。两个明显的是 points 的初始化——对于较旧的编译器，您可能需要稍微更改语法，以:

std::vector<Poin> points( 
    (std::istream_iterator<Poin>(std::cin)),
    std::istream_iterator<Poin>());

您遇到的另一个明显问题是 lambda 表达式的使用:

[&](Poin const &p) { return p.in_rect(TL, BR); }

这也可以解决，但如果您遇到任何问题，我建议您更新编译器而不是修改代码。这些是在 C++11 中引入的，并且(许多)编译器甚至在标准最终确定之前就支持它们。反其道而行之，这是一个与其争斗不如转换的地方。