3 replies

TObjectIterators the speedy-er way

So as best I understand, in general using

for (TObjectIterator<UStaticMeshComponent> It; It; ++It)

for (TObjectIterator<UStaticMeshComponent> It; It; ++It)

is bad - it's not a good idea to iterate all the UStaticMeshComponents (in a fresh save there's about 10k of them) or whatever other component you want to iterate over, but sometimes it seems somewhat unavoidable for whatever reason. I just wanted to share the results of my profiling in case it helps

This simple approach takes between 5-7ms to run on my PC.

for (TObjectIterator<UStaticMeshComponent> It; It; ++It)
{
    UStaticMeshComponent* StaticMeshComponent = *It;
    if (StaticMeshComponent && StaticMeshComponent->GetWorld() == World && !StaticMeshComponent->ComponentTags.
        Contains(ResourceRouletteTag))
    {
        if (const UStaticMesh* StaticMesh = StaticMeshComponent->GetStaticMesh())
        {
            FName MeshPath = FName(*StaticMesh->GetPathName());
            if (MeshesToDestroy.Contains(MeshPath))
            {
                StaticMeshComponent->SetActive(false);
                StaticMeshComponent->SetVisibility(false);
                StaticMeshComponent->DestroyComponent();
            }
        }
    }
}

for (TObjectIterator<UStaticMeshComponent> It; It; ++It)
{
    UStaticMeshComponent* StaticMeshComponent = *It;
    if (StaticMeshComponent && StaticMeshComponent->GetWorld() == World && !StaticMeshComponent->ComponentTags.
        Contains(ResourceRouletteTag))
    {
        if (const UStaticMesh* StaticMesh = StaticMeshComponent->GetStaticMesh())
        {
            FName MeshPath = FName(*StaticMesh->GetPathName());
            if (MeshesToDestroy.Contains(MeshPath))
            {
                StaticMeshComponent->SetActive(false);
                StaticMeshComponent->SetVisibility(false);
                StaticMeshComponent->DestroyComponent();
            }
        }
    }
}

TObjectIterators the speedy-er way

So as best I understand, in general using

for (TObjectIterator<UStaticMeshComponent> It; It; ++It)

for (TObjectIterator<UStaticMeshComponent> It; It; ++It)

for (TObjectIterator<UStaticMeshComponent> It; It; ++It)
{
    UStaticMeshComponent* StaticMeshComponent = *It;
    if (StaticMeshComponent && StaticMeshComponent->GetWorld() == World && !StaticMeshComponent->ComponentTags.
        Contains(ResourceRouletteTag))
    {
        if (const UStaticMesh* StaticMesh = StaticMeshComponent->GetStaticMesh())
        {
            FName MeshPath = FName(*StaticMesh->GetPathName());
            if (MeshesToDestroy.Contains(MeshPath))
            {
                StaticMeshComponent->SetActive(false);
                StaticMeshComponent->SetVisibility(false);
                StaticMeshComponent->DestroyComponent();
            }
        }
    }
}

for (TObjectIterator<UStaticMeshComponent> It; It; ++It)
{
    UStaticMeshComponent* StaticMeshComponent = *It;
    if (StaticMeshComponent && StaticMeshComponent->GetWorld() == World && !StaticMeshComponent->ComponentTags.
        Contains(ResourceRouletteTag))
    {
        if (const UStaticMesh* StaticMesh = StaticMeshComponent->GetStaticMesh())
        {
            FName MeshPath = FName(*StaticMesh->GetPathName());
            if (MeshesToDestroy.Contains(MeshPath))
            {
                StaticMeshComponent->SetActive(false);
                StaticMeshComponent->SetVisibility(false);
                StaticMeshComponent->DestroyComponent();
            }
        }
    }
}

TObjectIterators the speedy-er way

Similar Threads

TObjectIterators the speedy-er way

Similar Threads

Similar Threads

Similar Threads