I'll need quite some help polishing my game

Unreal Courses Ask
,
#21

Done and done. However, there’s still a problem. I apologize for taking long to reply but lately, I was having schedule issues and paranoia(concerning COPPA).

2019-12-02
2019-12-021359×610 125 KB

Yeah… This is what got me stumped.

#22

Sounds like you have two contructors instead of one and missing () in places for function calls.

#23

2019-12-04
2019-12-041359×610 114 KB

I think I’m almost done… But not too sure. I tried doing what the errors and warnings are telling me, but I’m definitely missing something on lines 28 and 37.

#24

That would be missing ()'s i.e.

FindComponentByClass<SomeType>();
#25

I’ve done it! …Or so I thought.

2019-12-05
2019-12-051372×610 91.8 KB

2019-12-05%20(1)
2019-12-05%20(1)1372×610 67.5 KB

I have no idea what to do now. I tried editing to see the full information, but the file type isn’t authorized for that either.
It may have something to do with this, though.
2019-12-05%20(2)
2019-12-05%20(2)1382×744 62.4 KB

#26

The header file?

#27

Right here.

2019-12-05%20(3)
2019-12-05%20(3)1382×744 62.1 KB

#28

Line 15 is the problem, you declared that but haven’t defined it anywhere. Remove it.

#29

And now I have pretty much finished. Now I just need to do all of that jazz in the previous videos.

#30

Well, I said pretty much, but I can’t move objects or trigger the TriggerVolume to open the doors. That right there is a serious issue.

#31

What does your code look like now?

#32

PositionReporter.h

// Copyright Project Jector

#pragma once

#include "CoreMinimal.h"
#include "Components/ActorComponent.h"
#include "PositionReporter.generated.h"

UCLASS( ClassGroup=(Custom), meta=(BlueprintSpawnableComponent) )
class GETOUTTAHEREQUICKLY_API UPositionReporter : public UActorComponent
{
	GENERATED_BODY()

public:	

protected:
	virtual void BeginPlay() override;

private:	
	virtual void TickComponent(float DeltaTime, ELevelTick TickType, FActorComponentTickFunction* ThisTickFunction) override;

		
};

PositionReporter.cpp

// Copyright Project Jector

#include "PositionReporter.h"
#include "GetOuttaHereQuickly.h"
#include "GameFramework/Actor.h"

// Called when the game starts
void UPositionReporter::BeginPlay()
{
	Super::BeginPlay();
	FString ObjectName = GetOwner()->GetName();
	FString ObjectPos = GetOwner()->GetTransform().GetLocation().ToString();
	UE_LOG(LogTemp, Warning, TEXT("%s is at %s"), *ObjectName, *ObjectPos)
}


// Called every frame
void UPositionReporter::TickComponent(float DeltaTime, ELevelTick TickType, FActorComponentTickFunction* ThisTickFunction)
{
	Super::TickComponent(DeltaTime, TickType, ThisTickFunction);

	// ...
}

OpenDoor.h

// Copyright Project Jector

#pragma once

#include "CoreMinimal.h"
#include "Components/ActorComponent.h"
#include "Engine/TriggerVolume.h"
#include "Engine/World.h"
#include "OpenDoor.generated.h"

#define OUT

DECLARE_DYNAMIC_MULTICAST_DELEGATE(FDoorEvent);

UCLASS( ClassGroup=(Custom), meta=(BlueprintSpawnableComponent) )
class GETOUTTAHEREQUICKLY_API UOpenDoor : public UActorComponent
{
	GENERATED_BODY()

public:	
	// Sets default values for this component's properties
	UOpenDoor();

protected:
	// Called when the game starts
	virtual void BeginPlay() override;

public:	
	// Called every frame
	virtual void TickComponent(float DeltaTime, ELevelTick TickType, FActorComponentTickFunction* ThisTickFunction) override;

	UPROPERTY(BlueprintAssignable)
	FDoorEvent OnOpen;

	UPROPERTY(BlueprintAssignable)
	FDoorEvent OnClose;

private:
	UPROPERTY(EditAnywhere)
	ATriggerVolume* PressurePlate = nullptr;

	UPROPERTY(EditAnywhere)
	float TriggerMass = 30.f;

	// The owning door
	AActor* Owner = nullptr; 

	// Returns total mass in kg
	float GetTotalMassOfActorsOnPlate();
};

OpenDoor.cpp

// Copyright Project Jector

#include "OpenDoor.h"
#include "GetOuttaHereQuickly.h"
#include "Components/PrimitiveComponent.h"
#include "GameFramework/Actor.h"

