What is wrong with this code?

#include<windows.h>
#include "stdafx.h"

#define WeaponMgr 0xA287A8
#define WepWeight1 0x21DC


float weaponweight =0.0f;


DWORD WINAPI Hacks(LPVOID)
{
bool weaponweight = false;


while(1)
{
DWORD CShell = (DWORD)GetModuleHandleA("CShell.dll");
DWORD pWeaponMgr = *(DWORD*)(CShell + WeaponMgr);


if(GetAsyncKeyState(VK_F9) || GetAsyncKeyState(VK_NUMPAD0) &1) { weaponweight = true; Beep(512, 100); }



if(weaponweight)
{
if (pWeaponMgr)
{
for(int i=0; i<445; i++)
{
if((*(DWORD*)((*(DWORD*)(CShell+WeaponMgr))+(4*i)) ) != NULL)
*(float*)((*(DWORD*)((*(DWORD*)(CShell+WeaponMgr)) +(4*i))) + WepWeight1) = weaponweight;
}
}
}

}
}

bool Ready2Hook()
{
if(GetModuleHandleA("CShell.dll") != NULL
&& GetModuleHandleA("ClientFx.fxd") != NULL)
return 1;
return 0;
}

DWORD WINAPI Wait(LPVOID)
{
while(!Ready2Hook()) Sleep(200);
CreateThread(NULL, NULL, (LPTHREAD_START_ROUTINE)Hacks, NULL, NULL, NULL);
return 0;
}