Ciao a tutti volevo sapere da voi cosa ne pensate di questo codice in c++,mi potete dare delle dritte per migliorare qualcosa grazie
#include <iostream>
using namespace std;
void inserisciDati(int& n,int& k);
float verificaPermutazione(int n,int k);
void stampaValori(float p);
int main(){
	int n,k;
	float perm;
	inserisciDati(n,k);
	perm = verificaPermutazione(n,k);
	stampaValori(perm);
	return 0;
}
void inserisciDati(int& n,int& k){
	cout<<"inserisci due numeri\n";
	cout<<"numero1:   ";
	cin>>n;
	cout<<"numero2:   ";
	cin>>k;
}
float verificaPermutazione(int n,int k){
	int d = 0;
	int f = 0;
	int c = n-k;
	if((n < 0) || (k < 0) || (k > n)){
		return 0;
	}else{
		for(int i = 1; i<n; i++){
			d += n * i;
		}
		for(int i = 1; i<c;i++){
			f += c * i;
		}	
	}
	return (float)d / f;
}
void stampaValori(float p){
	cout<<"le permutazioni possibili sono"<<" "<<p<<endl;
}