// Sets default values for this component's properties
UOpenDoor::UOpenDoor()
{
	PrimaryComponentTick.bCanEverTick = true;

	// ...
}

// Called when the game starts
void UOpenDoor::BeginPlay()
{
	Super::BeginPlay();
	Owner = GetOwner();
	if (!PressurePlate)
	{
		UE_LOG(LogTemp, Error, TEXT("%s missing pressure plate"), *GetOwner()->GetName())
	}
}

// Called every frame
void UOpenDoor::TickComponent(float DeltaTime, ELevelTick TickType, FActorComponentTickFunction* ThisTickFunction)
{
	Super::TickComponent(DeltaTime, TickType, ThisTickFunction);

	// Poll the Trigger Volume
	if (GetTotalMassOfActorsOnPlate() > TriggerMass)
	{ 
		OnOpen.Broadcast();
	}
	else
	{
		OnClose.Broadcast();
	}
}

float UOpenDoor::GetTotalMassOfActorsOnPlate()
{
	float TotalMass = 0.f;

	// Find all the overlapping actors
	if (!PressurePlate) { return TotalMass; }
	TArray<AActor*> OverlappingActors;
	PressurePlate->GetOverlappingActors(OUT OverlappingActors);

	// Iterate through them adding through their masses
	for (const auto* AActor : OverlappingActors)
	{

		TotalMass += AActor->FindComponentByClass<UPrimitiveComponent>()->GetMass();
		UE_LOG(LogTemp, Warning, TEXT("%s on pressure plate"), *AActor->GetName())
	}

	return TotalMass;
}

Grabber.h

// Copyright Project Jector

#pragma once

#include "CoreMinimal.h"
#include "Components/ActorComponent.h"
#include "PhysicsEngine/PhysicsHandleComponent.h"
#include "Components/InputComponent.h"
#include "GameFramework/PlayerController.h"
#include "Engine/World.h"
#include "Components/PrimitiveComponent.h"
#include "Grabber.generated.h"


UCLASS( ClassGroup=(Custom), meta=(BlueprintSpawnableComponent) )
class GETOUTTAHEREQUICKLY_API UGrabber : public UActorComponent
{
	GENERATED_BODY()

public:	
	// Sets default values for this component's properties
	UGrabber();

protected:
	// Called when the game starts
	virtual void BeginPlay() override;

public:	
	// Called every frame
	virtual void TickComponent(float DeltaTime, ELevelTick TickType, FActorComponentTickFunction* ThisTickFunction) override;

private:
	// How far ahead of the player can we reach in cm
	float Reach = 100.f;

	UPhysicsHandleComponent* PhysicsHandle = nullptr;

	UInputComponent* InputComponent = nullptr;

	// Ray-cast and grab what's in reach
	void Grab();

	// Call when grab is released
	void Release();

	// Find (assumed) attached physics handle
	void FindPhysicsHandleComponent();

	// Setup (assumed) attached input component
	void SetupInputComponent();

	// Return hit for first physics body in reach
	const FHitResult GetFirstPhysicsBodyInReach();

	// Returns current start of reach line
	FVector GetReachLineStart();

	// Returns current end of reach line
	FVector GetReachLineEnd();
};

Grabber.cpp

// Copyright Project Jector

#include "GetOuttaHereQuickly.h"
#include "Grabber.h"

#define OUT

// Sets default values for this component's properties
UGrabber::UGrabber()
{
	// Set this component to be initialized when the game starts, and to be ticked every frame.  You can turn these features
	// off to improve performance if you don't need them.
	bWantsBeginPlay = true;
	PrimaryComponentTick.bCanEverTick = true;
}

// Called when the game starts
void UGrabber::BeginPlay()
{
	Super::BeginPlay();
	FindPhysicsHandleComponent();
	SetupInputComponent();
}

/// Look for attached Physics Handle
void UGrabber::FindPhysicsHandleComponent()
{
	PhysicsHandle = GetOwner()->FindComponentByClass<UPhysicsHandleComponent>();
	if (PhysicsHandle == nullptr)
	{
		UE_LOG(LogTemp, Error, TEXT("%s missing physics handle component"), *GetOwner()->GetName())
	}
}

void UGrabber::SetupInputComponent()
{
	InputComponent = GetOwner()->FindComponentByClass<UInputComponent>();
	if (InputComponent)
	{
		InputComponent->BindAction("Grab", IE_Pressed, this, &UGrabber::Grab);
		InputComponent->BindAction("Grab", IE_Released, this, &UGrabber::Release);
	}
	else
	{
		UE_LOG(LogTemp, Error, TEXT("%s missing input component"), *GetOwner()->GetName());
	}
}

