Ejercicio, una renta de automóviles desea un programa para emitir las facturas

#include <iostream>
using namespace std;
double totalPago(int numKilometros);
int main() {
int numKilometros;
cout << "Introduce el numero de Km que has utilizado: ";
cin >> numKilometros;
cout << "La Cantidad a pagar por sus Km es: "<< totalPago(numKilometros) <<"$";
return 0;
}
double totalPago(int numKilometros)
{
double pago;
int kmAdicionales;
if(numKilometros<=300)
{
pago = 1000;
}
if(numKilometros>300 || numKilometros<1000)
{
kmAdicionales = numKilometros - 300;
kmAdicionales = kmAdicionales % 10;
pago = 1000 +(kmAdicionales * 5);
}
if(numKilometros==1000)
{
pago = 4500;
}
if (numKilometros>1000)
{
kmAdicionales = numKilometros - 700;
kmAdicionales = kmAdicionales % 700;
pago = 1000 + (kmAdicionales*5);
kmAdicionales = kmAdicionales % 10;
pago = pago + (kmAdicionales*3);
}
return pago;
}

el programa tiene un error logico

primero se recomienda uso de if, else-if, else

segundo operadores logicos mal usados

Código

numKilometros>300 || numKilometros<1000

"si kilomentros mayor que 300 o kilometros menor que 1000", se puede ver como "se hará si 1 mayor que 300 o 1 menor que mil" como uno es menor que mil, se cumple la condición, las acotaciones deben cumplir ambas condiciones (and)

tercero error matemático

Código

kmAdicionales = kmAdicionales % 10;

esto carece de sentido... esto te retorna es simplemente el ultimo digito de los km adicionales... es decir si son 15km adicionales tendrá el mismo resultado que si son 565 kilometros adicionales... esta operación no tiene sentido aquí (ni en ningún punto de este ejercicio)

cuarto error de formulación

justamente el else-if te ayuda a acotar los rangos,

Código

if(numKilometros==1000)

no debe estar ahí porque no es una condición solicitada, se obtiene calculada

quinto error sintactico

Código

double totalPago(int numKilometros)

estás retornando un tipo double, pero tu parámetro de entrada es un entero y tus operaciones internas son solo entre enteros, por consecuencia toda operación interna tendrá como resultado enteros (incluso divisiones) por un asunto de promoción de operandos, a demás tienes enteros y solo se multiplican por enteros, el resultado nunca será decimal... el resultado carece de sentido como double

el algoritmo debe ser algo como
sopniendo el ejericio
"se venden caramelos en 1$, si son más de 5 caramelos en 0.95 cada uno y si son más de 10, en 0.9 cada uno"

Código

if(caramelos > 10){
  precio = caramelos*0.9;
}else if(caramelos > 5){
  precio = caramelos *0.95;
}else{
  precio = caramelos;
}

tu caso es el mismo solo que con operaciones adicionales