字谜检查器没有通过所有测试用例？

Question

我正在做一个在线代码挑战，要求编写一个字谜检查程序。所以我做了以下程序，由于一些奇怪的原因它没有通过所有的测试用例。我自己编写了一个 main 函数并尝试自己测试它，它似乎工作正常。

#include<iostream>


using namespace std;

bool anagrams(string str1, string str2);

int main(){
    string str1="abcd";
    string str2="dcba";

    cout<<anagrams(str1,str2)<<endl;
    cin.get();
    return 0;    
}

bool anagrams(string str1, string str2){
    if(str1.length()!=str2.length()) return false;
    for(int i=0;i <str1.length(); i++){
        for(int j=0;j<str2.length();j++){
            if(str1[i]==str2[j]){
                str1[i]='*';
                break;
            }
        }    
    }
    for(int i=0;i<str1.length();i++){
        if(str1[i]!='*') return false;
    }
    return true;
}

Answer 1

您的代码似乎在检查 str1 中的字符是否存在于 str2 中。

例如，如果 str1="abca" 和 str2="abcd" 它们显然不是字谜。
但是它不会将 str1 变成“****”，因为它会将 str1 中的最后一个 a 与 str2 中的第一个 a 匹配（两次）。

Answer 2

好的，所以程序不能处理诸如

之类的输入

abcc
abcd

因此，当您遍历str1 中的每个字符时，快速解决方法是将str2 中的字符标记为*。

#include<iostream>


using namespace std;

bool anagrams(string str1, string str2);

int main(){
    string str1="abcc";
    string str2="dcba";

    cout<<anagrams(str1,str2)<<endl;
    cin.get();
    return 0;    
}

bool anagrams(string str1, string str2){
    if(str1.length()!=str2.length()) return false;
    for(int i=0;i <str1.length(); i++){
        for(int j=0;j<str2.length();j++){
            if(str1[i]==str2[j]){
                str2[j]='*';
                break;
            }
        }    
    }
    for(int i=0;i<str2.length();i++){
        if(str2[i]!='*') return false;
    }
    return true;
}

对于字谜检查器有更好的算法，例如排序和使用哈希。

Answer 3

package com.datastructures.programs;

public class Anagram {

    public static void main(String[] args) {
        // TODO Auto-generated method stub

        String s1 = "abcc";
        String s2 = "abcd";

        System.out.println(Anagram.isAnagram(s1, s2));
    }

    private static boolean isAnagram(String s1, String s2) {
        // TODO Auto-generated method stub

        int[] temp = new int[256];
        boolean isAnagram = true;
        if (s1.length() != s2.length()) {
            return false;
        }

        for (char s : s1.toCharArray()) {
            int index = (int) s;
            temp[index]++;
        }
        for (char s : s2.toCharArray()) {
            int index = (int) s;
            temp[index]--;
        }
        for (int i = 0; i < 256; i++) {
            if (temp[i] != 0) {
                return false;
            }
        }
        return true;
    }

}