// Called every frame
void UGrabber::TickComponent(float DeltaTime, ELevelTick TickType, FActorComponentTickFunction* ThisTickFunction)
{
	Super::TickComponent(DeltaTime, TickType, ThisTickFunction);

	if (!PhysicsHandle) { return; }
	// if the physics handle is attached
	if (PhysicsHandle->GrabbedComponent)
	{
		// move the object that we're holding
		PhysicsHandle->SetTargetLocation(GetReachLineEnd());
	}
}

void UGrabber::Grab() {
	/// LINE TRACE and see if we can reach any actors with physics body collision channel set
	auto HitResult = GetFirstPhysicsBodyInReach();
	auto ComponentToGrab = HitResult.GetComponent(); // gets the mesh in our case
	auto ActorHit = HitResult.GetActor();

	/// If we hit something then attach a physics handle
	if (ActorHit)
	{
		if (!PhysicsHandle) { return; }
		PhysicsHandle->GrabComponent(
			ComponentToGrab,
			NAME_None, // No bones needed
			ComponentToGrab->GetOwner()->GetActorLocation(),
			true // allow rotation
		);
	}
}

void UGrabber::Release()
{
	if (!PhysicsHandle) { return; }
	PhysicsHandle->ReleaseComponent();
}

const FHitResult UGrabber::GetFirstPhysicsBodyInReach()
{
	/// Line-trace (AKA ray-cast) out to reach distance
	FHitResult HitResult;
	FCollisionQueryParams TraceParameters(FName(TEXT("")), false, GetOwner());
	GetWorld()->LineTraceSingleByObjectType(
		OUT HitResult,
		GetReachLineStart(),
		GetReachLineEnd(),
		FCollisionObjectQueryParams(ECollisionChannel::ECC_PhysicsBody),
		TraceParameters
	);
	return HitResult;
}

FVector UGrabber::GetReachLineStart()
{
	FVector PlayerViewPointLocation;
	FRotator PlayerViewPointRotation;
	GetWorld()->GetFirstPlayerController()->GetPlayerViewPoint(
		OUT PlayerViewPointLocation,
		OUT PlayerViewPointRotation
	);
	return PlayerViewPointLocation;
}

FVector UGrabber::GetReachLineEnd()
{
	FVector PlayerViewPointLocation;
	FRotator PlayerViewPointRotation;
	GetWorld()->GetFirstPlayerController()->GetPlayerViewPoint(
		OUT PlayerViewPointLocation,
		OUT PlayerViewPointRotation
	);
	return PlayerViewPointLocation + PlayerViewPointRotation.Vector() * Reach;
}
#33

Well I’m not seeing problems in the code. Have you tried adding logs to see where the problem lies? For example

  • Is GetTotalMassOfActorsOnPlate even being called?
  • What values is it returning?
#34

Hello again! I’m back after all this time and will be active for eight hours a day from any range from 6 o’ clock MST through 14 o’clock MST and 12 o’clock MST through 20 o’clock MST. I’m running way behind, so I must catch up.

Now, back to where I was.

It must’ve been a while, because I tried typing “15” in the parentheses and I got an error. I think I need to look at one of the previous videos if I could ever find it.

#35

Hang on. I think = 15.f will work outside the parentheses.

#36

Except now I have two errors about GetTotalMassOfActorsOnPlate in OpenDoor.cpp. (By the way, sorry for spamming a bit. I live in America.)

(Actually, I’ll just edit from here.)

I replaced TriggerVolume with 50.f and put the rest back. Yet, it fails to compile because of this?!

2020-01-06 (1)
2020-01-06 (1)1382×744 58.8 KB

2020-01-06
2020-01-061372×610 70.2 KB

Then I realized, I actually did do it twice. However, I did notice that the objects won’t move even if I press Shift, and the TriggerVolume won’t activate when the DefaultPawn(Or rather the DefaultPawn_Blueprint) at least gets in the TriggerVolume.

#37

Did you make sure the objects were set to PhysicsActors for their collision preset?

#38

Well, they were PhysicsActor. What about it?

#39

Just that they needed to be. Is the line trace succeeding?

#40

During this time, I’m at Starbucks looking for help from time to time, and so far no game developers.

But back on topic.

I don’t see the circles transporting to their direct paths.

It may be because SetActorTransform isn’t connected to anything.

Privacy & Terms